Skip to content

feat(attributes): Change sentry.description scrubbing policy to manual - #555

Merged
Lms24 merged 2 commits into
mainfrom
lms/ref-sentry.description-scrubbing-manual
Aug 10, 2026
Merged

feat(attributes): Change sentry.description scrubbing policy to manual#555
Lms24 merged 2 commits into
mainfrom
lms/ref-sentry.description-scrubbing-manual

Conversation

@Lms24

@Lms24 Lms24 commented Aug 5, 2026

Copy link
Copy Markdown
Member

This PR reduces the apply_scrubbing policy from auto to manual. We've observed span descriptions being scrubbed for example because a route name contained "auth" (e.g. /(auth)/lists/:id) or an HTML selector contained opacity (from a tailwind class, in this case users had city in their custom scrubbing fields).

Since description is inferred for v2 spans, the attributes matching the description should already get scrubbed (cc @Dav1dde or @loewenheim to confirm (UPDATE confirmed)), so it's redundant to auto-scrub again here.

@Lms24
Lms24 requested review from a team, cleptric, mjq and nsdeschenes as code owners August 5, 2026 06:47
@github-actions

github-actions Bot commented Aug 5, 2026

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 ✨

Op

  • Add handler op by msonnb in #554
  • Add router span operation by msonnb in #551

Other

  • (attributes) Change sentry.description scrubbing policy to manual by Lms24 in #555

Bug Fixes 🐛

  • (ops) Add http.client.stream to http.client name and description rules by msonnb in #553

Internal Changes 🔧

Deps

  • Bump dompurify from 3.4.12 to 3.4.13 by dependabot in #559
  • Bump js-yaml from 4.3.0 to 4.3.1 by dependabot in #560

Other

  • (docs) Add Sentry Astro SDK by Lms24 in #563
  • Unstale yarn.lock by Lms24 in #561

🤖 This preview updates automatically when you update the PR.

@Lms24 Lms24 self-assigned this Aug 5, 2026
@Lms24 Lms24 changed the title feat(attributes): Change sentry.description scrubbing to manual feat(attributes): Change sentry.description scrubbing policy to manual Aug 5, 2026
@Lms24
Lms24 merged commit 20bb8b1 into main Aug 10, 2026
16 checks passed
@Lms24
Lms24 deleted the lms/ref-sentry.description-scrubbing-manual branch August 10, 2026 16:02
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.

2 participants