fix(asgi): Stop duplicating scope["root_path"] in URLs

[alexander-alderman-webb]

Description

Add the path_includes_root_path boolean parameter to _get_url() and related functions.

If the parameter is True (the default), scope["path"] is considered to include scope["root_path"], where scope is the ASGI scope. Otherwise, the existing behavior that prepends scope["root_path"] when forming the url is preserved.

Add tests to each ASGI-based integration. Some are compliant with the ASGI spec in which scope["path"] includes scope["root_path"], while others are not. In particular, Starlite, LiterStar and old versions of Starlette and Quart are not compliant with the spec. In these cases, path_includes_root_path=False is set.

Issues

Closes #6577

Reminders

• Please add tests to validate your changes, and lint your code using uv run ruff.
• Add GH Issue ID & Linear ID (if applicable)
• PR title should use conventional commit style (feat:, fix:, ref:, meta:)
• For external contributors: CONTRIBUTING.md, Sentry SDK development docs, Discord community

[github-actions]

Codecov Results 📊

✅ 90998 passed | ⏭️ 6135 skipped | Total: 97133 | Pass Rate: 93.68% | Execution Time: 318m 48s

📊 Comparison with Base Branch

Metric	Change
Total Tests	📈 +178
Passed Tests	📈 +172
Failed Tests	—
Skipped Tests	📈 +6

All tests are passing successfully.

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

Files with missing lines (1)

File	Patch %	Lines
sentry_sdk/integrations/quart.py	100.00%	⚠️ 1 partials

Coverage diff

@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.85%    89.86%    +0.01%
==========================================
  Files          192       192         —
  Lines        23745     23748        +3
  Branches      8198      8198         —
==========================================
+ Hits         21334     21339        +5
- Misses        2411      2409        -2
- Partials      1344      1344         —

---

Generated by Codecov Action

[cursor]

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

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

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

[wahajahmed010]

Review: #6579

Summary: Clean fix for ASGI URL duplication when scope["root_path"] is already included in scope["path"]. The approach is sound — thread a path_includes_root_path boolean through the URL construction pipeline.

✅ What works well

• Backward compatible — default is True, preserving existing behavior for frameworks that don't include root_path in path
• Framework-specific handling — correctly version-gated for Starlette (>=0.33), Quart (>=0.19), and hardcoded for Litestar/Starlite
• Test coverage — all 6 integrations tested (Django ASGI, FastAPI, Litestar, Quart, Starlette, Starlite), both span streaming and static lifecycle modes
• Clean threading — parameter flows through _get_url → _get_request_data → _get_request_attributes → _get_transaction_name_and_source → _get_segment_name_and_source

⚠️ Potential issues

1. Django ASGI test — The test sets comm.scope["root_path"] = "/root" but Django defaults to path_includes_root_path=True. Does Django ASGI actually include root_path in path? If not, this test might pass but produce incorrect URLs in production. Worth verifying Django's ASGI behavior.
2. No test for path_includes_root_path=False — The Starlette/FastAPI tests only exercise the True path (Starlette >= 0.33). The old behavior (False) has no dedicated test.
3. Edge case: empty path — When path_includes_root_path=True and scope["path"] is empty, the URL would be just scheme+host with no path. Unlikely but worth noting.

Verdict

Solid PR. The core logic is correct and well-structured. The main risk is the Django default assumption — worth double-checking before merging.