fix: additional path escaping issue #80 (#81)

liuweiGL · Jan 9, 2024 · a3bbe6c · a3bbe6c
1 parent 6bada0c
commit a3bbe6c
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/.vscode/settings.json b/.vscode/settings.json
@@ -1,5 +1,5 @@
 {
   "editor.codeActionsOnSave": {
-    "source.fixAll": true
+    "source.fixAll": "explicit"
   }
 }
diff --git a/plugin/mkcert/index.ts b/plugin/mkcert/index.ts
@@ -159,7 +159,7 @@ class Mkcert {
     }
 
     const mkcertBinary = await this.getMkcertBinary()
-    const commandStatement = `${mkcertBinary} -CAROOT`
+    const commandStatement = `${escape(mkcertBinary)} -CAROOT`
 
     debug(`Exec ${commandStatement}`)
 
@@ -204,7 +204,7 @@ class Mkcert {
     await ensureDirExist(this.savePath)
     await this.retainExistedCA()
 
-    const cmd = `${mkcertBinary} -install -key-file ${escape(
+    const cmd = `${escape(mkcertBinary)} -install -key-file ${escape(
       this.keyFilePath
     )} -cert-file ${escape(this.certFilePath)} ${names}`