Skip to content

fix(ci): resolve zizmor security findings in GitHub Actions workflows - #6601

Draft
rohityan wants to merge 1 commit into
google:mainfrom
rohityan:fix/zizmor-security-findings
Draft

fix(ci): resolve zizmor security findings in GitHub Actions workflows#6601
rohityan wants to merge 1 commit into
google:mainfrom
rohityan:fix/zizmor-security-findings

Conversation

@rohityan

@rohityan rohityan commented Aug 5, 2026

Copy link
Copy Markdown
Collaborator

This PR resolves 58 security and workflow linting findings identified by zizmor across 13 GitHub Actions workflow files.

Summary of Fixes Applied

  • artipacked (15 instances): Added persist-credentials: false to all actions/checkout steps.
  • unpinned-uses (18 instances): Pinned third-party and GitHub action references to explicit 40-character commit SHAs.
  • ref-version-mismatch (9 instances): Updated version comments to match exact release tags (# v6.0.3, # v6.2.0, # v6.4.0).
  • template-injection (12 instances): Replaced inline ${{ ... }} expression expansions in run: scripts with step-level env: variables.
  • excessive-permissions (2 instances): Added explicit permissions: blocks to block-merge.yml and issue-maintenance.yml.
  • dangerous-triggers (1 instance): Scoped permissions and avoided untrusted head code execution on pull_request_target in pr-triage.yml.
  • use-trusted-publishing (1 instance): Added id-token: write permission for PyPI Trusted Publishing in release-publish.yml.

@rohityan rohityan added the workflow [Component] This issue is related to ADKworkflow label Aug 5, 2026
@rohityan rohityan assigned rohityan and unassigned rohityan Aug 5, 2026
@rohityan
rohityan marked this pull request as draft August 5, 2026 20:20
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

workflow [Component] This issue is related to ADKworkflow

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant