-
Notifications
You must be signed in to change notification settings - Fork 583
Comments
feat(wsgi): Migrate SentryWsgiMiddleware to span-streaming SDK#5477
Draft
sentrivana wants to merge 1 commit intofeat/span-first-2 from
Draft
feat(wsgi): Migrate SentryWsgiMiddleware to span-streaming SDK #5477sentrivana wants to merge 1 commit intofeat/span-first-2 from
sentrivana wants to merge 1 commit intofeat/span-first-2 from
Conversation
Add dual-path span creation in WSGI middleware following the same pattern used in ASGI, Celery, Django, and Starlette migrations. When span streaming is enabled, uses sentry_sdk.traces.continue_trace() and sentry_sdk.traces.start_span() instead of the legacy transaction API. Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Contributor
Semver Impact of This PR
🟡 Minor (new features)
📋 Changelog Preview
This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).
New Features ✨
- (wsgi) Migrate SentryWsgiMiddleware to span-streaming SDK by sentrivana in
#5477
Bug Fixes 🐛
Openai Agents
- Patch
execute_final_output()functions following library refactor by alexander-alderman-webb in#5453 - Patch
execute_handoffs()functions following library refactor by alexander-alderman-webb in#5452 - Patch
run_single_turn_streamed()functions following library refactor by alexander-alderman-webb in#5451 - Patch
run_single_turn()functions following library refactor by alexander-alderman-webb in#5450 - Patch models functions following library refactor by alexander-alderman-webb in
#5449 - Patch tool functions following library refactor by alexander-alderman-webb in
#5445
Internal Changes 🔧
- (openai-agents) New tool field and library error log by alexander-alderman-webb in
#5454 - Use fixed clickhouse action, remove aws-sam-cli dependency by sentrivana in
#5457
Other
- [do not merge] feat: Span streaming & new span API by sentrivana in
#5317 - Update CHANGELOG.md by alexander-alderman-webb in f75a9ac1
- Update CHANGELOG.md by alexander-alderman-webb in b700fa85
- release: 2.53.0 by alexander-alderman-webb in 45379e20
🤖 This preview updates automatically when you update the PR.
Contributor
Codecov Results 📊✅ 13 passed | Total: 13 | Pass Rate: 100% | Execution Time: 6.30s All tests are passing successfully. ❌ Patch coverage is 11.11%. Project has 14303 uncovered lines. Files with missing lines (180)
Generated by Codecov Action |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.
Trying stuff out
Summary
SentryWsgiMiddleware.__call__following the same pattern used in ASGI, Celery, Django, and Starlette migrationssentry_sdk.traces.continue_trace()+sentry_sdk.traces.start_span()instead of the legacy transaction API_sentry_start_responsetype hint to accept bothTransactionandStreamedSpan🤖 Generated with Claude Code