Skip to content

fix: skip 429 rate limit errors from GitHub domains#2387

Merged
jcstein merged 3 commits intomainfrom
copilot/fix-ignore-429-errors
Jan 20, 2026
Merged

fix: skip 429 rate limit errors from GitHub domains#2387
jcstein merged 3 commits intomainfrom
copilot/fix-ignore-429-errors

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented Jan 14, 2026

GitHub URLs returning 429 (Too Many Requests) after retry exhaustion were being reported as broken links, causing false positives in CI.

Changes

  • After all retries exhausted, 429 responses from GitHub domains are now marked as skipped instead of broken
  • Uses proper hostname parsing to match github.com, *.github.com, and *.githubusercontent.com
  • Prevents false matches on domains like fakegithub.com or evil.com/github.com/path
// After retries exhausted
if (lastResult && lastResult.status === 429) {
  const hostname = new URL(url).hostname;
  if (hostname === 'github.com' || 
      hostname.endsWith('.github.com') || 
      hostname.endsWith('.githubusercontent.com')) {
    return { valid: true, skipped: true, reason: 'GitHub 429 - Rate limited, skipped' };
  }
}

Non-GitHub 429s and other GitHub error codes (404, 500, etc.) continue to be reported as failures.

Original prompt

This section details on the original issue you should resolve

<issue_title>fix: ignore all 429s on github</issue_title>
<issue_description>after #2384 there are still 429s from github

[429 - Too Many Requests] https://github.com/celestiaorg/celestia-app/blob/2b8cc9e23826ccb658b7dd5aa6cd51a0921a0c29/go.mod#L35 (in app/operate/data-availability/ibc-relayer/page.mdx)
```</issue_description>

<agent_instructions>this should allow skipping all 429s on github urls</agent_instructions>

## Comments on the Issue (you are @copilot in this section)

<comments>
</comments>


💬 We'd love your input! Share your thoughts on Copilot coding agent in our 2 minute survey.

Co-authored-by: jcstein <46639943+jcstein@users.noreply.github.com>
Comment thread scripts/check-links.mjs Fixed
Co-authored-by: jcstein <46639943+jcstein@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix issue to ignore all 429 responses on GitHub fix: skip 429 rate limit errors from GitHub domains Jan 14, 2026
Copilot AI requested a review from jcstein January 14, 2026 21:07
@jcstein jcstein marked this pull request as ready for review January 15, 2026 15:56
@jcstein jcstein merged commit 81dc136 into main Jan 20, 2026
4 checks passed
@jcstein jcstein deleted the copilot/fix-ignore-429-errors branch January 20, 2026 20:21
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

Successfully merging this pull request may close these issues.

fix: ignore all 429s on github

3 participants