Skip to content

Fixed CI injection risk and proved sealed bundle handoff end to end - #13

Merged
kridaydave merged 1 commit into
mainfrom
fix/ci-injection-and-seal-handoff
Sep 16, 2026
Merged

kridaydave merged 1 commit into
mainfrom
fix/ci-injection-and-seal-handoff

Conversation

@kridaydave

Copy link
Copy Markdown
Contributor

PR made by union-alpha from opencode on behalf of Kriday.

The dogfood job interpolated `${{ github.head_ref }}) straight into a shell command, so a crafted branch name could run arbitrary code on the runner. The sealed-bundle test also stopped at encryption: nothing proved a maintainer could actually open the artifact.

Summary :

  • Pass the branch name through an env var instead of shell interpolation
  • Extend the separate-keyring seal test to the full handoff: sender cannot decrypt (NO_SECKEY), recipient decrypts with VALIDSIG against the sender fingerprint and DECRYPTION_OKAY
  • Assert the received bundle matches the source HEAD, still holds the private .env blob, and carries a correct MANIFEST.json

@kridaydave
kridaydave merged commit 72eeded into main Sep 16, 2026
5 checks passed
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