sentry-docs icon indicating copy to clipboard operation
sentry-docs copied to clipboard

doc(relay): PII guide is for static mode

Open jjbayer opened this issue 1 year ago • 3 comments

Pre-merge checklist

If you work at Sentry, you're able to merge your own PR without review, but please don't unless there's a good reason.

  • [ ] Checked Vercel preview for correctness, including links
  • [x] PR was reviewed and approved by any necessary SMEs
  • [ ] PR was reviewed and approved by a member of the Sentry docs team

Description of changes

In order to clarify that editing PII settings in project configs is for Relay Static Mode only:

  1. Move the guide under "Relay Modes,
  2. add "Static Mode" to the title.

Fixes https://github.com/getsentry/sentry/issues/67755.

jjbayer avatar Mar 28 '24 14:03 jjbayer

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
sentry-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Apr 2, 2024 7:47am

vercel[bot] avatar Mar 28 '24 14:03 vercel[bot]

Bundle Report

Changes will increase total bundle size by 27 bytes :arrow_up:

Bundle name Size Change
sentry-docs-server 6.11MB 36 bytes :arrow_up:
sentry-docs-client 5.52MB 6 bytes :arrow_down:
sentry-docs-edge-server 618.05kB 3 bytes :arrow_down:

codecov[bot] avatar Mar 28 '24 14:03 codecov[bot]

+1 on adding PII scrubbing examples! that would be super useful!

vivianyentran avatar Mar 28 '24 17:03 vivianyentran

Would it also make sense, either here or as a separate PR, to add PII scrubbing examples for managed mode?

@cstavitsky added a paragraph about managed mode now. In managed mode, rules are configured through the UI and then propagated to the Relay instances running on the user's premises.

jjbayer avatar Apr 02 '24 07:04 jjbayer