fix(auth): send username in correct key (#3)

Fixes: https://npm.community/t/npm-6-6-0-broke-authentication-with-npm-registry-couchapp/4752
npm · Jan 22, 2019 · 38422d0 · 38422d0
1 parent 0af23f2
commit 38422d0
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 8 deletions.
diff --git a/package-lock.json b/package-lock.json
diff --git a/publish.js b/publish.js
@@ -88,7 +88,7 @@ function patchedManifest (spec, auth, base, opts) {
     // currently gets filled in by the npm registry itself, based on auth
     // information.
     manifest._npmUser = {
-      username: auth.username,
+      name: auth.username,
       email: auth.email
     }
   }

diff --git a/test/publish.js b/test/publish.js
@@ -197,7 +197,7 @@ test('retry after a conflict', t => {
           _nodeVersion: process.versions.node,
           _npmVersion: '6.9.0',
           _npmUser: {
-            username: 'other',
+            name: 'other',
             email: 'other@idk.tech'
           },
           name: 'libnpmpublish',
@@ -282,7 +282,7 @@ test('retry after a conflict -- no versions on remote', t => {
           _nodeVersion: process.versions.node,
           _npmVersion: '6.9.0',
           _npmUser: {
-            username: 'other',
+            name: 'other',
             email: 'other@idk.tech'
           },
           name: 'libnpmpublish',
@@ -432,7 +432,7 @@ test('publish with basic auth', t => {
           _nodeVersion: process.versions.node,
           _npmVersion: '6.9.0',
           _npmUser: {
-            username: 'zkat',
+            name: 'zkat',
             email: 'kat@example.tech'
           },
           maintainers: [{