Skip to content

feat(tracing): ignore configured HTTP status codes - #1059

Open
shahidul5 wants to merge 13 commits into
getsentry:masterfrom
shahidul5:feat/ignore-http-status-code-traces
Open

feat(tracing): ignore configured HTTP status codes#1059
shahidul5 wants to merge 13 commits into
getsentry:masterfrom
shahidul5:feat/ignore-http-status-code-traces

Conversation

@shahidul5

Copy link
Copy Markdown

Summary

  • add tracing.ignored_http_status_codes, defaulting to [404]
  • drop only matching main-request transactions after the response is available
  • cover YAML, PHP, XML config and response behavior

Closes #958

Testing

  • Not run locally: PHP and Composer are unavailable in this environment.
  • git diff --check

Notes

  • This is a breaking default; release notes and the external configuration migration documentation should explain how to retain 404 transaction tracing by setting tracing.ignored_http_status_codes: [].

AI assistance

I used AI assistance to research, implement, and review this change; I verified the code structure and diff manually.

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.

Change default behavior to stop tracing 404s unless explicitly enabled

1 participant