Feature Request: Can we have retry logic applied for HTTP 429 (Rate limiting)

[pritamsarkar007]

@sethgrid Please let me know your thoughts. We are planning to implement rate limiting on our vault cluster and envconsul, consul-template is one of our main tools to read secrets. As per RFC, we will return 429 in the event of rate limiting. With this feature, envconsul and consul template will retry if pester add 429 in its retry list.

[sethgrid]

That is a really solid point!
https://github.com/sethgrid/pester/blob/master/pester.go#L280
Seems like a minor update here to also check for that status code would do the trick. Plus an added test.

I'm not able to do it today. Maybe this or next weekend. If you need it sooner, PRs are welcome :)

[pritamsarkar007]

@sethgrid thanks for quick response. I will see if I can manage sometime over the weekend and submit the PR sooner or else will wait for you.

[pritamsarkar007]

@sethgrid I have to go through some approval process before submitting the PR. I will submit the PR once I get the approval.

[sethgrid]

sounds good; keep me in the loop. If approval doesn't work out, let me know too.

[pritamsarkar007]

@sethgrid I just got approval today. I will try to submit my changes ASAP.

[pritamsarkar007]

@sethgrid PR submitted #32