urllib.splituser(): '@' in usrname

[slyphon]

BPO	596581
Nosy	@gvanrossum, @rhettinger
Files	urllib.patch: a patch for urllib.py

^{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 = 'https://github.com/rhettinger'
closed_at = <Date 2002-08-20.19:52:14.000>
created_at = <Date 2002-08-17.21:55:10.000>
labels = ['extension-modules']
title = "urllib.splituser(): '@' in usrname"
updated_at = <Date 2002-08-20.19:52:14.000>
user = 'https://bugs.python.org/slyphon'

bugs.python.org fields:

activity = <Date 2002-08-20.19:52:14.000>
actor = 'gvanrossum'
assignee = 'rhettinger'
closed = True
closed_date = None
closer = None
components = ['Extension Modules']
creation = <Date 2002-08-17.21:55:10.000>
creator = 'slyphon'
dependencies = []
files = ['4519']
hgrepos = []
issue_num = 596581
keywords = ['patch']
message_count = 3.0
messages = ['40962', '40963', '40964']
nosy_count = 3.0
nosy_names = ['gvanrossum', 'rhettinger', 'slyphon']
pr_nums = []
priority = 'normal'
resolution = 'fixed'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue596581'
versions = ['Python 2.3']

[slyphon]

This was in response to "[ 581529 ] bug in
splituser(host) in urllib". This was a one-line
change. The issue was that if a username contained
the '@' symbol, the urllib.splituser(host) method
wouldn't return the correct information.

if you were to try
urllib.splituser(Jones@CrunchyFrog.net@Whizzo.com),
you'd get ['Jones', 'CrunchyFrog.net@Whizzo.com']

after applying this patch, you get
['Jones@CrunchyFrog.net', 'Whizzo.com']

So, there you are... :-)

[rhettinger]

Logged In: YES
user_id=80475

Applied as urllib.py 1.150 and 1.135.6.4
Closing patch and related bug.

[gvanrossum]

Logged In: YES
user_id=6380

Fine, but I'm baffled that someone would try this. RFC1738 says:

"Within the user and password field, any ":", "@", or "/"
must be encoded."