Skip to content

Handle Sentry envelope transport failures gracefully#90

Draft
cursor[bot] wants to merge 1 commit intomainfrom
cursor/high-impact-sentry-errors-7a2e
Draft

Handle Sentry envelope transport failures gracefully#90
cursor[bot] wants to merge 1 commit intomainfrom
cursor/high-impact-sentry-errors-7a2e

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented Apr 20, 2026

Summary

  • catch requests.RequestException in GithubClient._send_envelope so outbound Sentry ingestion failures do not crash webhook handling
  • add a request timeout for envelope POSTs to avoid hanging request threads on unreachable DSN hosts
  • include host-focused logging for failed envelope delivery attempts
  • add regression tests verifying connection failures and HTTP 500 responses from Sentry return None instead of raising

Validation

  • python3 -m pytest tests/test_github_sdk.py tests/test_web_app_handler.py
  • Result: 17 passed, 1 skipped

Context

This addresses Sentry issue SENTRY-GITHUB-ACTIONS-APP-77 (https://sentry.sentry.io/issues/7354001476/).

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