Skip to content

fix(ci): correct release-tag trust policy claims#519

Merged
jack-edmonds-dd merged 1 commit into
mainfrom
fix/release-tag-policy
May 20, 2026
Merged

fix(ci): correct release-tag trust policy claims#519
jack-edmonds-dd merged 1 commit into
mainfrom
fix/release-tag-policy

Conversation

@jack-edmonds-dd
Copy link
Copy Markdown
Collaborator

For pull_request: closed events, GitHub OIDC uses the base-branch context, so the actual claims are ref: refs/heads/main and job_workflow_ref: ...@refs/heads/main — not refs/pull/N/merge as the initial policy assumed. Also tighten via head_ref regex and base_ref for defense in depth.

What does this PR do?

Motivation

Additional Notes

Checklist

  • The code change follows the project conventions (see CONTRIBUTING.md)
  • Tests have been added/updated (if applicable)
  • Documentation has been updated (if applicable)
  • All CI checks pass
  • Code coverage is maintained or improved

Related Issues

For `pull_request: closed` events, GitHub OIDC uses the base-branch
context, so the actual claims are `ref: refs/heads/main` and
`job_workflow_ref: ...@refs/heads/main` — not `refs/pull/N/merge` as
the initial policy assumed. Also tighten via `head_ref` regex and
`base_ref` for defense in depth.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
@jack-edmonds-dd jack-edmonds-dd requested a review from a team as a code owner May 20, 2026 16:51
@jack-edmonds-dd jack-edmonds-dd merged commit 83ece28 into main May 20, 2026
7 checks passed
@jack-edmonds-dd jack-edmonds-dd deleted the fix/release-tag-policy branch May 20, 2026 17:09
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