urllib.parse.urlsplit() must not lowercase() IPv6 scope value

[socketpair]

BPO	32323
Nosy	@bitdancer, @asvetlov, @socketpair
PRs	bpo-32323: urllib.parse.urlsplit() must not lowercase() IPv6 scope value #4867 [3.6] bpo-32323: urllib.parse.urlsplit() must not lowercase() IPv6 scope value (GH-4867) #4959

^{Note: these values reflect the state of the issue at the time it was migrated and might not reflect the current state.}

Show more details

GitHub fields:

assignee = None
closed_at = <Date 2017-12-21.12:58:46.744>
created_at = <Date 2017-12-14.15:30:39.799>
labels = ['3.7', 'type-bug', 'library']
title = 'urllib.parse.urlsplit() must not lowercase() IPv6 scope value'
updated_at = <Date 2017-12-21.17:19:58.223>
user = 'https://github.com/socketpair'

bugs.python.org fields:

activity = <Date 2017-12-21.17:19:58.223>
actor = 'r.david.murray'
assignee = 'none'
closed = True
closed_date = <Date 2017-12-21.12:58:46.744>
closer = 'asvetlov'
components = ['Library (Lib)']
creation = <Date 2017-12-14.15:30:39.799>
creator = 'socketpair'
dependencies = []
files = []
hgrepos = []
issue_num = 32323
keywords = ['patch']
message_count = 7.0
messages = ['308306', '308326', '308328', '308329', '308870', '308871', '308886']
nosy_count = 3.0
nosy_names = ['r.david.murray', 'asvetlov', 'socketpair']
pr_nums = ['4867', '4959']
priority = 'normal'
resolution = 'fixed'
stage = 'resolved'
status = 'closed'
superseder = None
type = 'behavior'
url = 'https://bugs.python.org/issue32323'
versions = ['Python 3.6', 'Python 3.7']

[socketpair]

qwe = urlsplit('http://[FE80::822a:a8ff:fe49:470c%Тест]:1234/keys')
qwe.hostname will be 'fe80::822a:a8ff:fe49:470c%тест'

Which is wrong.

correct value is

'fe80::822a:a8ff:fe49:470c%Тест'

so, IP-address is lowercased and zone id does not.

[bitdancer]

In quick search the only RFC reference to this I found was https://tools.ietf.org/id/draft-sweet-uri-zoneid-01.html, which doesn't match what you are requesting (not that urlsplit's current behavior matches that either).

Do you have RFC references?

[socketpair]

https://url.spec.whatwg.org/#host-representation -> Support for <zone_id> is intentionally omitted.

https://tools.ietf.org/html/rfc6874

[socketpair]

Also this:

http://potaroo.net/ietf/idref/draft-kitamura-ipv6-zoneid-free/

So, I'm confused. Will investigate.

[asvetlov]

New changeset fdb148f by Andrew Svetlov (Miss Islington (bot)) in branch '3.6':
bpo-32323: urllib.parse.urlsplit() must not lowercase() IPv6 scope value (GH-4867) (bpo-4959)
fdb148f

[asvetlov]

RFC doesn't specify a case for Zone ID, let's keep it untouched (no lowercasing)

[bitdancer]

I don't think a zone id in that form is actually valid in a URI, but I agree that not messing with whatever is there is probably the best policy as long as we aren't directly supporting whatever *is* valid.