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.

Sign up for GitHub

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

Jump to bottom

"You have exceeded a secondary rate limit" does not throw RateLimitExceededException #2466

Closed
HenrikPoulsen opened this issue Jun 27, 2022 · 0 comments · Fixed by #2473
Closed

"You have exceeded a secondary rate limit" does not throw RateLimitExceededException #2466

HenrikPoulsen opened this issue Jun 27, 2022 · 0 comments · Fixed by #2473
Labels
Status: Up for grabs Issues that are ready to be worked on by anyone Type: Bug Something isn't working as documented

Comments

@HenrikPoulsen
Copy link

As seen here when determining to throw a RateLimitExceededException it does not handle the case where it prints You have exceeded a secondary rate limit. Please wait a few minutes before you try again. and instead you get a Octokit.ForbiddenException
@JonruAlveus JonruAlveus mentioned this issue Jul 5, 2022
Merged
@nickfloyd nickfloyd added Type: Bug Something isn't working as documented Status: Up for grabs Issues that are ready to be worked on by anyone and removed category: bug labels Oct 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
Status: Up for grabs Issues that are ready to be worked on by anyone Type: Bug Something isn't working as documented
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Adding in handling for secondary rate limit exceptions JonruAlveus/octokit.net
2 participants
@nickfloyd @HenrikPoulsen