Skip to content

fix(ci): resolve zizmor security findings in workflow#89

Merged
chorrell merged 1 commit into
mainfrom
fix/zizmor-findings
May 26, 2026
Merged

fix(ci): resolve zizmor security findings in workflow#89
chorrell merged 1 commit into
mainfrom
fix/zizmor-findings

Conversation

@chorrell
Copy link
Copy Markdown
Owner

@chorrell chorrell commented May 26, 2026

Fixes all findings reported by zizmor v1.25.2:

.github/workflows/docker-publish.yml

  • unpinned-uses (9 high): Pin all action references to commit SHAs with full version tag comments
  • excessive-permissions (3 medium): Add permissions: {} at workflow level; add explicit contents: read per job and packages: write for the push job
  • artipacked (2 low): Add persist-credentials: false to both actions/checkout steps

.github/dependabot.yml

  • dependabot-cooldown (2 medium): Add cooldown: default-days: 7 to both github-actions and docker ecosystems

@chorrell chorrell force-pushed the fix/zizmor-findings branch from 03e7ce6 to 8cc6499 Compare May 26, 2026 02:08
- Pin all action references to SHA hashes to prevent supply chain attacks
- Add permissions: {} at workflow level to deny all by default
- Add explicit job-level permissions (contents: read, packages: write)
- Set persist-credentials: false on checkout steps to prevent artipacked
- Add cooldown: default-days: 7 to all dependabot ecosystems
@chorrell chorrell force-pushed the fix/zizmor-findings branch from 8cc6499 to 51554c1 Compare May 26, 2026 02:09
@chorrell chorrell merged commit a00106d into main May 26, 2026
3 checks passed
@chorrell chorrell deleted the fix/zizmor-findings branch May 26, 2026 02:11
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