Skip to content

Handle GitHub token revocation failures#101

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

Handle GitHub token revocation failures#101
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sentry-error-investigation-2881

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented May 2, 2026

Summary

  • Treat GitHub App token revocation as best-effort cleanup so transient requests failures do not turn successfully processed webhooks into 500s.
  • Log revocation failures with stack context for observability.
  • Add tests covering revocation ConnectionError handling and preserving token acquisition failures.

Validation

  • pytest tests/test_github_app.py
  • pytest

Fixes SENTRY-GITHUB-ACTIONS-APP-6M.

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