Skip to content

fix: harden GitHub Actions supply chain - #3

Open
looztra wants to merge 3 commits into
wassimk:mainfrom
looztra:fix-action-supply-chain
Open

looztra wants to merge 3 commits into
wassimk:mainfrom
looztra:fix-action-supply-chain

Conversation

@looztra

@looztra looztra commented Sep 18, 2026

Copy link
Copy Markdown

Summary

  • Pin all GitHub Actions to immutable commits for the latest stable releases.
  • Restrict CI to contents: read and scope release contents: write to the release job.
  • Add Dependabot GitHub Actions updates with a seven-day default cooldown.
  • Disable checkout credential persistence, document release permissions, name jobs, and add concurrency limits (Zizmor findings).
  • Add a workflow-lint workflow that double-checks all workflows on every push to main and pull request, running reviewdog/action-actionlint and zizmorcore/zizmor-action in parallel jobs.

Security context

Addresses the mutable action reference supply-chain finding from the supplied security review gist. The separate local branch deletion behavior is intentionally out of scope.

Validation

  • git diff --check
  • go test ./...
  • actionlint
  • zizmor --pedantic .github/workflows (no findings)

Source

  • Branch: looztra/gh-sync:fix-action-supply-chain
  • Target: wassimk/gh-sync:main

looztra and others added 3 commits September 18, 2026 15:33
Pin workflow actions to immutable latest stable commits, scope release write permissions, and configure a seven-day Dependabot cooldown.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Disable checkout credential persistence, document and scope release permissions, name jobs, and serialize workflow runs.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
Run reviewdog/action-actionlint and zizmorcore/zizmor-action in parallel jobs on push to main and on pull requests, to continuously validate workflow changes.

Co-authored-by: Copilot App <223556219+Copilot@users.noreply.github.com>
@looztra

looztra commented Sep 18, 2026

Copy link
Copy Markdown
Author

@wassimk would you be ok to review and merge this PR so that we can prevent any supply chain attack?

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