Skip to content

ci(signing): scope promote-main-to-stable cosign identity to this repo - #382

Open
kylerankin wants to merge 1 commit into
projectbluefin:mainfrom
kylerankin:fix/cosign-identity-scope
Open

kylerankin wants to merge 1 commit into
projectbluefin:mainfrom
kylerankin:fix/cosign-identity-scope

Conversation

@kylerankin

Copy link
Copy Markdown

Problem

.github/workflows/promote-main-to-stable.yml passed the shared release gate with:

cosign_identity_regexp: >-
  ^https://github\.com/${{ github.repository_owner }}/[^/]+/\.github/workflows/

The [^/]+ wildcard matches any repository under the org, so the cosign
verification of the :testing image accepts a keyless signature produced by
any projectbluefin repo's workflow, not just this repository's
build-image.yml. Since run_e2e: false (#281), the cosign check is
effectively the only functional gate on stable promotion, so its identity
scope should be as tight as possible.

Fix

Scope the gate identity to this repository:

cosign_identity_regexp: >-
  ^https://github\.com/${{ github.repository }}/\.github/workflows/

Refs: #378

— hive: backend=pi model=lemonade/Ornith-1.5-35B-A3B-GGUF-Q6_K

The cosign_identity_regexp used ${{ github.repository_owner }}/[^/]+ which
accepted a keyless cosign signature from any repo in the org. Scope it to
this repository with ${{ github.repository }} so only finpilot's own
workflows can satisfy the stable-promotion release gate.

Refs: projectbluefin#378

Assisted-by: lemonade/Ornith-1.5-35B-A3B-GGUF via pi
Signed-off-by: kylerankin <kylerankin@users.noreply.github.com>

@kubestellar-hive kubestellar-hive 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.

Reviewed — no findings from this perspective.

— hive: agent=reviewer backend=copilot model=claude-fable-5 copilot=1.0.78

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.

1 participant