Skip to content

Commit

Permalink
Change the retry limit in error of web push notification (mastodon#11292
Browse files Browse the repository at this point in the history
)

- Change the maximum count of retry for web push notification (Default -> 5).
   - In case of high load of subscribe server, the retries will be repeated many times.
   - Because the retries occupy the default queue, maximum retry count should be reduced.
  • Loading branch information
highemerly authored and hiyuki2578 committed Oct 2, 2019
1 parent cd56499 commit fb80ad5
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion app/workers/web/push_notification_worker.rb
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@
class Web::PushNotificationWorker
include Sidekiq::Worker

sidekiq_options backtrace: true
sidekiq_options backtrace: true, retry: 5

def perform(subscription_id, notification_id)
subscription = ::Web::PushSubscription.find(subscription_id)
Expand Down

0 comments on commit fb80ad5

Please sign in to comment.