parameters: Fix for Decoding Query Parameters

[lroolle]

I spotted an inconsistency in how we decode query parameters, especially around the + character. Here's a quick breakdown:

In Python:

from urllib.parse import urlencode
print(urlencode({'p_string': '+ +'}))  # Outputs: 'p_string=%2B+%2B'

In JS:

console.log(decodeURIComponent('%2B+%2B'))  // Outputs: '+++'

Versions:

• Python: 3.11.4
• Node: v18.15.0

What I did:

• Used the URL object's searchParams for the extraction of query parameters.

The solution is from: Cloudflare community post.

[G4brym]

Hey @lroolle we cannot use the URL object to parse query parameters because it doesn't support receiving multiple parameters with the same name, used in arrays

That was why you could fix the unit tests in your pr

here is a little example of what i'm talking

params = new URL('https://localhost?p_string=%2B+%2B&p_string=another').searchParams

params.get('p_string')

It prints only + + ehre it should have printed ['+ +', 'another']

I will take a look into the issue you are having and solve it in the next release

[G4brym]

Hey @lroolle i've pushed a new release (v1.0.1) that fixes your problem

[lroolle]

Hey @lroolle i've pushed a new release (v1.0.1) that fixes your problem

thx, hey I've just take a short review of your fixing. It appears that multiple parameters with the same name might not be an issue. I've only heard that some form parameters may use this format?