Skip to content

Handle transient GitHub request failures#103

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/high-impact-sentry-errors-b15a
Draft

Handle transient GitHub request failures#103
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/high-impact-sentry-errors-b15a

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented May 8, 2026

Summary

  • Add a shared GitHub API request helper with timeout and retries for transient connection failures.
  • Use the helper for GitHub metadata/config/token requests.
  • Ignore token revocation network failures so cleanup does not mask the original webhook processing result.

Validation

  • .venv/bin/python -m pytest (22 passed, 1 skipped)

Sentry issue: https://sentry.sentry.io/issues/7468278997/

Open in Web View Automation 

Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
Comment thread src/github_requests.py
attempts = retries + 1
for attempt in range(1, attempts + 1):
try:
return requests.request(method, url, timeout=timeout, **kwargs)
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.

2 participants