Skip to content

CI(security): harden GitHub Actions workflows - #329

Merged
AlexV525 merged 2 commits into
mainfrom
fix/zizmor-security-hardening
Aug 13, 2026
Merged

CI(security): harden GitHub Actions workflows#329
AlexV525 merged 2 commits into
mainfrom
fix/zizmor-security-hardening

Conversation

@AlexV525

Copy link
Copy Markdown
Member

Summary

Hardens GitHub Actions workflows to resolve zizmor CI/CD security findings reported in fluttercandies/security-scanner#11.

Changes

All changes are limited to .github/workflows/.

artipacked (Low) — persist-credentials: false

Added persist-credentials: false to every actions/checkout step across all workflows. None of these workflows push back via git using the checkout's own token, so this is always safe.

  • codeql.yml — 1 checkout step
  • publish.yml — 1 checkout step
  • publishable.yml — 1 checkout step
  • runnable.yml — 3 checkout steps

excessive-permissions (Medium) — workflow-level permissions:

codeql.yml: hoisted the existing job-level permissions: block (under the single analyze job) up to workflow level. The block (security-events: write, packages: read, actions: read, contents: read) is unchanged in content — only its placement moved from job-level to workflow-level.

The other three workflows (publish.yml, publishable.yml, runnable.yml) already have a restrictive workflow-level permissions: {} and needed no change.

Not addressed

  • stale-action-refs (Low): All action references are already pinned to commit SHAs (the secure pattern). The Low finding only notes that some SHAs do not resolve to a release tag — this is informational, not a security defect, and is intentionally left as-is.

Resolves zizmor findings in fluttercandies/security-scanner#11.

Comment thread .github/workflows/codeql.yml Outdated
Move the CodeQL permissions (security-events: write, packages: read,
actions: read, contents: read) back to the analyze job and set the
workflow-level default to permissions: {} (least privilege). This
keeps the zizmor excessive-permissions finding resolved while matching
the repo convention.
@AlexV525
AlexV525 merged commit f7fa5fd into main Aug 13, 2026
7 checks passed
@AlexV525
AlexV525 deleted the fix/zizmor-security-hardening branch August 13, 2026 08: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.

2 participants