Avoid invoking ActiveSupport::BroadcastLogger if not defined (#2169)

* Avoid instantiating BroadcastLogger if not defined * Update CHANGELOG.md * Update CHANGELOG.md --------- Co-authored-by: Stan Lo <[email protected]>
getsentry · Nov 21, 2023 · 4edfdf1 · 4edfdf1
1 parent c7c8e5b
commit 4edfdf1
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -15,6 +15,7 @@
 ### Bug Fixes
 
 - Fixed a deprecation in `sidekiq-ruby` error handler [#2160](https://github.com/getsentry/sentry-ruby/pull/2160)
+- Avoid invoking ActiveSupport::BroadcastLogger if not defined [#2169](https://github.com/getsentry/sentry-ruby/pull/2169)
 
 ## 5.13.0
 

diff --git a/sentry-rails/lib/sentry/rails/configuration.rb b/sentry-rails/lib/sentry/rails/configuration.rb
@@ -12,7 +12,7 @@ class Configuration
       @excluded_exceptions = @excluded_exceptions.concat(Sentry::Rails::IGNORE_DEFAULT)
 
       if ::Rails.logger
-        if ::Rails.logger.respond_to?(:broadcasts)
+        if defined?(::ActiveSupport::BroadcastLogger) && ::Rails.logger.is_a?(::ActiveSupport::BroadcastLogger)
           dupped_broadcasts = ::Rails.logger.broadcasts.map(&:dup)
           @logger = ::ActiveSupport::BroadcastLogger.new(*dupped_broadcasts)
         else