Skip to content

chore(api): replace GlitchTip config with Sentry#144

Merged
vvatelot merged 3 commits into
mainfrom
chore/sentry
Jul 8, 2026
Merged

chore(api): replace GlitchTip config with Sentry#144
vvatelot merged 3 commits into
mainfrom
chore/sentry

Conversation

@vvatelot

@vvatelot vvatelot commented Jul 8, 2026

Copy link
Copy Markdown
Member

Summary

  • replace the legacy GLITCHTIP_DSN configuration with SENTRY_DSN while keeping backward compatibility
  • centralize Sentry initialization for the API and worker, including FastAPI and RQ integrations
  • capture internal API and worker failures in Sentry, add env/docs updates, and cover the new config with unit tests

Test plan

  • uv run pytest test/components/ecoindex/config/test_sentry.py -q
  • Verify API 500 errors are reported to Sentry in a non-local environment
  • Verify worker failures are reported to Sentry with task context

@vvatelot @cursoragent
chore(api): Update Sentry integration and bump API version to 3.13.0
0f443f2 
Co-authored-by: Cursor <cursoragent@cursor.com>
@vvatelot vvatelot self-assigned this Jul 8, 2026
@github-actions github-actions Bot added documentation Improvements or additions to documentation API tests components labels Jul 8, 2026
vvatelot added 2 commits July 8, 2026 15:14
@vvatelot
fix(api): Refactor Sentry integration and update settings
df6b4f9 
- Updated Sentry integration to use type hints for better clarity.
- Removed legacy GLITCHTIP_DSN support from settings and documentation.
- Adjusted tests to reflect the removal of GLITCHTIP_DSN environment variable.
@vvatelot
refactor(api): Migrate Sentry integration to monitoring module
23bd086 
- Moved Sentry integration from the config module to a new monitoring module for better organization.
- Updated imports in main application, exception handler, and worker tasks to reflect the new module structure.
- Removed the legacy Sentry configuration file to streamline the codebase.
@github-actions

github-actions Bot commented Jul 8, 2026

Copy link
Copy Markdown

Coverage PR 144

Branch coverage •
FileStmtsMissCoverMissing
components/ecoindex/config
   settings.py280100% 
components/ecoindex/monitoring
   __init__.py20100% 
   sentry.py44198%62
TOTAL77423170% 

@vvatelot vvatelot merged commit 5ae85ef into main Jul 8, 2026
6 checks passed
@vvatelot vvatelot deleted the chore/sentry branch July 8, 2026 15:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@vvatelot vvatelot

Labels

API components documentation Improvements or additions to documentation tests

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@vvatelot