Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Retry when server reports 429 Too Many Requests occurs #220

Merged
merged 1 commit into from
Aug 3, 2023

Conversation

rickysarraf
Copy link
Owner

An HTTP/1.1 429 Too Many Requests occurs when:

  • there are too many requests
  • server is overloaded

The server may respond back with a Retry-after field with the value in seconds. For now, we'll just retry immediately. A potential enhancement could be to honor Retry-after but, thinking of it, it doesn't fit the bill for apt-offline because we want the download operation to conclude immediately rather than wait for Retry-after seconds.

Thanks: Zoltan Kelemen (Github: misterzed88)
Closes: #213

An HTTP/1.1 429 Too Many Requests occurs when:

* there are too many requests
* server is overloaded

The server may respond back with a `Retry-after` field with the value in
seconds. For now, we'll just retry immediately. A potential enhancement
could be to honor `Retry-after` but, thinking of it, it doesn't fit the
bill for `apt-offline` because we want the download operation to
conclude immediately rather than wait for `Retry-after` seconds.

Thanks: Zoltan Kelemen (Github: misterzed88)
Closes: #213
@rickysarraf rickysarraf self-assigned this Aug 3, 2023
@rickysarraf rickysarraf merged commit a6cc640 into master Aug 3, 2023
1 check passed
@rickysarraf rickysarraf deleted the wip/ritesh/429-retry.master branch August 3, 2023 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Problems with large download due to HTTP error 429
1 participant