[bug #17116] HTTP: Can not download URLs containing "+"

orbit <[email protected]>
Newsgroups gmane.network.mldonkey.bugs
Message-ID <[email protected]>
Follow-up Comment #1, bug #17116 (project mldonkey):

for testing de-/encoding
http://www.blooberry.com/indexdot/html/topics/urlencoding.htm

i don't could test it yet but this could help

diff -u -r1.6 url.ml
--- src/utils/lib/url.ml	25 Feb 2006 21:09:03 -0000	1.6
+++ src/utils/lib/url.ml	2 Mar 2007 23:24:52 -0000
@@ -42,7 +42,7 @@
     else Char.chr (Char.code '0' + x) in
   for i=0 to len-1 do
     match s.[i] with
-    | 'a'..'z' | 'A'..'Z' | '0'..'9' | '.' | '-' | '*' | '_' ->
+    | 'a'..'z' | 'A'..'Z' | '0'..'9' | '.' | '-' | '*' | '_' | '(' | ')' ->
         res.[!pos] <- s.[i]; incr pos
 (*    | ' ' -> res.[!pos] <- '+'; incr pos *)
     | c ->
@@ -68,7 +68,6 @@
   let rec iter i =
     if i < len then
       match s.[i] with
-      | '+' -> Buffer.add_char r  ' '; iter (i+1)
       | '%' ->
           let n = 
             try

* why should '+' be replaced by ' '?
* to the list above ' should be added but I'm to lazy to look up how to write
it, ''' won't work i guess ;)

    _______________________________________________________

Reply to this item at:

  <http://savannah.nongnu.org/bugs/?17116>

_______________________________________________
  Nachricht geschickt von/durch Savannah
  http://savannah.nongnu.org/
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.