Skip to content

Commit 382e61f

Browse files
committed
fix(publish-cross-org): drop NPM_TOKEN fallback, trusted-publishing only
Mirrors socket-addon@4caf99f's parallel change — see that commit for the rationale. socket-bin's allowlist is empty today, so the first real publish will be the first one that hits this trust path; better to ship with no fallback than to discover the fallback was relied upon when the OIDC path was broken.
1 parent 74cdc79 commit 382e61f

1 file changed

Lines changed: 7 additions & 1 deletion

File tree

.github/workflows/publish-cross-org.yml

Lines changed: 7 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -62,7 +62,13 @@ jobs:
6262
RELEASE_TAG: ${{ inputs.release-tag }}
6363
DRY_RUN: ${{ inputs.dry-run }}
6464
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
65-
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
65+
# Auth is trusted-publishing only: `npm publish --provenance`
66+
# mints an OIDC token from this workflow's `id-token: write`
67+
# permission and npm verifies against the package's trusted-
68+
# publisher config (configured per-package on npmjs.com,
69+
# pointing at SocketDev/socket-bin + this workflow path).
70+
# No NPM_TOKEN env on purpose — a long-lived token would
71+
# widen the trust gate.
6672
run: node scripts/publish-cross-org.mts
6773

6874
- name: Summarize publish

0 commit comments

Comments
 (0)