Skip to content

ci: add workflow auditing and fix violations#413

Open
Saviq wants to merge 2 commits intomainfrom
actions-security-review
Open

ci: add workflow auditing and fix violations#413
Saviq wants to merge 2 commits intomainfrom
actions-security-review

Conversation

@Saviq
Copy link
Contributor

@Saviq Saviq commented Mar 19, 2026

No description provided.

@Saviq Saviq requested a review from a team as a code owner March 19, 2026 10:15
Copilot AI review requested due to automatic review settings March 19, 2026 10:15
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR tightens CI/workflow security posture by adding zizmor workflow auditing and applying zizmor-driven remediations across existing GitHub Actions workflows.

Changes:

  • Add zizmor as a pre-commit hook and introduce a .github/zizmor.yml config to manage rule exceptions.
  • Harden multiple GitHub Actions workflows by setting permissions: {} at the workflow level and granting least-privilege per job.
  • Pin third-party GitHub Actions to specific commit SHAs and disable persisted checkout credentials.

Reviewed changes

Copilot reviewed 7 out of 7 changed files in this pull request and generated 2 comments.

Show a summary per file
File Description
.pre-commit-config.yaml Adds zizmor pre-commit hook with --fix to audit/fix workflow issues locally.
.github/zizmor.yml Adds zizmor configuration to ignore specific workflows for secrets-outside-env.
.github/workflows/tics.yml Applies least-privilege permissions, pins actions, and disables persisted credentials.
.github/workflows/tarball.yml Applies least-privilege permissions, pins actions, and refactors inputs usage (introduces a quoting bug).
.github/workflows/spread.yml Applies least-privilege permissions, pins actions, and disables persisted credentials.
.github/workflows/pre-commit.yaml Applies least-privilege permissions and pins actions (but likely breaks auto-fix pushes).
.github/workflows/ppa-upload.yml Applies least-privilege permissions, pins actions, and disables persisted credentials.

@Saviq Saviq force-pushed the actions-security-review branch from 3bbbe47 to 7aeae77 Compare March 19, 2026 10:28
@Saviq Saviq force-pushed the actions-security-review branch from 7aeae77 to f1458ce Compare March 19, 2026 10:31
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