Skip to content

fix(ci): harden release workflow against tag script injection#299

Merged
steipete merged 2 commits intosteipete:mainfrom
salmonumbrella:fix/release-tag-injection-289
Mar 7, 2026
Merged

fix(ci): harden release workflow against tag script injection#299
steipete merged 2 commits intosteipete:mainfrom
salmonumbrella:fix/release-tag-injection-289

Conversation

@salmonumbrella
Copy link
Copy Markdown
Contributor

@salmonumbrella salmonumbrella commented Feb 16, 2026

Summary

  • validate workflow_dispatch tag input against an expected release-tag pattern before use
  • avoid direct expression interpolation in shell by passing inputs.tag via env and quoting it in git checkout

Fixes #289.

Testing

  • make ci (currently fails on existing unrelated typecheck errors in internal/cmd/contacts_crud.go on latest main)

@steipete steipete force-pushed the fix/release-tag-injection-289 branch from c6a8bff to d99df22 Compare March 7, 2026 18:00
@steipete steipete merged commit 09bfedf into steipete:main Mar 7, 2026
2 of 5 checks passed
@steipete
Copy link
Copy Markdown
Owner

steipete commented Mar 7, 2026

Landed via temp rebase onto main.

  • Gate: actionlint .github/workflows/release.yml && make fmt && make lint && make test && make ci
  • Land commit: d99df22
  • Merge commit: 09bfedf

Thanks @salmonumbrella!

klodr pushed a commit to klodr/gogcli that referenced this pull request Apr 22, 2026
…te#299)

* fix(ci): harden workflow_dispatch tag handling in release

* fix(ci): land release workflow hardening note (steipete#299) (thanks @salmonumbrella)

---------

Co-authored-by: Peter Steinberger <steipete@gmail.com>
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.

Security: Fix script injection in release workflow via inputs.tag

2 participants