using clone instead of JSON.parse(JSON.stringify(...)) errors when given undefined

Author: Thomas Hallock

lib/publish.js

@@ -7,6 +7,7 @@ var Stream = require('stream').Stream
 var assert = require('assert')
 var fixer = require('normalize-package-data').fixer
 var concat = require('concat-stream')
+var clone = require('clone')
 
 function escaped (name) {
   return name.replace('/', '%2f')
@@ -75,7 +76,7 @@ function putFirst (registry, data, tarbuffer, access, auth, cb) {
 
   if (!auth.token) {
     root.maintainers = [{ name: auth.username, email: auth.email }]
-    data.maintainers = JSON.parse(JSON.stringify(root.maintainers))
+    data.maintainers = clone(root.maintainers)
   }
 
   root.versions[ data.version ] = data
@@ -166,8 +167,7 @@ function putNext (registry, newVersion, root, current, auth, cb) {
         current[i] = root[i]
     }
   }
-  var maint = JSON.parse(JSON.stringify(root.maintainers))
-  root.versions[newVersion].maintainers = maint
+  root.versions[newVersion].maintainers = clone(root.maintainers)
 
   var uri = url.resolve(registry, escaped(root.name))
   var options = {

package.json

@@ -12,6 +12,7 @@
   },
   "dependencies": {
     "chownr": "^1.0.1",
+    "clone": "^1.0.2",
     "concat-stream": "^1.4.6",
     "graceful-fs": "^4.1.2",
     "mkdirp": "^0.5.0",

test/publish.js

@@ -5,6 +5,7 @@ var fs = require('fs')
 var server = require('./lib/server.js')
 var common = require('./lib/common.js')
 var client = common.freshClient()
+var clone = require('clone')
 
 function nop () {}
 
@@ -142,7 +143,7 @@ test('publish call contract', function (t) {
   )
 
   t.test('malformed semver in publish', function (t) {
-    var metadata = JSON.parse(JSON.stringify(METADATA))
+    var metadata = clone(METADATA)
     metadata.version = '%!@#$'
     var params = {
       metadata: metadata,