Fix: Unbounded concurrent API requests in explore() causes Secondary Rate Limit (Abuse) bans#125
Fix: Unbounded concurrent API requests in explore() causes Secondary Rate Limit (Abuse) bans#125Dotify71 wants to merge 1 commit into
Conversation
|
No actionable comments were generated in the recent review. 🎉 ℹ️ Recent review info⚙️ Run configurationConfiguration used: Path: .coderabbit.yaml Review profile: ASSERTIVE Plan: Pro Plus Run ID: 📒 Files selected for processing (1)
WalkthroughThe ChangesExplore contributor fetching
Estimated code review effort: 2 (Simple) | ~10 minutes Possibly related PRs
Suggested labels: Suggested reviewers: Poem
🚥 Pre-merge checks | ✅ 4✅ Passed checks (4 passed)
✨ Finishing Touches🧪 Generate unit tests (beta)
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Fixes #124.
This PR adds a batching loop to the
explore()function when fetching contributors. Previously, it fired off API requests for all repositories concurrently, which instantly triggered GitHub's secondary rate limits (abuse bans) for large organizations. The requests are now chunked in batches of 5, identical to the pattern used inrunAudit().Summary by CodeRabbit