Bug fix 548176: urlparse('http://foo?blah') errs

[staschuk]

BPO	712317
Files	urlparse-netloc-end-at-qmark.patch

^{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 2005-01-09.15:32:47.000>
created_at = <Date 2003-03-30.20:16:56.000>
labels = ['library']
title = "Bug fix 548176: urlparse('http://foo?blah') errs"
updated_at = <Date 2005-01-09.15:32:47.000>
user = 'https://bugs.python.org/staschuk'

bugs.python.org fields:

activity = <Date 2005-01-09.15:32:47.000>
actor = 'jlgijsbers'
assignee = 'jlgijsbers'
closed = True
closed_date = None
closer = None
components = ['Library (Lib)']
creation = <Date 2003-03-30.20:16:56.000>
creator = 'staschuk'
dependencies = []
files = ['5145']
hgrepos = []
issue_num = 712317
keywords = ['patch']
message_count = 3.0
messages = ['43240', '43241', '43242']
nosy_count = 3.0
nosy_names = ['jlgijsbers', 'staschuk', 'mrovner']
pr_nums = []
priority = 'normal'
resolution = 'accepted'
stage = None
status = 'closed'
superseder = None
type = None
url = 'https://bugs.python.org/issue712317'
versions = ['Python 2.3']

[staschuk]

For detailed description of the problem, see
http://www.python.org/sf/548176
In summary, URLs such as
http://www.example.com?query=spam
are misparsed by urlparse.urlparse, which decides that
everything after the '//' is the host name. This is contrary to
RFC 2396 and probably contrary to the intent of RFC 1738.

The patch corrects the problem, adds a test to expose it,
and rearranges some of the tests to better exercise the
code in question.

[mrovner]

Logged In: YES
user_id=162094

-1. See my comment at http://www.python.org/sf/548176

[jlgijsbers]

Logged In: YES
user_id=469548

See bug bpo-548176 for why this patch is okay anyway. Checked
in on maint24 and HEAD.