Skip to content

feat(starlette): Add http.route attribute - #7338

Merged
alexander-alderman-webb merged 2 commits into
masterfrom
webb/starlette/http-route
Sep 2, 2026
Merged

feat(starlette): Add http.route attribute#7338
alexander-alderman-webb merged 2 commits into
masterfrom
webb/starlette/http-route

Conversation

@alexander-alderman-webb

Copy link
Copy Markdown
Contributor

Description

Set the http.route attribute on the ASGI server span in patches for Starlette endpoints.

Issues

Reminders

@alexander-alderman-webb
alexander-alderman-webb marked this pull request as ready for review September 2, 2026 12:37
@alexander-alderman-webb
alexander-alderman-webb requested a review from a team as a code owner September 2, 2026 12:37
@github-actions

github-actions Bot commented Sep 2, 2026

Copy link
Copy Markdown
Contributor

Codecov Results 📊

130404 passed | ⏭️ 7110 skipped | Total: 137514 | Pass Rate: 94.83% | Execution Time: 463m 9s

📊 Comparison with Base Branch

Metric Change
Total Tests 📈 +32113
Passed Tests 📈 +31482
Failed Tests
Skipped Tests 📈 +631

All tests are passing successfully.

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

Coverage diff
@@            Coverage Diff             @@
##          main       #PR       +/-##
==========================================
+ Coverage    89.92%    90.25%    +0.33%
==========================================
  Files          193       193         —
  Lines        24908     25677      +769
  Branches      8986      9466      +480
==========================================
+ Hits         22398     23173      +775
- Misses        2510      2504        -6
- Partials      1430      1440       +10

Generated by Codecov Action

@alexander-alderman-webb
alexander-alderman-webb merged commit 5c6a9eb into master Sep 2, 2026
164 checks passed
@alexander-alderman-webb
alexander-alderman-webb deleted the webb/starlette/http-route branch September 2, 2026 13:17
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