fix: strip non-ansi characters from registry config (#5808)

haoqunjiang · web-flow · commit 27db6bfaa8e7 · 2020-08-24T15:52:37.000+08:00
To deal with malformed stdout result retrieved from child processes. Fixes #5802
diff --git a/packages/@vue/cli/lib/util/ProjectPackageManager.js b/packages/@vue/cli/lib/util/ProjectPackageManager.js
@@ -5,6 +5,8 @@ const ini = require('ini')
 const minimist = require('minimist')
 const LRU = require('lru-cache')
 
+const stripAnsi = require('strip-ansi')
+
 const {
   chalk,
   execa,
@@ -152,6 +154,7 @@ class PackageManager {
       }
     }
 
+    this._registry = stripAnsi(this._registry).trim()
     return this._registry
   }
 
diff --git a/packages/@vue/cli/package.json b/packages/@vue/cli/package.json
@@ -55,6 +55,7 @@
     "resolve": "^1.17.0",
     "shortid": "^2.2.15",
     "slash": "^3.0.0",
+    "strip-ansi": "^6.0.0",
     "validate-npm-package-name": "^3.0.0",
     "vue": "^2.6.11",
     "vue-codemod": "^0.0.4",

Original file line number	Diff line number	Diff line change
`@@ -5,6 +5,8 @@ const ini = require('ini')`
`5`	`5`	`const minimist = require('minimist')`
`6`	`6`	`const LRU = require('lru-cache')`
`7`	`7`
	`8`	`+const stripAnsi = require('strip-ansi')`
	`9`	`+`
`8`	`10`	`const {`
`9`	`11`	`chalk,`
`10`	`12`	`execa,`
`@@ -152,6 +154,7 @@ class PackageManager {`
`152`	`154`	`}`
`153`	`155`	`}`
`154`	`156`
	`157`	`+ this._registry = stripAnsi(this._registry).trim()`
`155`	`158`	`return this._registry`
`156`	`159`	`}`
`157`	`160`