Skip to content

Commit c7862ea

Browse files
sb8244kattrali
authored andcommitted
Move the Sidekiq reporter to after the logging middleware (#326)
The Server::Logging middleware is the outermost middleware that Sidekiq provides by default. This is a very safe place to put error reporting middleware, because any custom code implemented by users of Bugsnag will probably not cause erroneous reports. Closes #326
1 parent f9f59b8 commit c7862ea

1 file changed

Lines changed: 1 addition & 5 deletions

File tree

lib/bugsnag/sidekiq.rb

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -27,10 +27,6 @@ def call(worker, msg, queue)
2727

2828
::Sidekiq.configure_server do |config|
2929
config.server_middleware do |chain|
30-
if Gem::Version.new(Sidekiq::VERSION) >= Gem::Version.new('3.3.0')
31-
chain.prepend ::Bugsnag::Sidekiq
32-
else
33-
chain.add ::Bugsnag::Sidekiq
34-
end
30+
chain.insert_after ::Sidekiq::Middleware::Server::Logging, ::Bugsnag::Sidekiq
3531
end
3632
end

0 commit comments

Comments
 (0)