Message 99635 - Python tracker

➜

This issue tracker has been migrated to GitHub, and is currently read-only.
For more information, see the GitHub FAQs in the Python's Developer Guide.

Author	orsenthil
Recipients	cgohlke, eric.araujo, orsenthil
Date	2010-02-20.22:07:27
SpamBayes Score	0.00015905836
Marked as misclassified	No
Message-id	<1266703649.02.0.425920672411.issue7751@psf.upfronthosting.co.za>
In-reply-to

Content
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.

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.

History
Date	User	Action	Args
2010-02-20 22:07:29	orsenthil	set	recipients: + orsenthil, eric.araujo, cgohlke
2010-02-20 22:07:29	orsenthil	set	messageid: <1266703649.02.0.425920672411.issue7751@psf.upfronthosting.co.za>
2010-02-20 22:07:27	orsenthil	link	issue7751 messages
2010-02-20 22:07:27	orsenthil	create