Skip to content
This repository has been archived by the owner on Jan 19, 2021. It is now read-only.

Default to retry-after header value for 503 in ClientContextExtensions.cs #2782

Open
tbithell opened this issue Sep 29, 2020 · 1 comment
Open

Comments

@tbithell
Copy link

tbithell commented Sep 29, 2020

Is your feature request related to a problem? Please describe.
Based on discussions I've had with various folks at Microsoft a 503 response should always honor the retry-after header value. The way 429s are handled is fine, but retrying a 503 sooner than the retry after header can result in being further penalized. Also, I've seen it vary as far as the value returned, sometimes being as low as 5 seconds, so in that case of a 503 it doesn't appear we are doomed to a 120 second back off.

Describe the solution you'd like
Handle the 503s separately from the 409s. In the case of the 503s always honor the retry-after header

@ghost
Copy link

ghost commented Sep 29, 2020

Thank you for reporting this issue. We will be triaging your incoming issue as soon as possible.

@ghost ghost added the Needs: Triage 🔍 label Sep 29, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

1 participant