Skip to content

ci: give the registry job least privilege and pin its action to a commit - #453

Merged
Mearman merged 2 commits into
mainfrom
ci/registry-job-least-privilege
Oct 22, 2026
Merged

Mearman merged 2 commits into
mainfrom
ci/registry-job-least-privilege

Conversation

@Mearman

@Mearman Mearman commented Oct 22, 2026

Copy link
Copy Markdown
Member

The workflow-level permissions grant write on contents, issues and pull requests to every job. The registry job needs a read checkout and the OIDC token the registry login uses, and it runs an action whose tag can move, so it now declares exactly those permissions and pins the action to the commit of its release, as GitHub's secure use guidance recommends. Dependabot's github-actions updates keep the pin current.

The pin is on the release that logs each retried failure's cause, so a slow publish is readable from the run afterwards.

The workflow-level permissions grant write on contents, issues and pull
requests to every job. The registry job needs a read checkout and the OIDC
token the registry login uses, and it runs an action whose tag can move,
so it now declares exactly those permissions and pins the action to the
commit of its release, as GitHub's secure use guidance recommends.
@chatgpt-codex-connector

Copy link
Copy Markdown

Codex Review Summary

This comment shows the latest Codex review activity on this pull request.

Review Status Commit Review trigger
🔒 Security Review ⚠️ Failed 2026-10-22T08:50:08.091109Z
We couldn’t complete this review. Comment @codex security review to try again.
911bca9 PR opened
ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review" or "@codex security review".

Codex reacts with 👀 while any review is running, comments if it has suggestions, and reacts with 👍 once all reviews finish with no findings.

@chatgpt-codex-connector

Copy link
Copy Markdown

This security review couldn’t start because the paying account has exhausted its Codex allowance and has insufficient available credits. The payer can check their balance and add credits in the Codex usage dashboard, then try again. If you do not manage the paying account, contact this repository's admins.

@Mearman
Mearman merged commit 0953d70 into main Oct 22, 2026
8 checks passed
@Mearman
Mearman deleted the ci/registry-job-least-privilege branch October 22, 2026 08:55
@github-actions

Copy link
Copy Markdown
Contributor

🎉 This PR is included in version 8.15.14 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant