Skip to content

feat/sentry masking #912

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 12 commits into
base: main
Choose a base branch
from
Open

feat/sentry masking #912

wants to merge 12 commits into from

Conversation

egelhaus
Copy link
Collaborator

@egelhaus egelhaus commented Aug 1, 2025

What kind of change does this PR introduce?

eg: Bug fix, feature, docs update, ...

Why was this change needed?

Please link to related issues when possible, and explain WHY you changed things, not WHAT you changed.

Other information:

eg: Did you discuss this change with anybody before working on it (not required, but can be a good idea for bigger changes). Any plans for the future, etc?

Checklist:

Put a "X" in the boxes below to indicate you have followed the checklist;

  • I have read the CONTRIBUTING guide.
  • I checked that there were not similar issues or PRs already open for this.
  • This PR fixes just ONE issue (do not include multiple issues or types of change in the same PR) For example, don't try and fix a UI issue and include new dependencies in the same PR.

Copy link

vercel bot commented Aug 1, 2025

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

1 Skipped Deployment
Name Status Preview Comments Updated (UTC)
postiz ⬜️ Ignored (Inspect) Aug 1, 2025 11:13am

@egelhaus egelhaus requested a review from Copilot August 1, 2025 09:37
Copy link

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR implements Sentry masking functionality to protect sensitive user data in session replays and error reports. The changes configure Sentry to selectively mask content while allowing manual control over what gets masked or unmasked.

  • Configures Sentry replay integration with manual masking controls instead of blanket text masking
  • Adds data-sentry-mask attributes to sensitive UI elements containing user data like emails, URLs, API keys, and names
  • Wraps the main preview content in a mask container to protect user-generated content

Reviewed Changes

Copilot reviewed 12 out of 12 changed files in this pull request and generated 3 comments.

Show a summary per file
File Description
libraries/react-shared-libraries/src/sentry/initialize.sentry.client.ts Updates Sentry configuration to enable manual masking with CSS selectors
apps/frontend/src/components/webhooks/webhooks.tsx Masks webhook names and URLs
apps/frontend/src/components/settings/teams.component.tsx Masks team member email identifiers
apps/frontend/src/components/settings/signatures.component.tsx Masks signature content previews
apps/frontend/src/components/sets/sets.tsx Masks set names
apps/frontend/src/components/public-api/public.component.tsx Masks API keys and endpoints
apps/frontend/src/components/preview/preview.wrapper.tsx Wraps preview content in mask container
apps/frontend/src/components/new-launch/picks.socials.component.tsx Masks social integration names
apps/frontend/src/components/new-launch/manage.modal.tsx Masks tags and customer selection components
apps/frontend/src/components/launches/launches.component.tsx Masks group and integration names
apps/frontend/src/components/autopost/autopost.tsx Masks autopost titles and URLs
apps/frontend/src/components/analytics/stars.table.component.tsx Masks analytics table and repository column header

Copy link
Contributor

@nevo-david nevo-david left a comment

Choose a reason for hiding this comment

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

Added some comments :)

@egelhaus egelhaus marked this pull request as ready for review August 1, 2025 10:07
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