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

Resiliency: respect "Retry-After" delays set in 429 responses #5872

Open
ItalyPaleAle opened this issue Feb 1, 2023 · 3 comments
Open

Resiliency: respect "Retry-After" delays set in 429 responses #5872

ItalyPaleAle opened this issue Feb 1, 2023 · 3 comments

Comments

@ItalyPaleAle
Copy link
Contributor

In what area(s)?

/area runtime

Describe the feature

(Spin-off from #5862)

When resiliency is enabled and has retries configured, if the endpoint responds with a 429 ("too many requests") status code and it contains a Retry-After header, it should take that into account when deciding the interval to retry.

Assuming both a delay is set in the retry policy, and a Retry-After header is present, the amount of time to wait should be the maximum of the two.

Release Note

RELEASE NOTE: ADD Respect "Retry-After" headers in 429 responses when retrying a request with Resiliency

@dapr-bot
Copy link
Collaborator

This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed in the next 7 days unless it is tagged (pinned, good first issue, help wanted or triaged/resolved) or other activity occurs. Thank you for your contributions.

@dapr-bot dapr-bot added the stale Issues and PRs without response label May 13, 2023
@dapr-bot
Copy link
Collaborator

This issue has been automatically closed because it has not had activity in the last 67 days. If this issue is still valid, please ping a maintainer and ask them to label it as pinned, good first issue, help wanted or triaged/resolved. Thank you for your contributions.

@dapr-bot dapr-bot closed this as not planned Won't fix, can't repro, duplicate, stale May 20, 2023
@yaron2 yaron2 reopened this May 21, 2023
@dapr-bot dapr-bot removed the stale Issues and PRs without response label May 21, 2023
@ItalyPaleAle
Copy link
Contributor Author

This is pending on cenkalti/backoff#134

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

No branches or pull requests

5 participants