Browse Source

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.
master
ThibG 5 years ago
committed by Eugen Rochko
parent
commit
9efcca3c54
1 changed files with 1 additions and 1 deletions
  1. +1
    -1
      app/workers/activitypub/delivery_worker.rb

+ 1
- 1
app/workers/activitypub/delivery_worker.rb 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

Loading…
Cancel
Save