Skip to content

feat(span-fields): Hide internal attributes#116384

Open
nsdeschenes wants to merge 2 commits into
masterfrom
nd/EXP-973/feat-span-fields-hide-internal-attributes
Open

feat(span-fields): Hide internal attributes#116384
nsdeschenes wants to merge 2 commits into
masterfrom
nd/EXP-973/feat-span-fields-hide-internal-attributes

Conversation

@nsdeschenes
Copy link
Copy Markdown
Contributor

The goal of this PR is to hide "internal" attributes as defined in sentry conventions from being returned in API responses.

Closes EXP-973

nsdeschenes and others added 2 commits May 28, 2026 11:56
Use the API-facing span attribute visibility helper for the legacy spans fields endpoint so internal Sentry convention attributes are not exposed to non-staff users.

Refs EXP-973
Co-Authored-By: Codex <noreply@openai.com>
Add endpoint coverage for the legacy spans fields API so internal Sentry convention attributes remain hidden while public attributes are returned.

Refs EXP-973
Co-Authored-By: Codex <noreply@openai.com>
@linear-code
Copy link
Copy Markdown

linear-code Bot commented May 28, 2026

EXP-973

@nsdeschenes
Copy link
Copy Markdown
Contributor Author

@cursor review

@github-actions github-actions Bot added the Scope: Backend Automatically applied to PRs that change backend components label May 28, 2026
Copy link
Copy Markdown
Contributor

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

✅ Bugbot reviewed your changes and found no new issues!

Comment @cursor review or bugbot run to trigger another review on this PR

Reviewed by Cursor Bugbot for commit 524f459. Configure here.

@nsdeschenes nsdeschenes marked this pull request as ready for review May 28, 2026 16:12
@nsdeschenes nsdeschenes requested review from a team as code owners May 28, 2026 16:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Backend Automatically applied to PRs that change backend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants