Browse Source

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

- 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.
master
han@highemelry 5 years ago
committed by Eugen Rochko
parent
commit
402302776c
1 changed files with 1 additions and 1 deletions
  1. +1
    -1
      app/workers/web/push_notification_worker.rb

+ 1
- 1
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)

Loading…
Cancel
Save