From eca0bbb35a9c062c21392c60455f624374c5cd17 Mon Sep 17 00:00:00 2001
From: Devin Ragotzy <devin.ragotzy@gmail.com>
Date: Sun, 8 Nov 2020 13:49:02 -0500
Subject: [PATCH] Fix federated join miss hashing the join event created

---
 src/server_server.rs | 13 +++++++++++--
 1 file changed, 11 insertions(+), 2 deletions(-)

diff --git a/src/server_server.rs b/src/server_server.rs
index de24c6d4..ccdeb749 100644
--- a/src/server_server.rs
+++ b/src/server_server.rs
@@ -412,10 +412,19 @@ pub async fn send_transaction_message_route<'a>(
                 "m.receipt" => {}
                 _ => {}
             },
-            Err(_err) => continue,
+            Err(_err) => {
+                log::error!("{}", _err);
+                continue;
+            }
         }
     }
-
+    // TODO: For RoomVersion6 we must check that Raw<..> is canonical do we?
+    // SPEC:
+    // Servers MUST strictly enforce the JSON format specified in the appendices.
+    // This translates to a 400 M_BAD_JSON error on most endpoints, or discarding of
+    // events over federation. For example, the Federation API's /send endpoint would
+    // discard the event whereas the Client Server API's /send/{eventType} endpoint
+    // would return a M_BAD_JSON error.
     let mut resolved_map = BTreeMap::new();
     for pdu in &body.pdus {
         println!("LOOP");