From 275c985e482f50cdf3cfd508df4862879247149d Mon Sep 17 00:00:00 2001
From: Benedetta <bparodi@lezzo.org>
Date: Wed, 1 May 2024 12:03:26 +0200
Subject: [PATCH] burrodiarachidi

---
 pam/bin/httpclient.ml | 12 +++++-------
 pam/bin/main.ml       | 39 +++++++++++++++++++++++----------------
 pam/bin/mq.ml         | 15 ++++++---------
 3 files changed, 34 insertions(+), 32 deletions(-)

diff --git a/pam/bin/httpclient.ml b/pam/bin/httpclient.ml
index 168f1e7..684889d 100644
--- a/pam/bin/httpclient.ml
+++ b/pam/bin/httpclient.ml
@@ -67,7 +67,7 @@ let extract_pagination resp =
   .@$("headers")
   .&("get") [|key|] (* not really a pydict, so need to use `get` *)
   |> Py.Object.to_string
-  |> int_of_string
+  |> int_of_string_opt
 
 let make_get_request {requests; repos} =
   let get = Py.Module.get_function_with_keywords requests "get" in
@@ -82,16 +82,14 @@ let make_get_request {requests; repos} =
         resp.@$("text")
         |> Py.String.to_string
       in
-      let _ = print_endline "linea 62" in
       let target_items_total = extract_pagination resp in
-      let _ = print_endline (url |> ForgejoUrl.to_string) in
-      let _ = print_endline "linea 64" in
       let matrix_room = Datatypes.MatrixRoom.make m_room_string in
       let issues = issues_of_json matrix_room jsontext in
-      match issues with
-      | Error err ->
+      match target_items_total, issues with
+      | None, _ -> Error "Can't extract pagination"
+      | _, Error err ->
         Error err
-      | Ok target_issues ->
+      | Some target_items_total, Ok target_issues ->
         let n_received = List.length target_issues in
         let accum' = target_issues@accum in
         let urls = 
diff --git a/pam/bin/main.ml b/pam/bin/main.ml
index a2f0137..98b0acc 100644
--- a/pam/bin/main.ml
+++ b/pam/bin/main.ml
@@ -1,5 +1,5 @@
 open Pam.Datatypes
-open Pam.Issue_parser
+open Pam
 open Batteries
 open Utils
 
@@ -13,34 +13,41 @@ let _HTTP_CLIENT = "http_client"
 let http_client (repos: Config.repo_data list) =
   let http_actor = Httpclient.init repos in
 
-  let now = Ptime_clock.now () in
-
-  let _ = print_endline "Initialized http client" in
+  let _ = Pamlog.info "Initialized http client" in
 
+  let check_alert_time now issues =
+    let rec aux acc = function
+      | [] -> Ok acc
+      | issue :: rest ->
+        match Issuelib.to_datetime issue with
+        | Error e -> Error e
+        | Ok None -> aux acc rest
+        | Ok (Some alert_times) ->
+          let acc' =
+            if Issuelib.should_alert now alert_times then
+              issue::acc
+            else
+              acc
+          in aux acc' rest
+    in aux [] issues
   in
   let rec loop () =
     let _ =
       match Riot.receive_any () with
       | ListIssues ->
+        let now = Ptime_clock.now () in
         let issues = Httpclient.make_get_request http_actor
         in
-        (* Result.bind issues pair_to_alert_time *) TODO: usa Pam.Issue_parser.should_alert 
-        |> Result.map (List.filter_map filter_issue_by_time)
+        Result.bind issues (check_alert_time now)
         |> Result.map_error internal_failure
         |> Result.fold ~ok:forgejo_issues ~error:List.singleton
         |> List.map (Riot.send_by_name ~name:_MQ_CLIENT)
-
-
       | m -> unhandled m
     in
     loop ()
   in loop ()
 
 let mq_client (mq_url, mq_user, mq_password) =
-  let pprint rem =
-    [%string "%{rem.title}|%{rem.matrix_target}"]
-  in
-
   let call_consumer { Amqp_client_lwt.Message.message = (_content, body); _ } =
     Pamlog.error [%string "Received msg from rabbitmq: %{body}. PAM will ignore."]
   in
@@ -56,9 +63,8 @@ let mq_client (mq_url, mq_user, mq_password) =
           let _ = Pamlog.error [%string "Got error from Forgejo: %{err}"] in
           Mq.mq_publish mq err 
         | Reminder reminder ->
-          let _ = [%string "Got reminders: %{Batteries.dump reminder}"] |> print_endline in
-          let rems = pprint reminder in
-          Mq.mq_publish mq rems
+          let formatted = Issuelib.issue_data_to_json reminder in
+          Mq.mq_publish mq formatted
         | m -> unhandled m
       with | Riot.Receive_timeout -> Lwt.return_unit
     in
@@ -73,7 +79,7 @@ let main (config: Config.config) =
   let _ = Riot.register _HTTP_CLIENT http_client_pid in
   let _ = Riot.register _MQ_CLIENT mq_client_pid
   in
-  let timeout = 6.0 in
+  let timeout = 3.0 in
   let rec loop_ () =
     let _ = send http_client_pid ListIssues in
     sleep timeout |> loop_
@@ -83,5 +89,6 @@ let main (config: Config.config) =
 
 
 let () =
+  if true then failwith "Testa che manda gli errori su matrix" else 2 |> ignore in
   let config = Config.configuration () |> Result.fold ~error:exit2 ~ok:identity in
   Riot.run (fun () -> main config)
diff --git a/pam/bin/mq.ml b/pam/bin/mq.ml
index c4791ae..bc9971c 100644
--- a/pam/bin/mq.ml
+++ b/pam/bin/mq.ml
@@ -25,13 +25,10 @@ let shutdown {queue=_queue; channel; connection} =
   let%lwt _ = Lwt.catch (fun () -> Amqp.Connection.close connection) ignore_exn in
   Lwt.return_unit
 
-let mq_publish {queue=_; channel=_; connection=_} msg =
-  let _ = Pamlog.info [%string "GOT MSG FOR MQ:: %{msg}"]
-  (* let%lwt _ = *)
-    (* Message.make msg *)
-    (* |> Exchange.publish channel Exchange.default  ~routing_key:"lanonna" *)
-  in Lwt.return_unit
 
-let rec mq_publish_all mq = function
-  | [] -> Lwt.return_unit
-  | r::rest -> let%lwt _ = mq_publish mq r in mq_publish_all mq rest
+let mq_publish {queue=_; channel=channel; connection=_} msg =
+  let _ = Pamlog.info [%string "GOT MSG FOR MQ:: %{msg}"] in
+  let%lwt _ =
+    Message.make msg
+    |> Exchange.publish channel Exchange.default  ~routing_key:"lanonna"
+  in Lwt.return_unit