Fix get_pdu asking every destination even after it finds an event

As discovered by @richvdh, #13320 (comment)
matrix-org · Jul 20, 2022 · 58317e6 · 58317e6
1 parent 0f971ca
commit 58317e6
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/synapse/federation/federation_client.py b/synapse/federation/federation_client.py
@@ -403,9 +403,9 @@ async def get_pdu(
                         # Prime the cache
                         self._get_pdu_cache[event.event_id] = event
 
-                        # FIXME: We should add a `break` here to avoid calling every
-                        # destination after we already found a PDU (will follow-up
-                        # in a separate PR)
+                        # Now that we have an event, we can break out of this
+                        # loop and stop asking other destinations.
+                        break
 
                 except SynapseError as e:
                     logger.info(