Skip to content

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

Merged
oioki merged 2 commits into
masterfrom
alextarasov/vuln-2346-improper-authorization-vulnerability-in-getsentrysentry-php
Aug 4, 2026
Merged

ci: Remove secrets: inherit from changelog-preview workflow#2177
oioki merged 2 commits into
masterfrom
alextarasov/vuln-2346-improper-authorization-vulnerability-in-getsentrysentry-php

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-2346

🤖 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-2346
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@linear-code

linear-code Bot commented Aug 4, 2026

Copy link
Copy Markdown

VULN-2346

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-2346
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@oioki
oioki requested a review from BYK August 4, 2026 20:43
@oioki
oioki merged commit de5d0a1 into master Aug 4, 2026
53 checks passed
@oioki
oioki deleted the alextarasov/vuln-2346-improper-authorization-vulnerability-in-getsentrysentry-php branch August 4, 2026 20:54
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