[issue7751] urllib.urlopen("///C|/foo/bar/spam.foo") IOError: [Errno 22]

[Senthil Kumaran]

Senthil Kumaran <orsenthil at gmail.com> added the comment:

cgohlke, thanks for the patches and sorry for the delay. The fix however is not to replace the %HH character of '|' with '|', in the nturl2path, but the keep the '|' as safe character in the urllib.urlopen.

-        fullurl = quote(fullurl, safe="%/:=&?~#+!$,;'@()*[]")
+        fullurl = quote(fullurl, safe="%/:=&?~#+!$,;'@()*[]|")

Fixed in the revision 78268 with tests added.

----------
assignee:  -> orsenthil
resolution:  -> fixed

_______________________________________
Python tracker <report at bugs.python.org>
<http://bugs.python.org/issue7751>
_______________________________________