fix(login): properly save scope if defined

setCredentialsByURI was clobbering the saving of the scope:registry config
npm · Jun 7, 2021 · 7787755 · 7787755
1 parent a49b8d7
commit 7787755
Showing 1 changed file with 1 addition and 3 deletions.
diff --git a/lib/adduser.js b/lib/adduser.js
@@ -75,11 +75,9 @@ class AddUser extends BaseCommand {
 
   async updateConfig ({ newCreds, registry, scope }) {
     this.npm.config.delete('_token', 'user') // prevent legacy pollution
-
+    this.npm.config.setCredentialsByURI(registry, newCreds)
     if (scope)
       this.npm.config.set(scope + ':registry', registry, 'user')
-
-    this.npm.config.setCredentialsByURI(registry, newCreds)
     await this.npm.config.save('user')
   }
 }