Merge pull request #20 from IamDavidovich/tracing

Update ot-tracer-sampled to use 'true' and 'false' instead of '0' and '1'
bigcommerce · May 31, 2023 · b400f02 · b400f02
2 parents a2bfee3 + 0fe8cee
commit b400f02
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,8 @@ Changelog for the gruf-lightstep gem.
 
 ### Pending Release
 
+- Use default value of `'true'` for the `ot-tracer-sampled` value in the client interceptor instead of `'1'`
+
 ### 1.7.0
 
 - Add support for Ruby 3.2

diff --git a/lib/gruf/lightstep/client_interceptor.rb b/lib/gruf/lightstep/client_interceptor.rb
@@ -28,7 +28,7 @@ def call(request_context:)
           logger.debug "[gruf-lightstep] Injecting current active span #{span_data[:span_guid]} into outbound request context for #{request_context.method_name}"
           request_context.metadata['ot-tracer-spanid'] = span_data[:span_guid].to_s
           request_context.metadata['ot-tracer-traceid'] = span_data[:trace_guid].to_s
-          request_context.metadata['ot-tracer-sampled'] = '1'
+          request_context.metadata['ot-tracer-sampled'] = 'true'
         end
 
         yield