fix: remove replace %20 to '+' (#489)

* fix: remove replace %20 to '+' * fix : eslint
ali-sdk · Jun 13, 2018 · 22363d0 · 22363d0
1 parent bf546ce
commit 22363d0
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 4 deletions.
diff --git a/lib/browser/client.js b/lib/browser/client.js
@@ -365,7 +365,7 @@ proto._getReqUrl = function _getReqUrl(params) {
 
  if (params.object) {
  // Preserve '/' in result url
- reourcePath += this._escape(params.object);
+ reourcePath += this._escape(params.object).replace(/\+/g, '%2B');
  }
  ep.pathname = reourcePath;
 
@@ -391,7 +391,7 @@ proto._getReqUrl = function _getReqUrl(params) {
  ep.query = query;
 
  // As '%20' is not recognized by OSS server, we must convert it to '+'.
- return urlutil.format(ep).replace(/%20/g, '+');
+ return urlutil.format(ep);
 };
 
 /*

diff --git a/lib/client.js b/lib/client.js
@@ -352,7 +352,7 @@ proto._getReqUrl = function _getReqUrl(params) {
 
  if (params.object) {
  // Preserve '/' in result url
- resourcePath += this._escape(params.object);
+ resourcePath += this._escape(params.object).replace(/\+/g, '%2B');
  }
  ep.pathname = resourcePath;
 
@@ -378,7 +378,7 @@ proto._getReqUrl = function _getReqUrl(params) {
  ep.query = query;
 
  // As '%20' is not recognized by OSS server, we must convert it to '+'.
- return urlutil.format(ep).replace(/%20/g, '+');
+ return urlutil.format(ep);
 };
 
 /*