Don't log a warning on NotRetryingDestination

Otherwise we're going to be logging `Failed to handle device list update
for @user:example.com` every 30s for every remote we're not retrying
because of backoff.
pull/7453/head
Brendan Abolivier 2020-05-07 17:54:39 +02:00
parent c62c77d704
commit da73470ffb
No known key found for this signature in database
GPG Key ID: 1E015C145F1916CD
1 changed files with 6 additions and 1 deletions

View File

@ -732,7 +732,12 @@ class DeviceListUpdater(object):
origin = get_domain_from_id(user_id)
try:
result = yield self.federation.query_user_devices(origin, user_id)
except (NotRetryingDestination, RequestSendFailed, HttpResponseException):
except NotRetryingDestination:
# Mark the remote user's device list as stale so we know we need to retry it
# later.
yield self.store.mark_remote_user_device_cache_as_stale(user_id)
return
except (RequestSendFailed, HttpResponseException):
logger.warning("Failed to handle device list update for %s", user_id)
# Mark the remote user's device list as stale so we know we need to retry it
# later.