Follow-up Comment #3, patch #5775 (project mldonkey): final patch, so far ;) * loads URLs with chars '()+
de-/encoding should look like this http://www.foo.foo/foo+foo.php?foo=foo foo encode to: http://www.foo.foo/foo%2Bfoo.php?foo=foo+foo and vice versa decode to first line ---- so only url.args should be decoded according to x-www-form-urlencoded (replaces '+' with ' ') the rest of the URL should be decoded RCF 1738 style I changed the default decoding method to raw RFC 1738 at some places in code we decode the whole URL with decode perhaps we shouldn't do that, see "network_parse_url" ---- the same goes for encode at the moment we encode all RFC (someone changed it and didn't correct the comment) in web interface this doesn't matter (browser encodes for us) but URLs submitted by telnet or gui are affected so dllink "http://foo.foo/foo?foo=foo foo" won't work with them, but I can live with that ;-) note: x-www-form-urlencoded is used by browsers when submitting input fields, try to wireshark a google search (file #12107) _______________________________________________________ Additional Item Attachment: File name: url_final.patch Size:6 KB _______________________________________________________ Reply to this item at: <http://savannah.nongnu.org/patch/?5775> _______________________________________________ Message sent via/by Savannah http://savannah.nongnu.org/ _______________________________________________ Mldonkey-users mailing list [email protected] http://lists.nongnu.org/mailman/listinfo/mldonkey-users
