Incorrect parsing of query parameters with %3B (;) inside

[shturman]

Some test below:

u = yarl.URL('/?user_agent=Mozilla%2F5.0+%28iPhone%3B+CPU+iPhone+OS+10_1_1+like+Mac+OS+X%29+AppleWebKit%2F602.2.14+%28KHTML%2C+like+Gecko%29+Mobile%2F14B100')
assert len(u.query.items()) == 1

But I'm getting 2 params.

[asvetlov]

I don't follow where do you have two params?

[shturman]

assert is failing due to

len(u.query.items()) == 2

[asvetlov]

Got it.
Thank you for report

[asvetlov]

Well, that's how urllib.parse.parse_sql works: split on both & and ;.

Not sure if this behavior should be fixed.

[shturman]

But that's why ';' was encoded with %3B.
The same way I can encode '&' with %26 and it works perfect. But with ';' is not.

len(yarl.URL('/?a=one%3Btwo').query) != len(urllib.parse.parse_qsl('a=one%3Btwo'))

[asvetlov]

Sorry for my stupidity.
I'll fix it ASAP.

[shturman]

np

[shturman]

Any updates on this issue? aiohttp query parsing works incorrectly due to it (

[fafhrd91]

i will push new release today