fix(params): removing '[' and ']' from URL encode exclude characters (a…

…xios#3316)
HealGaren · May 19, 2023 · ec83976 · ec83976
1 parent 21a5ad3
commit ec83976
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 7 deletions.
diff --git a/lib/helpers/buildURL.js b/lib/helpers/buildURL.js
@@ -16,9 +16,7 @@ function encode(val) {
     replace(/%3A/gi, ':').
     replace(/%24/g, '$').
     replace(/%2C/gi, ',').
-    replace(/%20/g, '+').
-    replace(/%5B/gi, '[').
-    replace(/%5D/gi, ']');
+    replace(/%20/g, '+');
 }
 
 /**

diff --git a/test/specs/helpers/buildURL.spec.js b/test/specs/helpers/buildURL.spec.js
@@ -37,7 +37,7 @@ describe('helpers::buildURL', function () {
       foo: {
         bar: 'baz'
       }
-    })).toEqual('/foo?foo[bar]=baz');
+    })).toEqual('/foo?foo%5Bbar%5D=baz');
   });
 
   it('should support date params', function () {
@@ -48,10 +48,10 @@ describe('helpers::buildURL', function () {
     })).toEqual('/foo?date=' + date.toISOString());
   });
 
-  it('should support array params', function () {
+  it('should support array params with encode', function () {
     expect(buildURL('/foo', {
-      foo: ['bar', 'baz', null, undefined]
-    })).toEqual('/foo?foo[]=bar&foo[]=baz');
+      foo: ['bar', 'baz']
+    })).toEqual('/foo?foo%5B%5D=bar&foo%5B%5D=baz');
   });
 
   it('should support special char params', function () {