Skip to content

[octocrab] Add support for 429 headers that github returns#6

Merged
nataliejameson merged 2 commits intodiscordfrom
nmj/add-rate-limit-retryer
Jun 2, 2025
Merged

[octocrab] Add support for 429 headers that github returns#6
nataliejameson merged 2 commits intodiscordfrom
nmj/add-rate-limit-retryer

Conversation

@nataliejameson
Copy link

We're currently seeing some issues where sometimes we hit secondary rate limits from github. https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28 describes what headers to look for and how long to wait between retries, so add a new retry implementation that respects these hints.

We're currently seeing some issues where sometimes we hit secondary rate limits from github. https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28 describes what headers to look for and how long to wait between retries, so add a new retry implementation that respects these hints.
@nataliejameson nataliejameson merged commit 17056cd into discord Jun 2, 2025
22 checks passed
nataliejameson added a commit that referenced this pull request Mar 19, 2026
We're currently seeing some issues where sometimes we hit secondary rate limits from github. https://docs.github.com/en/rest/using-the-rest-api/rate-limits-for-the-rest-api?apiVersion=2022-11-28 describes what headers to look for and how long to wait between retries, so add a new retry implementation that respects these hints.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants