Skip to content

Harden GitHub Sentry config fetch against transient SSL failures#79

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

Harden GitHub Sentry config fetch against transient SSL failures#79
cursor[bot] wants to merge 1 commit intomainfrom
cursor/high-impact-sentry-errors-da61

Conversation

@cursor
Copy link
Copy Markdown

@cursor cursor Bot commented Apr 19, 2026

Summary

  • add bounded retry logic for transient GitHub API request failures (SSL, timeout, connection) when fetching .sentry/sentry_config.ini
  • add explicit timeout to the GitHub API request to avoid hanging request workers
  • keep non-retryable failures fast-fail behavior unchanged
  • add tests for retry success path, max-attempt exhaustion, and non-retryable error behavior

Validation

  • python3 -m pytest -q

Sentry

Open in Web View Automation 

Co-authored-by: Armen Zambrano G. <armenzg@users.noreply.github.com>
Comment thread src/sentry_config.py
def _get_with_retry(url: str, headers: dict[str, str]) -> requests.Response:
for attempt in range(1, GITHUB_API_MAX_ATTEMPTS + 1):
try:
return requests.get(url, headers=headers, timeout=GITHUB_API_TIMEOUT_SECONDS)
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