🔧 [PB-1278] Clear NODE_AUTH_TOKEN to enable OIDC authentication#18
🔧 [PB-1278] Clear NODE_AUTH_TOKEN to enable OIDC authentication#18
Conversation
|
Warning Rate limit exceeded@crisap94 has exceeded the limit for the number of commits that can be reviewed per hour. Please wait 4 minutes and 7 seconds before requesting another review. ⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. 📒 Files selected for processing (1)
Note 🎁 Summarized by CodeRabbit FreeYour organization is on the Free plan. CodeRabbit will generate a high-level summary and a walkthrough for each pull request. For a comprehensive line-by-line review, please upgrade your subscription to CodeRabbit Pro by visiting https://app.coderabbit.ai/login. Comment |
🔍 PR Quality Check Summary
📋 Checks Performed:
🎉 All checks passed! This PR is ready for review. |
Summary
Fixes OIDC authentication by explicitly clearing
NODE_AUTH_TOKENthatactions/setup-nodesets automatically.Root Cause - CONFIRMED
actions/setup-nodesetsNODE_AUTH_TOKENautomatically even withoutregistry-urlconfiguration. This default token interferes with OIDC authentication, causing npm to attempt token-based auth instead of OIDC.The Issue
From GitHub community discussion #176761:
Even though we:
id-token: writepermissionnpm publish(not pnpm)registry-urlfrom setup-nodesetup-node STILL sets NODE_AUTH_TOKEN, blocking OIDC.
Solution
Explicitly clear
NODE_AUTH_TOKENin the publish step:How It Works
setup-noderuns and sets defaultNODE_AUTH_TOKENNODE_AUTH_TOKEN: ''id-token: writeis used for authenticationComplete Configuration
Changes Made
File:
.github/workflows/deploy.ymlAdded
envblock to publish step:Why Previous Attempts Failed
pnpm publish→ pnpm doesn't support OIDC ❌registry-url→ Created conflicting token ❌registry-url→ setup-node still set default token ❌Verification
All requirements met:
id-token: writepermissionubidotsreact-html-canvasdeploy.ymlnpm-productionnpm publish(OIDC support)NODE_AUTH_TOKENcleared (no interference)References