Skip to content
Snippets Groups Projects

github: Don't record rate limit for 401 response

Merged Warren Gifford requested to merge ratelimit-401 into main

Created by: ryanslade

If we do, the next call we make can block for up to an hour since our rate limit monitor RecommendedWaitForBackgroundOp function believes we've just run out of tokens.

Instead, we ignore the rate limits returned and fail fast on subsequent steps.

Our background syncer already backs off on 401 errors.

Merge request reports

Merged by avatar (Jul 13, 2025 12:15am UTC)

Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
Please register or sign in to reply
Loading