Skip to content

feat(tracing): [Ignore HTTP Status 4] Ignore incoming 404s by default - #6567

Open
adinauer wants to merge 3 commits into
feat/ignore404-filterfrom
feat/ignore404-default
Open

adinauer wants to merge 3 commits into
feat/ignore404-filterfrom
feat/ignore404-default

Conversation

@adinauer

@adinauer adinauer commented Nov 11, 2026 •

Copy link
Copy Markdown
Member

PR Stack (Ignore HTTP Status)


📜 Description

Default ignoredServerHttpStatusCodesForTracing to a fresh list containing HTTP 404. Completed incoming server transactions that finish with 404 are therefore no longer sent to Sentry unless users replace or clear the configuration.

Explicit custom lists still replace the default, while an empty list or empty external/Spring configuration restores tracing for every incoming HTTP status. The Spring Boot documentation and breaking-change entry include Java, properties, and YAML opt-out examples.

💡 Motivation and Context

404 transactions are commonly low-value tracing noise. This changes the default requested in #4736 while retaining the narrow filtering rules from the previous stack PR: only root server transactions with a numeric root response status are affected. Outgoing HTTP requests, errors, logs, metrics, replay, continuous profile chunks, and external OTLP export remain independent.

Fixes #4736

💚 How did you test it?

  • Core SentryOptionsTest and SentryClientTest
  • Spring Boot auto-configuration tests for Spring Boot 2, 3, and 4
  • Servlet and WebFlux integration tests for all three Spring Boot generations
  • OpenTelemetry agent and agentless integration tests for all three Spring Boot generations
  • Manual Spring Boot 4 sample verification against a local envelope receiver: default 404 dropped, empty configuration retained 404, custom ranges replaced the default, and combined configurations such as 404,500-599 and 200-399,404 filtered each configured status when root response metadata was available
  • make format api
  • git diff --check

📝 Checklist

  • I added GH Issue ID & Linear ID
  • I added tests to verify the changes.
  • No new PII added or SDK only sends newly added PII if sendDefaultPII is enabled.
  • I updated the docs if needed.
  • I updated the wizard if needed.
  • Review from the native team if needed.
  • No breaking change or entry added to the changelog.
  • No breaking change for hybrid SDKs or communicated to hybrid SDKs.
  • Public API changes reviewed by another Mobile SDK team member or implemented according to the develop docs spec.

🔮 Next steps

Coordinate the corresponding migration/options documentation in sentry-docs before the 9.x release.

⚠️ Merge this PR using a merge commit (not squash). Only the collection branch is squash-merged into main.

Stop sending incoming server transactions that finish with HTTP 404 unless users replace or clear the ignored status configuration. Document the opt-out and cover native Spring and OpenTelemetry paths across supported Spring Boot generations.

Fixes #4736
@github-actions

github-actions Bot commented Nov 11, 2026 •

Copy link
Copy Markdown
Contributor
Messages
📖 Do not forget to update Sentry-docs with your feature once the pull request gets approved.

Generated by 🚫 dangerJS against a76505b

@sentry

sentry Bot commented Nov 11, 2026 •

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.63.0 (1) release

⚙️ sentry-android Build Distribution Settings

@adinauer
adinauer marked this pull request as ready for review November 12, 2026 09:04
@adinauer
adinauer requested a review from a team as a code owner November 12, 2026 09:04
@adinauer
adinauer requested review from 0xadam-brown and romtsn and removed request for a team November 12, 2026 09:04

This branch has not been deployed

No deployments
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.

1 participant