Skip to content

feat(starlite): Set name and source on route span when streaming#6552

Open
alexander-alderman-webb wants to merge 5 commits into
masterfrom
webb/starlite/transaction-name
Open

feat(starlite): Set name and source on route span when streaming#6552
alexander-alderman-webb wants to merge 5 commits into
masterfrom
webb/starlite/transaction-name

Conversation

@alexander-alderman-webb

@alexander-alderman-webb alexander-alderman-webb commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Description

Call Scope.set_transaction_name() on isolation and current scopes instead of manually overwriting event fields in an event processor.

Using the function ensures that the name and the sentry.span.source attribute are set on the segment span when the streaming trace lifecycle is enabled.

Issues

Reminders

@alexander-alderman-webb alexander-alderman-webb changed the title feat(starlite): Set name and source on route span feat(starlite): Set name and source on route span when streaming Jun 10, 2026
@alexander-alderman-webb alexander-alderman-webb marked this pull request as ready for review June 10, 2026 09:04
@alexander-alderman-webb alexander-alderman-webb requested a review from a team as a code owner June 10, 2026 09:04
Comment thread sentry_sdk/integrations/starlite.py
@github-actions

github-actions Bot commented Jun 10, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

87870 passed | ⏭️ 5993 skipped | Total: 93863 | Pass Rate: 93.62% | Execution Time: 294m 29s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -317
Passed Tests 📉 -402
Failed Tests
Skipped Tests 📈 +85

All tests are passing successfully.

✅ Patch coverage is 82.35%. Project has 2373 uncovered lines.
✅ Project coverage is 89.87%. Comparing base (base) to head (head).

Files with missing lines (1)
File Patch % Lines
sentry_sdk/integrations/starlite.py 82.35% ⚠️ 3 Missing and 2 partials
Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.83%    89.87%    +0.04%
==========================================
  Files          192       192         —
  Lines        23430     23432        +2
  Branches      8050      8050         —
==========================================
+ Hits         21049     21059       +10
- Misses        2381      2373        -8
- Partials      1333      1329        -4

Generated by Codecov Action

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants