Skip to content

feat(attributes): Add sentry.status and sentry.trace.status#453

Open
mjq wants to merge 1 commit into
mainfrom
mjq/span-trace-status
Open

feat(attributes): Add sentry.status and sentry.trace.status#453
mjq wants to merge 1 commit into
mainfrom
mjq/span-trace-status

Conversation

@mjq

@mjq mjq commented Jun 25, 2026

Copy link
Copy Markdown
Member

Description

Both of these attributes are set during ingestion. For v2 spans they can only be "ok" or "error"; for v1 spans they have a longer list of valid attributes.

⚠️ I set these both to apply_scrubbing: never because they're meant to be hardcoded values and scrubbing would break them. Maybe these should be manual anyway though?

PR Checklist

  • I have run yarn test and verified that the tests pass.
  • I have run yarn generate to generate and format code and docs.

If an attribute was added:

  • The attribute is in a namespace (e.g. nextjs.function_id, not function_id)
  • I have used the correct value for apply_scrubbing (i.e. manual or auto. Use never only for values that should never be scrubbed such as IDs)

@github-actions

Copy link
Copy Markdown

Semver Impact of This PR

🟡 Minor (new features)

📋 Changelog Preview

This is how your changes will appear in the changelog.
Entries from this PR are highlighted with a left border (blockquote style).


New Features ✨

  • (attributes) Add sentry.status and sentry.trace.status by mjq in #453

🤖 This preview updates automatically when you update the PR.

@mjq mjq marked this pull request as ready for review June 25, 2026 20:18
@mjq mjq requested review from a team, Lms24, cleptric and nsdeschenes as code owners June 25, 2026 20:18
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.

3 participants