Skip to content

Commit

Permalink
fix: sidekiq leader check only for version > 6.5 (#594)
Browse files Browse the repository at this point in the history
  • Loading branch information
roelbondoc authored Jul 23, 2024
1 parent 368fb16 commit 3bc005b
Showing 1 changed file with 17 additions and 11 deletions.
28 changes: 17 additions & 11 deletions lib/honeybadger/plugins/sidekiq.rb
Original file line number Diff line number Diff line change
Expand Up @@ -127,25 +127,31 @@ def call(worker, msg, queue, _redis)
if config.load_plugin_insights?(:sidekiq)
require "sidekiq"
require "sidekiq/api"
require "sidekiq/component"

class SidekiqClusterCollectionChecker
include ::Sidekiq::Component
def initialize(config)
@config = config
end
if Gem::Version.new(Sidekiq::VERSION) >= Gem::Version.new('6.5')
require "sidekiq/component"

class SidekiqClusterCollectionChecker
include ::Sidekiq::Component
def initialize(config)
@config = config
end

def collect?
return true unless defined?(::Sidekiq::Enterprise)
leader?
def collect?
return true unless defined?(::Sidekiq::Enterprise)
leader?
end
end
end

::Sidekiq.configure_server do |config|
config.server_middleware { |chain| chain.add(ServerMiddlewareInstrumentation) }
config.client_middleware { |chain| chain.add(ClientMiddlewareInstrumentation) }
config.on(:startup) do
leader_checker = SidekiqClusterCollectionChecker.new(config)

if defined?(SidekiqClusterCollectionChecker)
config.on(:startup) do
leader_checker = SidekiqClusterCollectionChecker.new(config)
end
end
end

Expand Down

0 comments on commit 3bc005b

Please sign in to comment.