Don't send client reports for profiles if profiling is disabled

CHANGELOG.md

@@ -28,6 +28,7 @@
 ### Internal
 
 - Archive [`sentry-raven`](https://github.com/getsentry/raven-ruby) ([#2708](https://github.com/getsentry/sentry-ruby/pull/2708))
+- Don't send `sample_rate` client reports for profiles if profiling is disabled ([#2728](https://github.com/getsentry/sentry-ruby/pull/2728))
 
 ## 5.28.0
 

sentry-ruby/lib/sentry/profiler.rb

@@ -83,7 +83,7 @@ def set_initial_sample_decision(transaction_sampled)
 
     def to_h
       unless @sampled
-        record_lost_event(:sample_rate)
+        record_lost_event(:sample_rate) if @profiling_enabled
         return {}
       end
 

sentry-ruby/lib/sentry/vernier/profiler.rb

@@ -92,7 +92,7 @@ def active_thread_id
 
       def to_h
         unless @sampled
-          record_lost_event(:sample_rate)
+          record_lost_event(:sample_rate) if @profiling_enabled
           return EMPTY_RESULT
         end
 

sentry-ruby/spec/sentry_spec.rb

@@ -390,8 +390,8 @@
 
       Sentry.get_current_client.flush
 
-      # 3 envelopes: log, transaction, and client_report about dropped profile
-      expect(sentry_envelopes.size).to be(3)
+      # 2 envelopes: log and transaction
+      expect(sentry_envelopes.size).to be(2)
 
       log_event = sentry_logs.first