[jamescrosswell]

Summary

TraceIgnoreStatusCodes now only filters http.server transactions. Until now it also dropped outgoing HttpClient calls traced with OpenTelemetry.

With OpenTelemetry, an HttpClient call that has no local parent (in a BackgroundService, a timer, startup code, or a consumer continuing a remote traceparent) becomes an http.client transaction of its own. SentrySpanProcessor.OnEnd copies http.response.status_code onto it, and TraceIgnoreStatusCodeTransactionProcessor then dropped it whenever the downstream status matched the list. For example, a 401 or 403 from a downstream API usually means expired credentials, and that transaction silently disappeared.

The spec says the option "applies exclusively to incoming requests". All of the SDK's incoming-request transactions use the http.server operation: the ASP.NET Core SentryTracingMiddleware, ASP.NET's HttpContextExtensions, and SentrySpanProcessor.ParseOtelSpanDescription for non-client HTTP spans. So the processor now checks the operation, as an allowlist rather than excluding http.client, so that other root transactions carrying a status code are left alone too.

Found by @ric-oliv while reviewing #5632, which turns on a non-empty default for TraceIgnoreStatusCodes in 7.0. That would expose this bug to every ASP.NET Core app using OpenTelemetry, so this fix needs to reach version7 before #5632 merges.

Notes for review

• Until now this only affected people who configured TraceIgnoreStatusCodes themselves, since the list is empty by default on main.
• Added ParseOtelSpanDescription_HttpServer alongside the existing http.client test, since the processor now depends on OpenTelemetry server spans being mapped to http.server.

🤖 Generated with Claude Code

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 74.93%. Comparing base (4af9645) to head (54a97ac).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #5643      +/-   ##
==========================================
+ Coverage   74.37%   74.93%   +0.56%     
==========================================
  Files         501      515      +14     
  Lines       18566    18973     +407     
  Branches     3603     3693      +90     
==========================================
+ Hits        13808    14217     +409     
+ Misses       3883     3876       -7     
- Partials      875      880       +5     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[ric-oliv]

Looks good!