Adopt Rails 7.1's new BroadcastLogger

CHANGELOG.md

@@ -3,6 +3,7 @@
 ### Features
 
 - Record client reports for profiles [#2107](https://github.com/getsentry/sentry-ruby/pull/2107)
+- Adopt Rails 7.1's new BroadcastLogger [#2120](https://github.com/getsentry/sentry-ruby/pull/2120)
 
 ### Bug Fixes
 

sentry-rails/.gitignore

@@ -5,7 +5,7 @@
 /doc/
 /pkg/
 /spec/reports/
-/spec/dummy/test_rails_app/db
+/spec/dummy/test_rails_app/db*
 /tmp/
 
 # rspec failure tracking

sentry-rails/lib/sentry/rails/configuration.rb

@@ -12,7 +12,12 @@
       @excluded_exceptions = @excluded_exceptions.concat(Sentry::Rails::IGNORE_DEFAULT)
 
       if ::Rails.logger
-        @logger = ::Rails.logger.dup
+        if ::Rails.logger.respond_to?(:broadcasts)
+          dupped_broadcasts = ::Rails.logger.broadcasts.map(&:dup)
+          @logger = ::ActiveSupport::BroadcastLogger.new(*dupped_broadcasts)
+        else
+          @logger = ::Rails.logger.dup
+        end
       else
         @logger.warn(Sentry::LOGGER_PROGNAME) do
           <<~MSG

sentry-rails/spec/sentry/rails_spec.rb

@@ -14,6 +14,12 @@
       make_basic_app
     end
 
+    after do
+      # We need to cleanup Rails.logger because after https://github.com/rails/rails/pull/49417
+      # Rails.logger could get recreated with the previous logger value and become deeply nested broadcast logger
+      Rails.logger = nil
+    end
+
     it "has version set" do
       expect(described_class::VERSION).to be_a(String)
     end
@@ -36,11 +42,24 @@
 
     describe "logger detection" do
       it "sets a duplicated Rails logger as the SDK's logger" do
-        expect(Sentry.configuration.logger).to be_a(ActiveSupport::Logger)
-        Sentry.configuration.logger.level = ::Logger::WARN
-        # Configuring the SDK's logger should not affect the Rails logger
-        expect(Rails.logger.level).to eq(::Logger::DEBUG)
-        expect(Sentry.configuration.logger.level).to eq(::Logger::WARN)
+        if Gem::Version.new(Rails.version) > Gem::Version.new("7.1.0.beta")
+          expect(Sentry.configuration.logger).to be_a(ActiveSupport::BroadcastLogger)
+
+          Sentry.configuration.logger.level = ::Logger::WARN
+
+          # Configuring the SDK's logger should not affect the Rails logger
+          expect(Rails.logger.broadcasts.first).to be_a(ActiveSupport::Logger)
+          expect(Rails.logger.broadcasts.first.level).to eq(::Logger::DEBUG)
+          expect(Sentry.configuration.logger.level).to eq(::Logger::WARN)
+        else
+          expect(Sentry.configuration.logger).to be_a(ActiveSupport::Logger)
+
+          Sentry.configuration.logger.level = ::Logger::WARN
+
+          # Configuring the SDK's logger should not affect the Rails logger
+          expect(Rails.logger.level).to eq(::Logger::DEBUG)
+          expect(Sentry.configuration.logger.level).to eq(::Logger::WARN)
+        end
       end
 
       it "respects the logger set by user" do