Skip to content

Commit 959e1c8

Browse files
authored
Replace query-string with qs (#11425)
1 parent fbec064 commit 959e1c8

File tree

8 files changed

+59
-141
lines changed

8 files changed

+59
-141
lines changed

core/base-service/redirector.js

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,7 +1,7 @@
11
import camelcase from 'camelcase'
22
import emojic from 'emojic'
33
import Joi from 'joi'
4-
import queryString from 'query-string'
4+
import qs from 'qs'
55
import BaseService from './base.js'
66
import {
77
serverHasBeenUpSinceResourceCached,
@@ -92,12 +92,16 @@ export default function redirector(attrs) {
9292
let urlSuffix = ask.uri.search || ''
9393

9494
if (transformQueryParams) {
95-
const specifiedParams = queryString.parse(urlSuffix)
95+
const specifiedParams = qs.parse(urlSuffix, {
96+
ignoreQueryPrefix: true,
97+
})
9698
const transformedParams = transformQueryParams(namedParams)
9799
const redirectParams = overrideTransformedQueryParams
98100
? Object.assign(transformedParams, specifiedParams)
99101
: Object.assign(specifiedParams, transformedParams)
100-
const outQueryString = queryString.stringify(redirectParams)
102+
const outQueryString = qs.stringify(redirectParams, {
103+
strictNullHandling: true,
104+
})
101105
urlSuffix = `?${outQueryString}`
102106
}
103107

core/base-service/redirector.spec.js

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -216,7 +216,7 @@ describe('Redirector', function () {
216216

217217
expect(statusCode).to.equal(301)
218218
expect(headers.location).to.equal(
219-
'/new/service/hello-world.svg?style=flat-square&token=def456',
219+
'/new/service/hello-world.svg?token=def456&style=flat-square',
220220
)
221221
})
222222
})

0 commit comments

Comments
 (0)