diff --git a/sentry-rails/lib/sentry/rails/configuration.rb b/sentry-rails/lib/sentry/rails/configuration.rb index c76c3ed75..d5589ab0f 100644 --- a/sentry-rails/lib/sentry/rails/configuration.rb +++ b/sentry-rails/lib/sentry/rails/configuration.rb @@ -224,7 +224,7 @@ class StructuredLoggingConfiguration }.freeze def initialize - @enabled = true + @enabled = false @subscribers = DEFAULT_SUBSCRIBERS.dup end diff --git a/sentry-rails/sentry-rails.gemspec b/sentry-rails/sentry-rails.gemspec index ecebd2b31..17734395a 100644 --- a/sentry-rails/sentry-rails.gemspec +++ b/sentry-rails/sentry-rails.gemspec @@ -32,4 +32,9 @@ Gem::Specification.new do |spec| spec.add_dependency "railties", ">= 5.2.0" spec.add_dependency "sentry-ruby", "~> 7.0.0" + + spec.post_install_message = <<~MSG + Starting with sentry-rails 7.1.0, Rails structured logging is disabled by default. + Set `config.rails.structured_logging.enabled = true` to enable it. + MSG end diff --git a/sentry-rails/spec/sentry/rails/configuration_spec.rb b/sentry-rails/spec/sentry/rails/configuration_spec.rb index afba50113..f45e61a0d 100644 --- a/sentry-rails/spec/sentry/rails/configuration_spec.rb +++ b/sentry-rails/spec/sentry/rails/configuration_spec.rb @@ -99,10 +99,10 @@ class MySubscriber; end expect(config.structured_logging.subscribers).to be_a(Hash) end - it "enables structured logging by default" do + it "disables structured logging by default" do make_basic_app - expect(config.structured_logging.enabled?).to be(true) + expect(config.structured_logging.enabled?).to be(false) end it "respects explicit disable" do