Skip to content

security: add zizmor check for unpinned GitHub Actions - #429

Merged
P0NDER0SA merged 1 commit into
mainfrom
security/add-zizmor-pinned-actions-check
Aug 27, 2026
Merged

security: add zizmor check for unpinned GitHub Actions#429
P0NDER0SA merged 1 commit into
mainfrom
security/add-zizmor-pinned-actions-check

Conversation

@P0NDER0SA

Copy link
Copy Markdown
Contributor

Adds the same check_pinned_actions.yml enforcement workflow already used in notification-terraform (and other repos in this initiative) to this repo.

Uses zizmor to scan .github/workflows (and .github/actions if present) on every PR touching those paths, and fails the check if any unpinned-uses finding is reported (i.e. an action referenced by a mutable tag/branch instead of a full commit SHA).

Ran zizmor locally against this repo's current workflows/actions — 0 unpinned-uses findings, so this is enabled with a clean baseline (everything here is already SHA-pinned via #427 and the codeql.yml workflow).

Copilot AI lite review requested due to automatic review settings August 27, 2026 13:45

Copilot AI 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.

Pull request overview

Adds a repository-level enforcement workflow to prevent GitHub Actions from being referenced via mutable refs (tags/branches) by running zizmor and failing the check when unpinned-uses findings are present. This aligns this repo with the org-wide SHA-pinning initiative referenced in #427.

Changes:

  • Introduces .github/workflows/check_pinned_actions.yml to scan workflows/actions paths on PRs and main pushes.
  • Installs uv and runs uvx zizmor to produce JSON output, then filters for unpinned-uses findings and emits actionable error output.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@P0NDER0SA
P0NDER0SA merged commit f634020 into main Aug 27, 2026
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.

3 participants