Skip to content

Drop setup-node registry-url so npm uses OIDC #3

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
cprice70 merged 2 commits into from
Oct 13, 2026
Merged

Drop setup-node registry-url so npm uses OIDC #3

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
84ab738
Drop setup-node registry-url so npm uses OIDC
cprice70 Oct 12, 2026
6cef842
Log npm publish at verbose level to expose OIDC exchange errors
cprice70 Oct 12, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 5 additions & 1 deletion .github/workflows/release.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -16,10 +16,11 @@ jobs:
steps:
- uses: actions/checkout@v4

# No registry-url here: with it, setup-node writes an .npmrc with a
# placeholder NODE_AUTH_TOKEN and npm uses that instead of OIDC.
- uses: actions/setup-node@v4
with:
node-version: 22.x
registry-url: https://registry.npmjs.org
cache: npm

# Trusted publishing (OIDC) needs npm 11.5.1 or newer; Node 22 ships an older npm.
Expand All @@ -32,3 +33,6 @@ jobs:

- name: Publish to npm
run: npm publish --access public
env:
# Surfaces the "oidc" log lines that explain a failed token exchange.
NPM_CONFIG_LOGLEVEL: verbose
Loading