ci: Add Danger check to warn about auth token handling changes#6087
ci: Add Danger check to warn about auth token handling changes#6087
Conversation
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Semver Impact of This PR⚪ None (no version bump detected) 📋 Changelog PreviewThis is how your changes will appear in the changelog.
🤖 This preview updates automatically when you update the PR. |
|
That's a nice way to validate it |
📢 Type of change
📜 Description
Adds a Danger check that warns PR reviewers when changes touch code related to Sentry auth token handling (
authToken,SENTRY_AUTH_TOKEN,auth.token,auth_token).When matched, the check posts a warning comment listing the affected files, linking to the prior security advisory for context.
.github/files (which legitimately referenceSENTRY_AUTH_TOKENas a secret) andCHANGELOG.mddanger.git.structuredDiffForFile()— consistent with the org-wide Danger setup💡 Motivation and Context
Prevents future incidents like GHSA-68c2-4mpx-qh95 by surfacing auth token changes for extra review.
Closes #3683
💚 How did you test it?
authToken,getAuthToken,sentryAuthToken,SENTRY_AUTH_TOKEN,auth.token,auth_token)authenticate,authorization,reauthorize,tokenize)📝 Checklist
sendDefaultPIIis enabled🔮 Next steps