Repository navigation
docs(dotnet): Document TraceIgnoreStatusCodes - #19898
Conversation
Add the option to the .NET options page for the ASP.NET-based guides, and mention it in the ASP.NET Core and ASP.NET tracing pages. The default is empty in 6.x and changes to 301, 305-399 and 401-404 in 7.0.0 (getsentry/sentry-dotnet#5632). Closes getsentry/sentry-dotnet#5708 Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
|
The latest updates on your projects. Learn more about Vercel for GitHub.
1 Skipped Deployment
|
Co-authored-by: James Crosswell <jamescrosswell@users.noreply.github.com>
|
The plan checks client errors on the changed pages, page-load counts, and error logs from those routes. An issue escalates when client errors rise or those pages stop loading. Monitoring in Services: Mention PlanWhat changedThe .NET options page now documents TraceIgnoreStatusCodes for inbound HTTP tracing. The ASP.NET and ASP.NET Core tracing pages now say which status codes skip transactions. sentry-docs must deploy to production before this text is live. RiskA broken MDX block can fail the options page for every .NET guide. New code samples can throw in the client code tabs. A static build failure stops the deploy before users see the pages. Intended effectThis change is a documentation edit. Telemetry cannot show that the new text is on the page. Confirmed means the three live pages include TraceIgnoreStatusCodes. Absent means those pages still omit the option. The pages are Regression watchThis diff can break MDX render and code tabs on the .NET options page and on inherited guide copies of that page.
If error counts rise, inspect client exceptions on the options page and on the two tracing pages. Check code tabs and platform-section render first. Not observableCorrectness of the new option text has no telemetry. Page-load latency for these routes is not available. Client transactions do not use the URL path as the transaction name. The 404 metric stores only the first two path segments, so it cannot isolate these pages. |
Co-authored-by: Ricardo Colombo Oliveira <github@ricoliv.com>
ric-oliv
left a comment
There was a problem hiding this comment.
Nice! Just added a few suggestions inline, nothing blocking though.
Two things outside the diff:
- The note in the PR description says the
[]/[0]caveat is worded as 7.0.0-only, but 6ca5bdc dropped that qualifier. The text is still true on 6.x, so only the description needs updating. - The .NET migration guide has no 7.x section yet. The Migration section in getsentry/sentry-dotnet#5632 is ready to copy. Worth tracking before the 7.0.0 release.
Co-authored-by: Ricardo Colombo Oliveira <github@ricoliv.com>
DESCRIBE YOUR PR
Documents
SentryOptions.TraceIgnoreStatusCodesfor the .NET SDK. The option shipped in 6.4.0 (getsentry/sentry-dotnet#5034), but the .NET docs don't mention it. In 7.0.0 its default changes from an empty list to 301, 305–399 and 401–404 (getsentry/sentry-dotnet#5632), so ASP.NET Core and ASP.NET apps will stop sending transactions for those responses.SdkOptionentry to the Tracing Options on the shared .NET options page. It covers what the option filters, how to set codes and ranges, the 7.0.0 default and why 302/303 aren't in it, how to trace everything again, andappsettings.json.Notes for review:
PlatformSectionfor the guides that run Sentry's ASP.NET or ASP.NET Core integration: ASP.NET, ASP.NET Core, Blazor Server, AWS Lambda and Google Cloud Functions. The option has no effect elsewhere.HttpClientrequests aren't affected" holds from 6.12.0 (getsentry/sentry-dotnet#5643). From 6.4.0 to 6.11.x, an app usingUseOpenTelemetrythat set the option could also lose roothttp.clienttransactions whose downstream status matched. I left that fixed bug off the page.UseOtlp, spans go straight to Sentry's OTLP endpoint and never reach the SDK's transaction processors, so the option doesn't apply. The page says so in one line.[]vs[0]caveat forappsettings.jsononly matters from 7.0.0, when the default is non-empty, and is worded that way. Configuration can't express ranges yet (getsentry/sentry-dotnet#5644).Closes getsentry/sentry-dotnet#5708
IS YOUR CHANGE URGENT?
Help us prioritize incoming PRs by letting us know when the change needs to go live.
Select exactly one option. For deadlines, replace
YYYY-MM-DDwith the due date. You can update this information later by editing the PR description.PRE-MERGE CHECKLIST
Make sure you've checked the following before merging your changes:
🤖 Generated with Claude Code