Skip to content

fix: harden GitHub Actions workflows#37

Merged
lelia merged 1 commit into
mainfrom
fix/zizmor-workflow-security
Mar 25, 2026
Merged

fix: harden GitHub Actions workflows#37
lelia merged 1 commit into
mainfrom
fix/zizmor-workflow-security

Conversation

@reberhardt7
Copy link
Copy Markdown
Contributor

@reberhardt7 Ryan Eberhardt (reberhardt7) commented Mar 25, 2026

Summary

  • Template injection fixes: Replaced direct ${{ }} expression interpolation in shell commands with environment variables to prevent script injection attacks (in release.yml)
  • Pinned actions to SHA: All third-party actions (actions/checkout, actions/setup-python, pypa/gh-action-pypi-publish, actions/github-script) are now pinned to full-length commit SHAs instead of mutable tags
  • Restricted permissions: Added top-level permissions blocks to all workflows, applying least-privilege token scopes
  • Hardened checkout: Added persist-credentials: false to all actions/checkout steps
  • Added zizmor.yml: Configuration file for the zizmor GitHub Actions security linter

- Fix template injection vulnerabilities by using environment variables
  instead of direct expression interpolation in shell commands
- Pin all third-party actions to full-length commit SHAs
- Add top-level permissions blocks to restrict token scope
- Add persist-credentials: false to checkout steps
- Add zizmor.yml configuration file

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@reberhardt7 Ryan Eberhardt (reberhardt7) requested a review from a team as a code owner March 25, 2026 04:02
@reberhardt7 Ryan Eberhardt (reberhardt7) changed the title fix: harden GitHub Actions workflows (zizmor) fix: harden GitHub Actions workflows Mar 25, 2026
@lelia lelia self-assigned this Mar 25, 2026
@lelia
Copy link
Copy Markdown
Contributor

lelia commented Mar 25, 2026

Note: The PR Preview workflow for this project has been failing since this Dependabot PR was merged ~9 months ago, but should not be blocking for the purposes of this PR.

@lelia lelia merged commit 971f0f9 into main Mar 25, 2026
5 of 6 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.

2 participants