fix

src/app.ml

@@ -42,14 +42,6 @@ let open_registration =
 
 let () = Dream.log "open_registration: %b" open_registration
 
-let hostname =
-  match Scfg.Query.get_dir "hostname" config with
-  | None -> failwith "no `hostname` in configuration file"
-  | Some hostname ->
-    Result.fold ~error:failwith ~ok:Fun.id (Scfg.Query.get_param 0 hostname)
-
-let () = Dream.log "hostname: %s" hostname
-
 let port =
   match Scfg.Query.get_dir "port" config with
   | None -> 8080
@@ -66,6 +58,14 @@ let port =
 
 let () = Dream.log "port: %d" port
 
+let hostname =
+  match Scfg.Query.get_dir "hostname" config with
+  | None -> Format.sprintf "localhost:%d" port
+  | Some hostname ->
+    Result.fold ~error:failwith ~ok:Fun.id (Scfg.Query.get_param 0 hostname)
+
+let () = Dream.log "hostname: %s" hostname
+
 let log =
   match Scfg.Query.get_dir "log" config with
   | None -> true

src/babillard.ml

@@ -271,11 +271,15 @@ let parse_comment comment =
   let pp_word fmt w =
     let trim_w = String.trim w in
     (* '>' is '>' after html_escape *)
-    if String.starts_with ~prefix:{|>>|} trim_w then
+    if String.length trim_w >= 8 then
       let sub_w = String.sub trim_w 8 (String.length trim_w - 8) in
-      if Option.is_some (Uuidm.of_string sub_w) then (
+      if
+        String.starts_with ~prefix:{|>>|} trim_w
+        && Option.is_some (Uuidm.of_string sub_w)
+      then begin
         citations := sub_w :: !citations;
-        Format.fprintf fmt {|<a href="#%s">%s</a>|} sub_w w )
+        Format.fprintf fmt {|<a href="#%s">%s</a>|} sub_w w
+      end
       else Format.pp_print_string fmt w
     else Format.pp_print_string fmt w
   in
@@ -290,10 +294,7 @@ let parse_comment comment =
       Format.fprintf fmt {|<span class="quote">%a</span>|}
         (Format.pp_print_list ~pp_sep:Format.pp_print_space pp_word)
         words
-    else
+    else Format.pp_print_list ~pp_sep:Format.pp_print_space pp_word fmt words
-      Format.fprintf fmt "%a"
-        (Format.pp_print_list ~pp_sep:Format.pp_print_space pp_word)
-        words
   in
 
   let comment = String.trim comment in
@@ -370,7 +371,7 @@ let build_reply ~comment ?image ~tags ?parent_id nick =
         let tag_list =
           List.map String.lowercase_ascii
           @@ List.sort_uniq String.compare
-          @@ List.filter (fun s -> not (String.equal "" s))
+          @@ List.filter (( <> ) "")
           @@ List.map String.trim
           @@ Str.split (Str.regexp ",+") tags
         in
@@ -467,7 +468,7 @@ let try_delete_post ~nick id =
   else Error "You can only delete your posts"
 
 let report ~nick ~reason id =
-  if not (post_exist id) then Error "This post doesn't exists"
+  if not (post_exist id) then Error "This post exists not"
   else if String.length reason > 2000 then Error "Your reason is too long.."
   else
     let reason = Dream.html_escape reason in

src/js/js_post_form.ml

@@ -10,7 +10,7 @@ let insert_quote post_id _event =
       let new_content =
         if String.ends_with ~suffix:"\n" content || String.length content = 0
         then Format.sprintf "%s>>%s " content post_id
-        else Format.sprintf "%s\n>>%s " content post_id
+        else Format.sprintf "%s@\n>>%s " content post_id
       in
       ignore @@ Jv.set textarea "value" (Jv.of_string new_content) )
     Jv.(find global "reply-comment")

src/permap.ml

@@ -291,8 +291,7 @@ let babillard_post request =
 let thread_feed_get request =
   let thread_id = Dream.param request "thread_id" in
   if Babillard.thread_exist thread_id then
-    let feed = Pp_babillard.feed thread_id in
+    match Pp_babillard.feed thread_id with
-    match feed with
     | Error e -> render_unsafe e request
     | Ok feed ->
       Dream.respond ~headers:[ ("Content-Type", "application/atom+xml") ] feed

src/pp_babillard.ml

@@ -301,7 +301,10 @@ let feed thread_id =
   let^ ids = Db.collect_list Q.get_thread_posts thread_id in
   let* posts = get_posts ids in
   let posts = List.sort (fun a b -> compare b.date a.date) posts in
-  let last_update = (List.nth posts 0).date in
+  let* last_update =
+    match posts with op :: _ -> Ok op.date | _ -> Error "empty thread"
+  in
+
   let entries fmt () =
     (Format.pp_print_list ~pp_sep:Format.pp_print_space pp_feed_entry) fmt posts
   in