summary refs log tree commit diff
diff options
context:
space:
mode:
authorRichard van der Hoff <1389908+richvdh@users.noreply.github.com>2021-10-07 12:20:03 +0100
committerGitHub <noreply@github.com>2021-10-07 12:20:03 +0100
commit86af6b2f0ef92a317900fd4a4f6d3436ff8a011c (patch)
treef77e2b7e39e07c23d12e697f6b5fb1ee31bc599d
parentCatch AttributeErrors when calling registerProducer (#10995) (diff)
downloadsynapse-86af6b2f0ef92a317900fd4a4f6d3436ff8a011c.tar.xz
Add a comment in _process_received_pdu (#11011)
-rw-r--r--changelog.d/11011.misc1
-rw-r--r--synapse/handlers/federation_event.py3
2 files changed, 4 insertions, 0 deletions
diff --git a/changelog.d/11011.misc b/changelog.d/11011.misc
new file mode 100644
index 0000000000..9a765435db
--- /dev/null
+++ b/changelog.d/11011.misc
@@ -0,0 +1 @@
+Clean up some of the federation event authentication code for clarity.
diff --git a/synapse/handlers/federation_event.py b/synapse/handlers/federation_event.py
index 243be46267..0645ce9392 100644
--- a/synapse/handlers/federation_event.py
+++ b/synapse/handlers/federation_event.py
@@ -894,6 +894,9 @@ class FederationEventHandler:
                 backfilled=backfilled,
             )
         except AuthError as e:
+            # FIXME richvdh 2021/10/07 I don't think this is reachable. Let's log it
+            #   for now
+            logger.exception("Unexpected AuthError from _check_event_auth")
             raise FederationError("ERROR", e.code, e.msg, affected=event.event_id)
 
         await self._run_push_actions_and_persist_event(event, context, backfilled)