philfr added the comment:

This fix introduces a nasty side-effect: "GET http://server//file"; (with
two /s) does not work anymore. It returns the directory index instead.

This is because urlparse is not applied to an URL, but to its right-hand
part starting at the path.

urlparse.urlparse("http://server//foo";)[2] correctly returns //foo, but 
urlparse.urlparse("//foo")[2] (as used in this library) returns an empty
string.

So the first proposed fix (msg27195) would be better. Or maybe this is
an urlparse issue, so that it should be able to process such a partial url.

----------
nosy: +philfr

_____________________________________
Tracker <[EMAIL PROTECTED]>
<http://bugs.python.org/issue1394565>
_____________________________________
_______________________________________________
Python-bugs-list mailing list 
Unsubscribe: 
http://mail.python.org/mailman/options/python-bugs-list/archive%40mail-archive.com

Reply via email to