Rails.logger methods return breadcrumbs instead of true when breadcrumbs logging is enabled

sentry-ruby/CHANGELOG.md

@@ -9,6 +9,7 @@
   - Fixes [#1307](https://github.com/getsentry/sentry-ruby/issues/1307)
 - Refactor tracing implementation [#1309](https://github.com/getsentry/sentry-ruby/pull/1309)
 - Allow configuring BreadcrumbBuffer's size limit [#1310](https://github.com/getsentry/sentry-ruby/pull/1310)
+- Return nil from logger methods instead of breadcrumb buffer [#1299](https://github.com/getsentry/sentry-ruby/pull/1299)
 
 ## 4.2.2
 

sentry-ruby/lib/sentry/breadcrumb/sentry_logger.rb

@@ -14,6 +14,7 @@ module SentryLogger
       def add(*args, &block)
         super
         add_breadcrumb(*args, &block)
+        nil
       end
 
       def add_breadcrumb(severity, message = nil, progname = nil)

sentry-ruby/spec/sentry/breadcrumb/sentry_logger_spec.rb

@@ -20,6 +20,10 @@
     expect(breadcrumb.message).to eq("foo")
   end
 
+  it "does not affect the return of the logger call" do
+    expect(logger.info("foo")).to be_nil
+  end
+
   it "ignores traces with #{Sentry::LOGGER_PROGNAME}" do
     logger.info(Sentry::LOGGER_PROGNAME) { "foo" }