Review fixes

getsentry · Aug 31, 2023 · 0cf4996 · 0cf4996
1 parent c8b8c4c
commit 0cf4996
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 6 deletions.
diff --git a/sentry-ruby/lib/sentry/event.rb b/sentry-ruby/lib/sentry/event.rb
@@ -59,6 +59,7 @@ def initialize(configuration:, integration_meta: nil, message: nil)
       @tags          = {}
 
       @fingerprint = []
+      @dynamic_sampling_context = nil
 
       # configuration data that's directly used by events
       @server_name = configuration.server_name

diff --git a/sentry-ruby/lib/sentry/hub.rb b/sentry-ruby/lib/sentry/hub.rb
@@ -233,14 +233,14 @@ def get_traceparent
       return nil unless current_scope
 
       current_scope.get_span&.to_sentry_trace ||
-        current_scope.propagation_context&.get_traceparent
+        current_scope.propagation_context.get_traceparent
     end
 
     def get_baggage
       return nil unless current_scope
 
       current_scope.get_span&.to_baggage ||
-        current_scope.propagation_context&.get_baggage&.serialize
+        current_scope.propagation_context.get_baggage&.serialize
     end
 
     def get_trace_propagation_headers

diff --git a/sentry-ruby/lib/sentry/propagation_context.rb b/sentry-ruby/lib/sentry/propagation_context.rb
@@ -65,12 +65,10 @@ def populate_head_baggage
         "sample_rate" => configuration.traces_sample_rate&.to_s,
         "environment" => configuration.environment,
         "release" => configuration.release,
-        "public_key" => configuration.dsn&.public_key
+        "public_key" => configuration.dsn&.public_key,
+        "user_segment" => @scope.user && @scope.user["segment"]
       }
 
-      user = @scope&.user
-      items["user_segment"] = user["segment"] if user && user["segment"]
-
       items.compact!
       @baggage = Baggage.new(items, mutable: false)
     end