Skip to content

chore: Drop more deprecated stuff - #7263

Open
sentrivana wants to merge 7 commits into
major/3.0from
ivana/major/more-deprecations
Open

chore: Drop more deprecated stuff#7263
sentrivana wants to merge 7 commits into
major/3.0from
ivana/major/more-deprecations

Conversation

@sentrivana

@sentrivana sentrivana commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

Changes:

  • remove deprecated max_spans option of LangchainIntegration
  • remove deprecated sentry_sdk.init() context manager
  • remove deprecated configure_debug_hub
  • remove deprecated Baggage.from_options()
  • remove deprecated Transport.capture_event()
  • remove deprecated FunctionTransport
  • the Scope.trace_propagation_meta function no longer accepts a span as argument
  • remove deprecated direct assignment to Scope.level
  • remove deprecated direct assignment to Scope.user
  • remove deprecated Scope.iter_headers

Closes https://linear.app/getsentry/issue/PY-1935/drop-deprecated-stuff

@linear-code

linear-code Bot commented Aug 26, 2026

Copy link
Copy Markdown

PY-1935

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

110471 passed | ⏭️ 5428 skipped | Total: 115899 | Pass Rate: 95.32% | Execution Time: 373m 24s

📊 Comparison with Base Branch

Metric Change
Total Tests 📉 -9128
Passed Tests 📉 -7823
Failed Tests 📉 -1
Skipped Tests 📉 -1304

➖ Removed Tests (1)

View removed tests
  • test_continuous_profiler_auto_start_and_stop_sampled[non-experiment-gevent]
    • File: tests.profiler.test_continuous_profiler

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    90.24%    90.59%    +0.35%
==========================================
  Files          193       185        -8
  Lines        25474     23775     -1699
  Branches      9378      8756      -622
==========================================
+ Hits         22988     21537     -1451
- Misses        2486      2238      -248
- Partials      1431      1368       -63

Generated by Codecov Action

@sentrivana
sentrivana marked this pull request as ready for review August 27, 2026 08:01
@sentrivana
sentrivana requested a review from a team as a code owner August 27, 2026 08:01

@cursor cursor Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 6f4bd53. Configure here.

Comment thread sentry_sdk/transport.py
if event is not None:
self.capture_event(event)


Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Callable transports silently send events

High Severity

After function transports were removed, a callable passed as transport is no longer recognized. make_transport then falls through and constructs the default HttpTransport whenever a DSN is set. Callers that still pass a function (for example a no-op or a list-append in tests) will silently send real events to Sentry instead of failing. The ClientConstructor type still lists Callable[[Event], None] as valid, so type checkers will not catch this.

Fix in Cursor Fix in Web

Reviewed by Cursor Bugbot for commit 6f4bd53. Configure here.

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.

1 participant