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.

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

"Abuse detected" slowdowns and possible solution #189

Open
DavidGriffith opened this issue Sep 22, 2023 · 2 comments
Open

"Abuse detected" slowdowns and possible solution #189

DavidGriffith opened this issue Sep 22, 2023 · 2 comments

Comments

@DavidGriffith
Copy link

I have a fairly small repo with 229 issues that gets stalled around Issue 40 with the following message:

Migrating issue #40 ('[PLACEHOLDER] - for issue #40')...
Abuse detected for request POST /repos/{owner}/{repo}/issues
Retrying after 1000 seconds!

At this point I can leave it running for hours and it won't ever finish. Is there some way I can throttle back API usage or avoid this some other way? Stopping and restarting doesn't help as it 1) starts from the beginning again and 2) I hit the "Abuse detected" immediately. This has a side-effect of ticking off Github such that when I try to file an issue, it is rejected with There was an error creating your Issue: was submitted too quickly.

@DavidGriffith
Copy link
Author

Putting a quick-and-dirty delay of ten seconds at the beginning of the for loop in transferIssues() seems to allow the transfer to take place without provoking Github's abuse-detector. I'm not sure what the shortest acceptable delay is -- hopefully a lot less because transferring https://github.com/DavidGriffith/minipro-import-test from Gitlab took a couple hours. Is simply sticking in a delay the right way to do this? If so, I'll polish this fix up and submit a pull request.

@DavidGriffith DavidGriffith changed the title "Abuse detected" slowdowns "Abuse detected" slowdowns and possible solution Sep 22, 2023
@klausmcm
Copy link

@DavidGriffith Could you please share a copy or at least a diff of your transferIssues() function with the delay? There are two for loops in that function. I guess the second loop is where the delay is?

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

No branches or pull requests

2 participants