From 6d84189349c43b1dcdd977b522610660cc4c7042 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?=EC=B5=9C=EC=98=88=EC=B0=AC?= Date: Sat, 2 Aug 2025 03:44:47 +0900 Subject: [PATCH] fix(params): removing '[' and ']' from URL encode exclude characters (#3316) (#5715) Co-authored-by: Jay --- lib/helpers/buildURL.js | 4 +--- test/specs/helpers/buildURL.spec.js | 8 ++++---- 2 files changed, 5 insertions(+), 7 deletions(-) diff --git a/lib/helpers/buildURL.js b/lib/helpers/buildURL.js index 5c5eb579..4f9c0d11 100644 --- 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 index 2d166e1d..cee95c62 100644 --- 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 () {