[Feature] OIDC should be exported and available in libnpmpublish
Nobody has claimed this yet.
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 76/100
- Issue type
- Feature
- Clarity
- Clearly specified
- Activity status
- Quiet
- Tech stack
- javascript, node.js
- Domain
- cli, developer-experience
Research direction
Start with workspaces/libnpmpublish/lib/index.js and the existing oidc.js implementation, then compare how libnpmpublish currently exposes publish and unpublish. Done means the OIDC function is available through the libnpmpublish package without downstream projects copying its implementation.
Written by the indexing model from the issue text.
Description
The OIDC function from oidc.js is not currently exported neither available from the libnpmpublish package. This mean that to get it working in Lerna/Lerna-Lite (I maintain the latter), we had to copy and reimplement the oidc.js into Lerna/Lerna-Lite to make it work.
There's 2 obvious issues related to this approach:
- duplicate code increasing Lerna/Lerna-Lite project size
- any changes made in
oidc.jsneeds to be resync manually in external projects like Lerna/Lerna-Lite- for example adding OIDC CircleCI support was added much later and this wasn't synched back to Lerna which means it doesn't currently work in Lerna (I did synch it myself in Lerna-Lite though)
- ref: Lerna
oidc.tsand Lerna-Liteoidc.ts - Lerna currently has an open issue related to their oidc being out of sync: https://github.com/lerna/lerna/issues/4358
libnpmpublish only exports 2 functions (publish and unpublish)
I mention Lerna/Lerna-Lite in here, but I'm sure there's few hundred projects that would benefit from having OIDC available directly in libnpmpublish
- Dominant language
- JavaScript
- Stars
- 10.1k
- Forks
- 4.7k
- Avg merge
- 2d 2h
- Merged PRs (30d)
- 21
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from npm/cli
-
[DOCS] `npm trust circle` docs should warn that OIDC token exchange in SSH reruns isn't supported OpenDocumentation Needs Triage
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
Documentation Needs Triage
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
-
Bug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
allowScripts Bug Needs Triage Priority 2
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
Similar issues
-
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
HarperFast/skills#96 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Automattic/studio#4908 ·
-
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 86/100
sugarlabs/musicblocks#8847 ·
-
client-controller-update ta-bot-triage team-money-movement
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
MetaMask/metamask-mobile#36594 ·