Merge branch 'pdu-success' into 'master'

fix: also return successful PDUs in /send/:txnId See merge request famedly/conduit!90
2024-11-16 10:18:18 -07:00 · 2021-05-27 21:24:43 +00:00 · 2021-05-27 21:24:43 +00:00 · 0805f4ac2a
commit 0805f4ac2a
parent 25b1cd2683 7db59c550f
1 changed files with 14 additions and 13 deletions
--- a/src/server_server.rs
+++ b/src/server_server.rs
@ -628,19 +628,20 @@ pub async fn send_transaction_message_route<'a>(
        };

        let start_time = Instant::now();
-        if let Err(e) = handle_incoming_pdu(
-            &body.origin,
-            &event_id,
-            value,
-            true,
-            &db,
-            &pub_key_map,
-            &mut auth_cache,
-        )
-        .await
-        {
-            resolved_map.insert(event_id.clone(), Err(e));
-        }
+        resolved_map.insert(
+            event_id.clone(),
+            handle_incoming_pdu(
+                &body.origin,
+                &event_id,
+                value,
+                true,
+                &db,
+                &pub_key_map,
+                &mut auth_cache,
+            )
+            .await
+            .map(|_| ()),
+        );

        let elapsed = start_time.elapsed();
        if elapsed > Duration::from_secs(1) {