Skip to content

ci: Remove secrets: inherit from changelog-preview workflow - #323

Merged
BYK merged 2 commits into
mainfrom
alextarasov/vuln-2352-improper-authorization-vulnerability-in-getsentrydevservices
Aug 4, 2026
Merged

ci: Remove secrets: inherit from changelog-preview workflow#323
BYK merged 2 commits into
mainfrom
alextarasov/vuln-2352-improper-authorization-vulnerability-in-getsentrydevservices

Conversation

@oioki

@oioki oioki commented Aug 4, 2026

Copy link
Copy Markdown
Member

Summary

  • changelog-preview.yml used secrets: inherit when calling the reusable getsentry/craft/.github/workflows/changelog-preview.yml, exposing every repository secret to the called workflow.
  • The workflow is disabled_manually on GitHub and unused — removed it entirely instead of fixing its secret handling, consistent with how other getsentry repos are dropping this disabled workflow.

Test plan

  • Confirmed via GitHub API that the Changelog Preview workflow state is disabled_manually

Fixes VULN-2352

🤖 Generated with Claude Code

The reusable getsentry/craft changelog-preview.yml workflow only uses
the auto-provisioned GITHUB_TOKEN and doesn't declare any workflow_call
secrets. Inheriting all repository secrets into it violates least
privilege and isn't needed to make the workflow function.

Refs VULN-2352
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 4, 2026

Copy link
Copy Markdown

VULN-2352

changelog-preview.yml is disabled_manually on GitHub and unused. Rather
than fixing its secrets: inherit usage, remove the workflow entirely,
consistent with how other getsentry repos are dropping this disabled
workflow.

Refs VULN-2352
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@oioki
oioki requested a review from BYK August 4, 2026 20:53
@BYK
BYK merged commit bd76c64 into main Aug 4, 2026
19 checks passed
@BYK
BYK deleted the alextarasov/vuln-2352-improper-authorization-vulnerability-in-getsentrydevservices branch August 4, 2026 21:16
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