Skip to content

ENG-115918: Use Retry-After header for rate limits, add retry to all …#80

Merged
dmeenaarmorcode merged 1 commit into
mainfrom
qa
Apr 10, 2026
Merged

ENG-115918: Use Retry-After header for rate limits, add retry to all …#80
dmeenaarmorcode merged 1 commit into
mainfrom
qa

Conversation

@dmeenaarmorcode
Copy link
Copy Markdown
Collaborator

…AC endpoints (#79)

  • Add retry_request() unified retry wrapper: 429 uses X-Rate-Limit-Retry-After-Seconds header (cap 300s), 5XX uses exponential backoff (5s base, 120s cap + jitter)
  • Add is_concurrent_limit_error() and get_retry_delay() helpers
  • Fix process() get-task: add 429 handling, fix '>500' to '>=500'
  • Refactor update_task(): replace recursive retry + fixed 2s sleep with retry_request()
  • Add retry to upload_response() (upload-result): was zero retry
  • Add retry to check_for_logs_fetch() (upload-logs): was zero retry
  • Add retry to get_s3_upload_url() (upload-url): was zero retry
  • Add retry to upload_s3() (S3 PUT): 5XX backoff only (max_retries=0)

…AC endpoints (#79)

- Add retry_request() unified retry wrapper: 429 uses X-Rate-Limit-Retry-After-Seconds
  header (cap 300s), 5XX uses exponential backoff (5s base, 120s cap + jitter)
- Add is_concurrent_limit_error() and get_retry_delay() helpers
- Fix process() get-task: add 429 handling, fix '>500' to '>=500'
- Refactor update_task(): replace recursive retry + fixed 2s sleep with retry_request()
- Add retry to upload_response() (upload-result): was zero retry
- Add retry to check_for_logs_fetch() (upload-logs): was zero retry
- Add retry to get_s3_upload_url() (upload-url): was zero retry
- Add retry to upload_s3() (S3 PUT): 5XX backoff only (max_retries=0)
@dmeenaarmorcode dmeenaarmorcode merged commit bcccacb into main Apr 10, 2026
4 checks passed
@dmeenaarmorcode dmeenaarmorcode deleted the qa branch April 10, 2026 16:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants