[faizansaiyed123]

Ref #7753

Follow-up to #7825.

This PR replaces FastAPI's private routing.get_request_handler monkey patch with an application-level global dependency.

Changes:

• Patch FastAPI.__init__ to register a Sentry global dependency before user-supplied global dependencies.
• Use Starlette's HTTPConnection injection so the dependency is safe for FastAPI versions where global dependencies also apply to WebSockets.
• Move FastAPI-specific transaction naming and http.route setup into the dependency, using FastAPI's effective route context when available.
• Install the Starlette sync-handler wrapper on the request's Dependant so synchronous endpoints update thread metadata in their actual worker thread.
• Preserve FastAPI request event processing and streamed request-body handling inside the global dependency.
• Preserve user-supplied global dependencies.
• Remove the private get_request_handler patch.

This branch includes #7825 so it can be reviewed end-to-end. Once #7825 is merged, this PR can be rebased/simplified to the dependency-only diff.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Stale Bugbot comment from a previous run.

[faizansaiyed123]

cursor review

[faizansaiyed123]

bugbot run verbose=true

[cursor]

Bugbot request id: 8939ba6b-1511-41cb-8bd6-5514646cb90c

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Bugbot rules debug

No rules were used for this review.

https://cursor.com/docs/bugbot#team-rules

Bugbot request id: 8939ba6b-1511-41cb-8bd6-5514646cb90c

[cursor]

Stale Bugbot comment from a previous run.

[sentrivana]

Hey @faizansaiyed123, can you please update your PR now with the current master now that we've merged the thread change?

[cursor]

Stale Bugbot comment from a previous run.

[faizansaiyed123]

@sentrivana Updated the PR with the current master after #7825 was merged. I also addressed the duplicate Starlette patching issue found by Bugbot. Thanks for the guidance!

[faizansaiyed123]

@getsentry/owners-python-sdk @sentrivana

Cursor Bugbot has completed successfully with no new issues found.

Could someone with maintainer access please approve the pending CI and Test workflows so they can run on the latest commit?

The previous Test run failed in the Mistral step with HTTP 401 (Invalid API Key). If that failure persists, please verify and update the repository's MISTRAL_API_KEY GitHub Actions secret.

Thanks!

[sentrivana]

Hey @faizansaiyed123, I haven't done a full review of this yet, but there are two things I wanted to point out:

1. The SDK now supports data_collection as a replacement for send_default_pii. All code on master needs to support both. If data_collection is defined, it takes precedence over send_default_pii.
2. Similarly, the SDK also supports span streaming with trace_lifecycle="stream" and code on master needs to support both transaction-based tracing and span streaming.

I just glanced at the tests and it might be we're missing taking data_collection and trace_lifecycle="stream" into account there. Please also double check the non-test code if we need to add some if branches. You can have a look at how other similar integration and integration test suites do this.

LMK if you have any questions.

[cursor]

Stale Bugbot comment from a previous run.

[cursor]

Cursor Bugbot has reviewed your changes and found 2 potential issues.

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

^{Reviewed by Cursor Bugbot for commit efc5072. Configure here.}