[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[Mldonkey-users] [patch #5775] [FIX] load URLs with ()'
From: |
orbit |
Subject: |
[Mldonkey-users] [patch #5775] [FIX] load URLs with ()' |
Date: |
Wed, 07 Mar 2007 00:07:40 +0000 |
User-agent: |
Mozilla/5.0 (X11; U; Linux i686; en-US; rv:1.8.1.1) Gecko/20061205 Iceweasel/2.0.0.1 (Debian-2.0.0.1+dfsg-2) |
Follow-up Comment #7, patch #5775 (project mldonkey):
no, patch doesn't break it
and I only assume it doesn't work (couldn't find such an URL)
ATM space is encoded %20 most servers should decode that to space
I think this line can't hurt either
diff -u -r1.36 http_client.ml
--- src/utils/net/http_client.ml 4 Feb 2007 17:27:45 -0000 1.36
+++ src/utils/net/http_client.ml 6 Mar 2007 23:51:25 -0000
@@ -90,8 +90,7 @@
then Url.to_string_no_args url
else url.short_file
in
- (* I get a lot more bittorrent urls with this line: *)
- let url = (Str.global_replace (Str.regexp " ") "%20" url) in
+ let url = Url.encode url in
let url = if is_real_post then url else
Url.put_args url args
in
anyway this encode decode URL code is a mess because it's used on many
places, without conception
e.g.: submits get decoded twice, some requests don't get encoded (see patch
above)
_______________________________________________________
Reply to this item at:
<http://savannah.nongnu.org/patch/?5775>
_______________________________________________
Message sent via/by Savannah
http://savannah.nongnu.org/
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/02
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/03
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/03
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/04
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/04
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/04
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', spiralvoice, 2007/03/06
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()',
orbit <=
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', spiralvoice, 2007/03/11
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', spiralvoice, 2007/03/11
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', orbit, 2007/03/16
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', spiralvoice, 2007/03/17
- [Mldonkey-users] [patch #5775] [FIX] load URLs with ()', spiralvoice, 2007/03/17
[Mldonkey-users] Adding Web Services to mldonkey, Alex, 2007/03/18