Skip creating LogEventBuffer if logging is not enabled

Author: solnic

sentry-ruby/lib/sentry/client.rb

@@ -42,7 +42,9 @@ def initialize(configuration)
 
       @spotlight_transport = SpotlightTransport.new(configuration) if configuration.spotlight
 
-      @log_event_buffer = LogEventBuffer.new(configuration, self).start
+      if configuration.enable_logs
+        @log_event_buffer = LogEventBuffer.new(configuration, self).start
+      end
     end
 
     # Applies the given scope's data to the event and sends it to Sentry.
@@ -114,7 +116,7 @@ def capture_envelope(envelope)
     def flush
       transport.flush if configuration.sending_to_dsn_allowed?
       spotlight_transport.flush if spotlight_transport
-      @log_event_buffer.flush
+      @log_event_buffer&.flush
     end
 
     # Initializes an Event object with the given exception. Returns `nil` if the exception's class is excluded from reporting.

sentry-ruby/spec/sentry/session_flusher_spec.rb

@@ -103,15 +103,15 @@
     it "spawns new thread" do
       expect do
         subject.add_session(session)
-      end.to change { Thread.list.count }.by(2)
+      end.to change { Thread.list.count }.by(1)
 
       expect(subject.instance_variable_get(:@thread)).to be_a(Thread)
     end
 
     it "spawns only one thread" do
       expect do
         subject.add_session(session)
-      end.to change { Thread.list.count }.by(2)
+      end.to change { Thread.list.count }.by(1)
 
       thread = subject.instance_variable_get(:@thread)
       expect(thread).to receive(:alive?).and_return(true)

sentry-ruby/spec/sentry_spec.rb

@@ -354,6 +354,7 @@
   describe ".capture_log" do
     before do
       perform_basic_setup do |config|
+        config.enable_logs = true
         config.traces_sample_rate = 1.0
         config.max_log_events = 1
       end