Skip to content

CI: Harden GHA configuration - #7

Merged
Max Rakitin (mrakitin) merged 6 commits into
NSLS2:mainfrom
tacaswell:harden_gha
Jul 9, 2025
Merged

CI: Harden GHA configuration#7
Max Rakitin (mrakitin) merged 6 commits into
NSLS2:mainfrom
tacaswell:harden_gha

Conversation

@tacaswell

@tacaswell Thomas A Caswell (tacaswell) commented Jul 9, 2025

Copy link
Copy Markdown
Contributor

Apply recommended hardening steps including:

  • pinning to a SHA any actions used
  • not persisting the read token on checkout
  • setting the default permissions to read-only

This adjusts the defaults per suggestions of zizmor to
reduce possible risks from giving GHA tasks more permissions
that required.
This eliminates the possibility of a tag being changed under
us.
This eliminates the possibility of a tag being changed under
us.
This eliminates the possibility of a tag being changed under
us.
Reduces risk of arbitrary code is run by attacker.
@tacaswell Thomas A Caswell (tacaswell) changed the title "CI: Harden GHA configuration" CI: Harden GHA configuration Jul 9, 2025
@codecov-commenter

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@mrakitin
Max Rakitin (mrakitin) merged commit 6ff5a88 into NSLS2:main Jul 9, 2025
7 checks passed
@tacaswell
Thomas A Caswell (tacaswell) deleted the harden_gha branch June 25, 2026 18:53
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.

3 participants