Skip to content

ci: Add GitHub Actions security linting with zizmor#3

Merged
nbrieussel merged 7 commits intomainfrom
ci/add-zizmor-security-lint
Mar 22, 2026
Merged

ci: Add GitHub Actions security linting with zizmor#3
nbrieussel merged 7 commits intomainfrom
ci/add-zizmor-security-lint

Conversation

@nbrieussel
Copy link
Copy Markdown
Contributor

Contexte

This PR adds security-focused linting for GitHub Actions workflows to detect vulnerabilities, secrets, and dangerous patterns before they reach production.

Changements

  • Dependency: Added zizmor 0.5 to mise.toml
  • Workflow: Added zizmor job to existing lint-workflows workflow
    • Detects hardcoded secrets
    • Identifies excessive permissions
    • Flags dangerous expression injections
    • Reports security anti-patterns
  • Pre-push Hook: Updated to run both actionlint (syntax) and zizmor (security)

Bénéfices

  • Early detection of security vulnerabilities in workflows
  • Prevents accidental secret leaks
  • Enforces principle of least privilege for permissions
  • Complements GitHub's Push Protection

Tests

  • Run locally: zizmor .github/workflows/*.yml
  • Both actionlint and zizmor must pass pre-push
  • Check that both checks appear in GitHub Actions on PR

@nbrieussel nbrieussel force-pushed the ci/add-zizmor-security-lint branch 7 times, most recently from 711d93e to 5688b85 Compare March 22, 2026 16:39
@github-advanced-security
Copy link
Copy Markdown

You are seeing this message because GitHub Code Scanning has recently been set up for this repository, or this pull request contains the workflow file for the Code Scanning tool.

What Enabling Code Scanning Means:

  • The 'Security' tab will display more code scanning analysis results (e.g., for the default branch).
  • Depending on your configuration and choice of analysis tool, future pull requests will be annotated with code scanning analysis results.
  • You will be able to see the analysis results for the pull request's branch on this overview once the scans have completed and the checks have passed.

For more information about GitHub Code Scanning, check out the documentation.

Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
Comment thread .github/workflows/lint-workflows.yml Fixed
@nbrieussel nbrieussel marked this pull request as ready for review March 22, 2026 17:54
@nbrieussel nbrieussel enabled auto-merge (rebase) March 22, 2026 17:54
@nbrieussel nbrieussel force-pushed the ci/add-zizmor-security-lint branch from 0a9a361 to af4399a Compare March 22, 2026 17:55
@nbrieussel nbrieussel merged commit f460e76 into main Mar 22, 2026
5 checks passed
@nbrieussel nbrieussel deleted the ci/add-zizmor-security-lint branch March 22, 2026 17:55
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