tarrien/Mastodon
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Change the retry limit in error of web push notification (#11292)

Browse Source 
- 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.
This commit is contained in:
han@highemelry 2019-07-13 01:46:21 +09:00 committed by Thibaut Girka
parent 99c1f0811b
commit 0442fcdbc8
1 changed files with 1 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/workers/web/push_notification_worker.rb
View File

@ -3,7 +3,7 @@
class Web::PushNotificationWorker class Web::PushNotificationWorker
include Sidekiq::Worker include Sidekiq::Worker
sidekiq_options backtrace: true sidekiq_options backtrace: true, retry: 5
def perform(subscription_id, notification_id) def perform(subscription_id, notification_id)
subscription = ::Web::PushSubscription.find(subscription_id) subscription = ::Web::PushSubscription.find(subscription_id)