Skip to content
This repository has been archived by the owner on Nov 13, 2023. It is now read-only.

OrTransientHttpError should also consider HTTP 429 (too many requests)? #26

Open
ghost opened this issue May 15, 2020 · 1 comment
Open

Comments

@ghost
Copy link

ghost commented May 15, 2020

There is a rationale behind the fact that OrTransientHttpError is not considering HTTP 429 as a transient error?

HTTP 429 (too many requests) is the correct error reply in case of throttling. I think it's a good case where we want to retry a request a little bit later (maybe with an exponential backoff).

@cocowalla
Copy link

I'd also add that it would be good to adhere to the Retry-After header, if present on a 429 response.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant