You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The current set of options for auth-type has numerous values which overlap (ex. legacy, sso, saml, oauth). Because of how the CLI expects the registry to handle authentication, it is essentially the server's job to interpret & request follow-up information from the end-user. The CLI handles prompts for step-up & MFA prompts utilizing a forward-looking polling strategy. With the introduction of web, there should only be two potential values for auth-type moving forward (legacy & web). If users have previously been defining sso, saml or oauthwe should be directing them to using legacy.
Bike Shedding / Questions
Do we want to change the name of the default value since it now is expected to be more all-encompassing? (ex. should legacy be changed to something like default)
nlf
changed the title
BREAKING CHANGE(config): Remove auth-type
BREAKING CHANGE(config): Remove all options but legacy and webauthn from auth-typeJun 1, 2022
Summary
The current set of options for
auth-type
has numerous values which overlap (ex.legacy
,sso
,saml
,oauth
). Because of how the CLI expects the registry to handle authentication, it is essentially the server's job to interpret & request follow-up information from the end-user. The CLI handles prompts for step-up & MFA prompts utilizing a forward-looking polling strategy. With the introduction ofweb
, there should only be two potential values forauth-type
moving forward (legacy
&web
). If users have previously been definingsso
,saml
oroauth
we should be directing them to usinglegacy
.Bike Shedding / Questions
legacy
be changed to something likedefault
)Actions
sso
,saml
&oauth
configurations & codelegacy
config value with team & at an RFC callcli/lib/auth/legacy.js
(ref. https://github.com/npm/cli/blob/f9abee79abe541226a249f50bdeec41317dd5712/lib/auth/legacy.js) intonpm-profile
Links & References
The text was updated successfully, but these errors were encountered: