Skip to content

Handle Sentry ingest SSL transport errors#108

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

Handle Sentry ingest SSL transport errors#108
cursor[bot] wants to merge 1 commit into
mainfrom
cursor/sentry-error-investigation-b7cf

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented May 20, 2026

Summary

  • Treat SSL transport failures while posting generated envelopes to Sentry ingest as failed telemetry delivery instead of app crashes.
  • Log the SSL failure at warning level so GitHub webhook processing can still complete.
  • Add a regression test that simulates a Sentry ingest SSL failure during send_trace.

Validation

  • python3 -m pytest tests/test_github_sdk.py
  • python3 -m pytest

Sentry

  • Addresses SENTRY-GITHUB-ACTIONS-APP-4E (SSLError while POSTing /envelope/ to Sentry ingest).
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