mirror of
https://github.com/element-hq/synapse
synced 2024-08-20 17:50:21 +00:00
Remove explicit < 400 check as apparently this is confusing
This commit is contained in:
parent
4c0ec15bdc
commit
fe08db2713
1 changed files with 1 additions and 3 deletions
|
@ -129,9 +129,7 @@ class RetryDestinationLimiter(object):
|
||||||
# APIs may expect to never received e.g. a 404. It's important to
|
# APIs may expect to never received e.g. a 404. It's important to
|
||||||
# handle 404 as some remote servers will return a 404 when the HS
|
# handle 404 as some remote servers will return a 404 when the HS
|
||||||
# has been decommissioned.
|
# has been decommissioned.
|
||||||
if exc_val.code < 400:
|
if exc_val.code == 404 and self.backoff_on_404:
|
||||||
valid_err_code = True
|
|
||||||
elif exc_val.code == 404 and self.backoff_on_404:
|
|
||||||
valid_err_code = False
|
valid_err_code = False
|
||||||
elif exc_val.code == 429:
|
elif exc_val.code == 429:
|
||||||
# 429 is us being aggresively rate limited, so lets rate limit
|
# 429 is us being aggresively rate limited, so lets rate limit
|
||||||
|
|
Loading…
Reference in a new issue