Skip to content

ci: Move the DSN for the jest project to github secrets#116776

Merged
ryan953 merged 2 commits into
masterfrom
ryan953/ci-secret-sentry-jest-dsn
Jun 4, 2026
Merged

ci: Move the DSN for the jest project to github secrets#116776
ryan953 merged 2 commits into
masterfrom
ryan953/ci-secret-sentry-jest-dsn

Conversation

@ryan953

@ryan953 ryan953 commented Jun 3, 2026

Copy link
Copy Markdown
Member

Some external folks are sending data into our project. its not a huge problem because we have special envs that let us filter out the random stuff. but someone could easily r-euse our envs and then we would be in trouble.

The new DSN is configured in sentry already: https://sentry.sentry.io/settings/projects/jest/keys/232ba39ed1567f1852c02e79600c3779/
The value is in github here: https://github.com/getsentry/sentry/settings/secrets/actions.

After this lands, maybe a few days later, we can disable or revoke the old key.

Some external folks are sending data into our project. its not a huge problem because we have special envs that let us filter out the random stuff. but someone could easily r-euse our envs and then we would be in trouble.

the value is already in github here: https://github.com/getsentry/sentry/settings/secrets/actions
@ryan953 ryan953 requested a review from a team as a code owner June 3, 2026 16:29
@ryan953 ryan953 requested a review from a team June 3, 2026 16:29
@github-actions github-actions Bot added the Scope: Frontend Automatically applied to PRs that change frontend components label Jun 3, 2026

@cursor cursor Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

❌ Bugbot Autofix is OFF. To automatically fix reported issues with cloud agents, enable autofix in the Cursor dashboard.

Reviewed by Cursor Bugbot for commit 9670c23. Configure here.

Comment thread jest.config.ts
Pass the SENTRY_JEST_DSN secret as SENTRY_DSN env var so the jest
balancer job can report to Sentry, matching the frontend workflow.

Co-Authored-By: Claude Opus 4.6 <noreply@example.com>
@ryan953 ryan953 requested a review from a team as a code owner June 3, 2026 21:21
@ryan953 ryan953 merged commit 0b5bb7c into master Jun 4, 2026
77 checks passed
@ryan953 ryan953 deleted the ryan953/ci-secret-sentry-jest-dsn branch June 4, 2026 17:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Scope: Frontend Automatically applied to PRs that change frontend components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants