Skip to content

ci: Resolve Zizmor static security audit findings in workflows and composite actions - #249

Merged
vkarampudi merged 12 commits into
google:masterfrom
vkarampudi:fix-zizmor-security
Aug 14, 2026
Merged

ci: Resolve Zizmor static security audit findings in workflows and composite actions#249
vkarampudi merged 12 commits into
google:masterfrom
vkarampudi:fix-zizmor-security

Conversation

@vkarampudi

@vkarampudi vkarampudi commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

Summary of Changes:

Resolves static security analysis findings identified by Zizmor across the CI/CD workflows and composite action:

  • Fix Template Injection Vulnerability: Updated .github/reusable-build/action.yml to pass ${{ inputs.python-version }} via an environment variable (PYTHON_VERSION) instead of direct string expansion in a run: bash script block.
  • Pin Action References: Pinned all GitHub Action uses: steps (actions/checkout, actions/setup-python, actions/upload-artifact, actions/download-artifact, actions/cache, mamba-org/setup-micromamba, pre-commit/action, pypa/gh-action-pypi-publish) to immutable 40-character commit SHAs.
  • Explicit Permissions: Added top-level permissions: { contents: read } to lint.yml, conda-build.yml, and conda-test.yml to enforce the principle of least privilege.
  • Disable Credential Persistence: Added persist-credentials: false across all actions/checkout steps to prevent token leakage into subsequent build/test steps.

Testing:

  • Validated locally with zizmor:
    No findings to report. (1 ignored, 10 suppressed)
    

@rwitcher rwitcher left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

LGTM

@vkarampudi
vkarampudi merged commit 7efdb25 into google:master Aug 14, 2026
26 checks passed
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