🐛 Fix: tencent cos upload error

PicGo · Aug 17, 2022 · 61df53a · 61df53a
1 parent 879b98d
commit 61df53a
Show file tree

Hide file tree

Showing 4 changed files with 19 additions and 6 deletions.
diff --git a/package.json b/package.json
@@ -15,8 +15,8 @@
     "start": "node ./bin/picgo",
     "lint": "eslint src/**/*.ts && npm run build",
     "test": "echo \"Error: no test specified\" && exit 1",
-    "build": "rimraf ./dist && rollup -c rollup.config.js",
-    "dev": "rollup -c rollup.config.js -w",
+    "build": "cross-env NODE_ENV=production rimraf ./dist && rollup -c rollup.config.js",
+    "dev": "cross-env NODE_ENV=development rollup -c rollup.config.js -w",
     "patch": "npm version patch && git push origin master && git push origin --tags",
     "minor": "npm version minor && git push origin master && git push origin --tags",
     "major": "npm version major && git push origin master && git push origin --tags",
@@ -74,6 +74,7 @@
     "builtins": "^4.0.0",
     "conventional-changelog": "^3.0.6",
     "copyfiles": "^2.1.0",
+    "cross-env": "^7.0.3",
     "cz-customizable": "^5.10.0",
     "eslint": "7",
     "eslint-config-standard-with-typescript": "^18.0.2",

diff --git a/rollup.config.js b/rollup.config.js
@@ -6,7 +6,6 @@ import { string } from 'rollup-plugin-string'
 import json from '@rollup/plugin-json'
 import builtins from 'builtins'
 import replace from '@rollup/plugin-replace'
-
 const version = process.env.VERSION || pkg.version
 const sourcemap = 'inline'
 const banner = `/*
@@ -33,7 +32,7 @@ const commonOptions = {
         }
       }
     }),
-    terser(),
+    // terser(),
     commonjs(),
     string({
       // Required to be specified
@@ -52,6 +51,12 @@ const commonOptions = {
   input
 }
 
+const isDev = process.env.NODE_ENV === 'development'
+
+if (!isDev) {
+  commonOptions.plugins.push(terser())
+}
+
 /** @type import('rollup').RollupOptions */
 const nodeCjs = {
   output: [{

diff --git a/src/plugins/uploader/tcyun.ts b/src/plugins/uploader/tcyun.ts
@@ -71,7 +71,7 @@ const postOptions = (options: ITcyunConfig, fileName: string, signature: ISignat
   } else {
     return {
       method: 'PUT',
-      url: `http://${options.bucket}.cos.${options.area}.myqcloud.com/${encodeURI(path)}${encodeURI(fileName)}`,
+      url: `http://${options.bucket}.cos.${options.area}.myqcloud.com/${encodeURIComponent(path)}${encodeURIComponent(fileName)}`,
       headers: {
         Host: `${options.bucket}.cos.${options.area}.myqcloud.com`,
         Authorization: `q-sign-algorithm=sha1&q-ak=${options.secretId}&q-sign-time=${signature.signTime}&q-key-time=${signature.signTime}&q-header-list=host&q-url-param-list=&q-signature=${signature.signature}`,

diff --git a/yarn.lock b/yarn.lock
@@ -1703,6 +1703,13 @@ create-require@^1.1.0:
   resolved "https://registry.yarnpkg.com/create-require/-/create-require-1.1.1.tgz#c1d7e8f1e5f6cfc9ff65f9cd352d37348756c333"
   integrity sha512-dcKFX3jn0MpIaXjisoRvexIJVEKzaq7z2rZKxf+MSr9TkdmHmsU4m2lcLojrj/FHl8mk5VxMmYA+ftRkP/3oKQ==
 
+cross-env@^7.0.3:
+  version "7.0.3"
+  resolved "https://registry.npmjs.org/cross-env/-/cross-env-7.0.3.tgz#865264b29677dc015ba8418918965dd232fc54cf"
+  integrity sha512-+/HKd6EgcQCJGh2PSjZuUitQBQynKor4wrFbRg4DtAgS1aWO+gU52xpH7M9ScGgXSYmAVS9bIJ8EzuaGw0oNAw==
+  dependencies:
+    cross-spawn "^7.0.1"
+
 cross-spawn@^5.0.1:
   version "5.1.0"
   resolved "https://registry.yarnpkg.com/cross-spawn/-/cross-spawn-5.1.0.tgz#e8bd0efee58fcff6f8f94510a0a554bbfa235449"
@@ -1723,7 +1730,7 @@ cross-spawn@^6.0.0, cross-spawn@^6.0.5:
     shebang-command "^1.2.0"
     which "^1.2.9"
 
-cross-spawn@^7.0.2, cross-spawn@^7.0.3:
+cross-spawn@^7.0.1, cross-spawn@^7.0.2, cross-spawn@^7.0.3:
   version "7.0.3"
   resolved "https://registry.yarnpkg.com/cross-spawn/-/cross-spawn-7.0.3.tgz#f73a85b9d5d41d045551c177e2882d4ac85728a6"
   integrity sha512-iRDPJKUPVEND7dHPO8rkbOnPpyDygcDFtWjpeWNCgy8WP2rXcxXL8TskReQl6OrB2G7+UJrags1q15Fudc7G6w==