Retry ActivityPub inbox delivery on HTTP 401 and 408 errors (#10812)

HTTP 401 responses returned by Mastodon's inbox controller may
be temporary if, for instance, the requesting user's actor/key json
could not be retrieved in a timely fashion. This changes allow retries
instead of dropping the message entirely.

Also added HTTP 408 as that error is by nature temporary.
lolsob-rspec
ThibG 2019-05-23 15:00:30 +02:00 committed by Eugen Rochko
parent f9ca3d62b6
commit 4eaa4a649e
1 changed files with 1 additions and 1 deletions

View File

@ -51,7 +51,7 @@ class ActivityPub::DeliveryWorker
end end
def response_error_unsalvageable?(response) def response_error_unsalvageable?(response)
(400...500).cover?(response.code) && response.code != 429 (400...500).cover?(response.code) && ![401, 408, 429].include?(response.code)
end end
def failure_tracker def failure_tracker