Skip to content

security: Run zizmor on CI, resolve actions - #611

Merged
djkoloski merged 3 commits into
rkyv:mainfrom
andrewpollack:andrewpollack/gha-persist-credentials-false
Jun 12, 2025
Merged

djkoloski merged 3 commits into
rkyv:mainfrom
andrewpollack:andrewpollack/gha-persist-credentials-false

Conversation

@andrewpollack

Copy link
Copy Markdown
Contributor

zizmor runs static analysis on GitHub actions with proposed remediation steps. I ran on rkyv, and attached remediations for each:

djkoloski and others added 3 commits June 12, 2025 16:11
zizmor runs static analysis on GitHub actions with proposed remediation steps.
I ran on rkyv, and attached remediations for each:
* https://docs.zizmor.sh/audits/#artipacked actions/checkout will by default persist git
  configuration for the duration of the workflow, which is not necessary in this case.
https://docs.zizmor.sh/audits/#excessive-permissions detects excessive permissions in
workflows, both at the workflow level and individual job levels.
@djkoloski
djkoloski force-pushed the andrewpollack/gha-persist-credentials-false branch from f74079b to 940519a Compare June 12, 2025 20:11
@djkoloski
djkoloski merged commit 64cb16b into rkyv:main Jun 12, 2025
s3bk pushed a commit to s3bk/rkyv that referenced this pull request Aug 24, 2026
* Appease rustc

* security: Run zizmor on CI, resolve actions

zizmor runs static analysis on GitHub actions with proposed remediation steps.
I ran on rkyv, and attached remediations for each:
* https://docs.zizmor.sh/audits/#artipacked actions/checkout will by default persist git
  configuration for the duration of the workflow, which is not necessary in this case.

* security: resolve excessive-permissions zizmor

https://docs.zizmor.sh/audits/#excessive-permissions detects excessive permissions in
workflows, both at the workflow level and individual job levels.

---------

Co-authored-by: David Koloski <djkoloski@gmail.com>
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