autonomic-cooperative/hometown
0
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Do not raise delivery failure on 4xx errors, increase stoplight threshold (#7541)

Browse Source 
* Do not raise delivery failure on 4xx errors, increase stoplight threshold

Stoplight failure threshold from 3 to 10
Status code 429 will raise a failure/get retried

* Oops
This commit is contained in:
Eugen Rochko 2018-05-19 00:23:19 +02:00 committed by GitHub
parent 9422b3e0d8
commit 97f02f2c08
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
1 changed files with 11 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
app/workers/activitypub/delivery_worker.rb
View File

@ -3,6 +3,9 @@
class ActivityPub::DeliveryWorker class ActivityPub::DeliveryWorker
include Sidekiq::Worker include Sidekiq::Worker
STOPLIGHT_FAILURE_THRESHOLD = 10
STOPLIGHT_COOLDOWN = 60
sidekiq_options queue: 'push', retry: 16, dead: false sidekiq_options queue: 'push', retry: 16, dead: false
HEADERS = { 'Content-Type' => 'application/activity+json' }.freeze HEADERS = { 'Content-Type' => 'application/activity+json' }.freeze
@ -31,17 +34,23 @@ class ActivityPub::DeliveryWorker
def perform_request def perform_request
light = Stoplight(@inbox_url) do light = Stoplight(@inbox_url) do
build_request.perform do |response| build_request.perform do |response|
raise Mastodon::UnexpectedResponseError, response unless response_successful?(response) raise Mastodon::UnexpectedResponseError, response unless response_successful?(response) || response_error_unsalvageable?(response)
end end
end end
light.run light.with_threshold(STOPLIGHT_FAILURE_THRESHOLD)
.with_cool_off_time(STOPLIGHT_COOLDOWN)
.run
end end
def response_successful?(response) def response_successful?(response)
response.code > 199 && response.code < 300 response.code > 199 && response.code < 300
end end
def response_error_unsalvageable?(response)
response.code > 399 && response.code < 500 && response.code != 429
end
def failure_tracker def failure_tracker
@failure_tracker ||= DeliveryFailureTracker.new(@inbox_url) @failure_tracker ||= DeliveryFailureTracker.new(@inbox_url)
end end