Merge pull request #2261 from fukayatsu/fix-sidekiq-deprecation-warning

Fix deprecation warning from sidekiq error handler
newrelic · Oct 17, 2023 · 0f6cb1a · 0f6cb1a
2 parents 67565e8 + b2e1325
commit 0f6cb1a
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/lib/new_relic/agent/instrumentation/sidekiq.rb b/lib/new_relic/agent/instrumentation/sidekiq.rb
@@ -33,7 +33,9 @@
       end
 
       if config.respond_to?(:error_handlers)
-        config.error_handlers << proc do |error, *_|
+        # Sidekiq 3.0.0 - 7.1.4 expect error_handlers to have 2 arguments
+        # Sidekiq 7.1.5+ expect error_handlers to have 3 arguments
+        config.error_handlers << proc do |error, _ctx, *_|
           NewRelic::Agent.notice_error(error)
         end
       end