Skip to content

Handle GitHub installation token revoke timeouts#115

Draft
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sentry-error-investigation-b9f0
Draft

Handle GitHub installation token revoke timeouts#115
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sentry-error-investigation-b9f0

Conversation

@cursor

@cursor cursor Bot commented May 21, 2026

Copy link
Copy Markdown

Summary

  • Add a bounded timeout to GitHub App installation token requests.
  • Treat installation token revocation failures as best-effort cleanup warnings so webhook handling does not fail after processing succeeds.
  • Add a regression test for ConnectTimeout during token revocation.

Validation

  • python3 -m pytest tests/test_github_app.py
  • python3 -m pytest
Open in Web View Automation 

Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
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.

1 participant