sortQueryParameters encodes the query string

[freearhey]

Is it supposed to be like this?

sortQueryParameters: true

normalizeUrl('http://sindresorhus.com/?url=http://example.com', {
    sortQueryParameters: true
});
//=> 'http://sindresorhus.com/?url=http%3A%2F%2Fexample.com'

sortQueryParameters: false

normalizeUrl('http://sindresorhus.com/?url=http://example.com', {
    sortQueryParameters: false
});
//=> 'http://sindresorhus.com/?url=http://example.com'

[ludofischer]

That's unforunately expected since normalize-url uses URL.searchParams() under the hood which follows different rules from the IETF URI spec since it is based on the WHATWG URL spec https://nodejs.org/api/url.html#urlsearchparams.

[freearhey]

Got it.

[freearhey]

why closed.

because my question was answered

how to resolve

pass the output string through the decodeURIComponent():

decodeURIComponent('http://sindresorhus.com/?url=http%3A%2F%2Fexample.com') 
// => 'http://sindresorhus.com/?url=http://example.com'

[ludofischer]

pass the output string through the decodeURIComponent():

decodeURIComponent('http://sindresorhus.com/?url=http%3A%2F%2Fexample.com') 
// => 'http://sindresorhus.com/?url=http://example.com'

Wow! Good find. But what if the original URL parameters were encoded?

[ludofischer]

I've made a PR to always decode the parameters, since after all the package name starts with normalize- :-)