diff options
| author | Eric Eastwood <erice@element.io> | 2022-10-26 16:10:55 -0500 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2022-10-26 16:10:55 -0500 |
| commit | 40fa8294e3096132819287dd0c6d6bd71a408902 (patch) | |
| tree | ff78c93920f07ae3e5d3b71e71cd4451c8023f99 /synapse/util/retryutils.py | |
| parent | Use poetry 1.2 for complement in latest deps (#14305) (diff) | |
| download | synapse-40fa8294e3096132819287dd0c6d6bd71a408902.tar.xz | |
Refactor MSC3030 `/timestamp_to_event` to move away from our snowflake pull from `destination` pattern (#14096)
1. `federation_client.timestamp_to_event(...)` now handles all `destination` looping and uses our generic `_try_destination_list(...)` helper. 2. Consistently handling `NotRetryingDestination` and `FederationDeniedError` across `get_pdu` , backfill, and the generic `_try_destination_list` which is used for many places we use this pattern. 3. `get_pdu(...)` now returns `PulledPduInfo` so we know which `destination` we ended up pulling the PDU from
Diffstat (limited to 'synapse/util/retryutils.py')
| -rw-r--r-- | synapse/util/retryutils.py | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/synapse/util/retryutils.py b/synapse/util/retryutils.py index d0a69ff843..dcc037b982 100644 --- a/synapse/util/retryutils.py +++ b/synapse/util/retryutils.py @@ -51,7 +51,7 @@ class NotRetryingDestination(Exception): destination: the domain in question """ - msg = "Not retrying server %s." % (destination,) + msg = f"Not retrying server {destination} because we tried it recently retry_last_ts={retry_last_ts} and we won't check for another retry_interval={retry_interval}ms." super().__init__(msg) self.retry_last_ts = retry_last_ts |