Skip to content
This repository has been archived by the owner on Oct 8, 2024. It is now read-only.

Rate limiting needed #9

Closed
leontebbens opened this issue Jul 12, 2019 · 4 comments · Fixed by #11
Closed

Rate limiting needed #9

leontebbens opened this issue Jul 12, 2019 · 4 comments · Fixed by #11

Comments

@leontebbens
Copy link
Contributor

leontebbens commented Jul 12, 2019 •
edited
Loading

Hi, when using this script against our Github Enterprise Cloud I get rate limit errors:
"You have triggered an abuse detection mechanism. Please wait a few minutes before you try again.
https://developer.github.com/v3/#abuse-rate-limits"

Suggestion: https://github.com/github/platform-samples/tree/master/api/ruby/find-inactive-members has rate limiting
@emtunc
Copy link

emtunc commented Jul 13, 2019

Was literally going to add an issue to this repo about this - rate limiting is needed for both security alerts + security fixes as they both trigger Github's anti-abuse controls.

@jimmed jimmed mentioned this issue Jul 15, 2019
Merged
@emtunc
Copy link

emtunc commented Jul 15, 2019

Thanks @jimmed - have tested and it works awesome!

@leontebbens take a look at the PR ^^

@leontebbens
Copy link
Contributor Author

Thanx @jimmed - it solves the problem

@leontebbens leontebbens reopened this Jul 16, 2019
@leontebbens
Copy link
Contributor Author

leontebbens commented Jul 16, 2019 •
edited
Loading

please accept the pull request
@jhutchings1 ?

@leontebbens leontebbens mentioned this issue Jul 16, 2019
Closed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Execute API calls in series to prevent hitting rate limit
2 participants
@leontebbens @emtunc