Skip to content

feat(streaming): Promote before_send_span to top-level option#6885

Open
sentrivana wants to merge 3 commits into
masterfrom
ivana/before-send-span-top-level
Open

feat(streaming): Promote before_send_span to top-level option#6885
sentrivana wants to merge 3 commits into
masterfrom
ivana/before-send-span-top-level

Conversation

@sentrivana

@sentrivana sentrivana commented Jul 24, 2026

Copy link
Copy Markdown
Contributor

Bump before_send_span to a top-level option alongside its other span streaming friends.

Add `before_send_span` as a top-level `sentry_sdk.init()` parameter
alongside `before_send_log` and `before_send_metric`, while keeping
backward compatibility with the `_experiments` dict. Warn when span
streaming is not enabled.
@sentrivana
sentrivana marked this pull request as ready for review July 24, 2026 13:00
@sentrivana
sentrivana requested a review from a team as a code owner July 24, 2026 13:00
@github-actions

Copy link
Copy Markdown
Contributor

Codecov Results 📊

96545 passed | ⏭️ 6340 skipped | Total: 102885 | Pass Rate: 93.84% | Execution Time: 336m 55s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +110
Passed Tests 📈 +110
Failed Tests 📉 -1
Skipped Tests 📈 +1

All tests are passing successfully.

✅ Patch coverage is 100.00%. Project has 2510 uncovered lines.
✅ Project coverage is 89.68%. Comparing base (base) to head (head).

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.68%    89.68%        —%
==========================================
  Files          193       193         —
  Lines        24330     24332        +2
  Branches      8578      8580        +2
==========================================
+ Hits         21820     21822        +2
- Misses        2510      2510         —
- Partials      1397      1397         —

Generated by Codecov Action

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