Skip to content

ci: fix zizmor audit findings#145

Merged
dsanders11 merged 1 commit intomainfrom
ci/zizmor-audit-fixes
Apr 2, 2026
Merged

ci: fix zizmor audit findings#145
dsanders11 merged 1 commit intomainfrom
ci/zizmor-audit-fixes

Conversation

@MarshallOfSound
Copy link
Copy Markdown
Member

Fixes zizmor audit findings surfaced in the latest scheduled audit run.

Changes:

  • dependabot.yml: add 7-day cooldown for github-actions updates (auto-fix)
  • build.yml / test.yml: set persist-credentials: false on actions/checkout (artipacked)
  • release.yml: add top-level permissions: {} and explicit contents: read for the reusable test/build job calls (excessive-permissions)

Remaining finding (left as-is):

  • dangerous-triggers on add-to-project.ymlpull_request_target is required so the workflow can authenticate as a GitHub App and add newly opened PRs/issues to the Ecosystem WG project board. The workflow only listens for types: opened, runs with permissions: {}, and never checks out PR-controlled code, so the typical pull_request_target risk does not apply.

cc @dsanders11

@dsanders11 dsanders11 marked this pull request as ready for review April 2, 2026 20:01
@dsanders11 dsanders11 requested a review from a team as a code owner April 2, 2026 20:01
@dsanders11 dsanders11 merged commit efc99ff into main Apr 2, 2026
6 checks passed
@dsanders11 dsanders11 deleted the ci/zizmor-audit-fixes branch April 2, 2026 20:02
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