]> matita.cs.unibo.it Git - helm.git/commitdiff
- bugfix: typo in XSL patching
authorStefano Zacchiroli <zack@upsilon.cc>
Thu, 26 Dec 2002 15:48:36 +0000 (15:48 +0000)
committerStefano Zacchiroli <zack@upsilon.cc>
Thu, 26 Dec 2002 15:48:36 +0000 (15:48 +0000)
- bugfix: fixed parameters passed to 'wget' command
- moved some comment from .ml to .mli

helm/http_getter/http_getter_common.ml
helm/http_getter/http_getter_common.mli

index 67d72f4f16148c19681caf99e85dfa76f427a8b0..c076c2e70d24d94bb691797f7107decdcf8186c8 100644 (file)
@@ -65,7 +65,7 @@ let patch_xsl =
     Pcre.replace
       ~pat:(sprintf "%s\\s+href=\"" tag)
       ~templ:(
-        sprintf "%s href=\"%s/getxslt?uri=" Http_getter_env.my_own_url tag)
+        sprintf "%s href=\"%s/getxslt?uri=" tag Http_getter_env.my_own_url)
       line
   in
   let (patch_import, patch_include) =
@@ -79,28 +79,21 @@ let patch_dtd line =
       sprintf "ENTITY $1 SYSTEM \"%s/getdtd?uri=" Http_getter_env.my_own_url)
     line
 
-let pp_error =
-  sprintf "<html><body><h1>Http Getter error: %s</h1></body></html>"
-let pp_internal_error =
-  sprintf "<html><body><h1>Http Getter Internal error: %s</h1></body></html>"
-let pp_msg = sprintf "<html><body><h1>%s</h1></body></html>"
+let pp_error s =
+  sprintf "<html><body><h1>Http Getter error: %s</h1></body></html>" s
+let pp_internal_error s =
+  sprintf "<html><body><h1>Http Getter Internal error: %s</h1></body></html>" s
+let pp_msg s = sprintf "<html><body><h1>%s</h1></body></html>" s
+let null_pp s = s
 
 let mk_return_fun pp_fun contype msg outchan =
   Http_daemon.respond
-    ~body:(pp_fun msg)
-    ~headers:["Content-Type", contype]
-    outchan
+    ~body:(pp_fun msg) ~headers:["Content-Type", contype] outchan
 
 let return_html_error = mk_return_fun pp_error "text/html"
 let return_html_internal_error = mk_return_fun pp_internal_error "text/html"
 let return_html_msg = mk_return_fun pp_msg "text/html"
-let return_xml_msg = mk_return_fun pp_msg "text/xml"
-  (**
-  @param fname name of the file to be sent
-  @param contype Content-Type header value
-  @param contenc Content-Enconding header value
-  @param patch_fun function used to patch file contents
-  @param outchan output channel over which sent file fname *)
+let return_xml_msg = mk_return_fun null_pp "text/xml"
 let return_file ~fname ?contype ?contenc ?(patch_fun = fun x -> x) outchan =
   let headers =
     match (contype, contenc) with
@@ -109,21 +102,21 @@ let return_file ~fname ?contype ?contenc ?(patch_fun = fun x -> x) outchan =
     | (None, Some e) -> [ "Content-Enconding", e ]
     | (None, None) -> []
   in
-  Http_daemon.send_basic_headers outchan;
+  Http_daemon.send_basic_headers ~code:200 outchan;
   Http_daemon.send_headers headers outchan;
   Http_daemon.send_CRLF outchan;
   Http_getter_misc.iter_file
     (fun line -> output_string outchan (patch_fun line ^ "\n"))
     fname
- (* return a bad request http response *)
 let return_400 body outchan = Http_daemon.respond_error ~code:400 ~body outchan
 
 let wget ?output url =
   let flags =
-    (match output with Some file -> ["-O " ^ file] | None -> []) @ [url]
+    (match output with Some file -> ["-O"; file] | None -> []) @ [url]
   in
   Shell.call
-    ~stdout:Shell.to_dev_null ~stderr:Shell.to_dev_null [Shell.cmd "wget" flags]
+    ~stdout:Shell.to_dev_null ~stderr:Shell.to_dev_null
+    [Shell.cmd "wget" flags]
 
   (* TODO gzip and gunzip create executables file, but umask seems to be
   correctly inherited from the shell .... boh *)
index 323c71d218eedbbd1c680d7e99674cb3f609aa2a..63c37ea01da54c5bc918ec7554ad33e951c68a3d 100644 (file)
@@ -43,7 +43,14 @@ val return_html_error: string -> out_channel -> unit
 val return_html_internal_error: string -> out_channel -> unit
 val return_html_msg: string -> out_channel -> unit
 val return_xml_msg: string -> out_channel -> unit
+ (** return a bad request http response *)
 val return_400: string -> out_channel -> unit
+  (**
+  @param fname name of the file to be sent
+  @param contype Content-Type header value
+  @param contenc Content-Enconding header value
+  @param patch_fun function used to patch file contents
+  @param outchan output channel over which sent file fname *)
 val return_file:
   fname:string ->
   ?contype:string -> ?contenc:string -> ?patch_fun:(string -> string) ->