summary refs log tree commit diff
diff options
context:
space:
mode:
authorRichard van der Hoff <richard@matrix.org>2018-04-20 11:41:03 +0100
committerRichard van der Hoff <richard@matrix.org>2018-04-20 11:41:03 +0100
commit3de7d9fe99445b1229411e7a4706c46ea94ded00 (patch)
tree18574b955087cbb49de9c0147dedf15fe7632dd6
parentAdd some comments (diff)
downloadsynapse-3de7d9fe99445b1229411e7a4706c46ea94ded00.tar.xz
accept stupid events over backfill
-rw-r--r--synapse/handlers/federation.py13
1 files changed, 9 insertions, 4 deletions
diff --git a/synapse/handlers/federation.py b/synapse/handlers/federation.py
index 1296e22af6..2523ae4a09 100644
--- a/synapse/handlers/federation.py
+++ b/synapse/handlers/federation.py
@@ -563,10 +563,15 @@ class FederationHandler(BaseHandler):
             extremities=extremities,
         )
 
-        # do some sanity-checking of the received events, before we go and
-        # do state resolution across 1000 events.
-        for ev in events:
-            self._sanity_check_event(ev)
+        # ideally we'd sanity check the events here for excess prev_events etc,
+        # but it's hard to reject events at this point without completely
+        # breaking backfill in the same way that it is currently broken by
+        # events whose signature we cannot verify (#3121).
+        #
+        # So for now we accept the events anyway. #3124 tracks this.
+        #
+        # for ev in events:
+        #     self._sanity_check_event(ev)
 
         # Don't bother processing events we already have.
         seen_events = yield self.store.have_events_in_timeline(