opentrace 0.14.1 → 0.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1e351721dd14bd088cfae4e5ab01a8ef16686fff3f2d1cb9aed984fad47c48c6
-  data.tar.gz: 8a098bfb548ea1a867aa1a57a50ebec3214a466e29d88f5a515a5409839650f5
+  metadata.gz: ad0233533a9c236a70cf379346c1f64f7ffecc58cf91ce413e55665e8b4f5675
+  data.tar.gz: 92d7052f7901fad33897b767441ea01f8ed299bc47d90d43153110a2fdd22c1e
 SHA512:
-  metadata.gz: b383410de64041a2b247d8b44d511bf5bde57f4e7c62fd7ceabdd0dd875a487d0ec3c21eb46d524d3ed55cb4faaef3e8acc4046bfec724040990f918d5f02fd2
-  data.tar.gz: e822d353c5b393c0a408fe63926df46631ef0b90b726fa48db9d6feaf033185778af3436988994eb3b555f32c4009d4603e4e5f3f96b9a092c7d4f3758275187
+  metadata.gz: 6f9b631bf0faab3e6671a41eccc04039339f2f95cfa90a6c380517572256c08c3115bb4c30ddae230933bac18f8c9858195a55beacda27ff2e44c5efb02ef4f7
+  data.tar.gz: 3671e28a4393225a5e891c09a38a734b974c05b70fd393ff892b06bb816fd5954880f7eab5da257248fe025df948f96b0fb94b2f526338d060a928514d5a4917

data/lib/opentrace/circuit_breaker.rb

@@ -14,6 +14,7 @@ module OpenTrace
       @state             = CLOSED
       @failure_count     = 0
       @last_failure_at   = nil
+      @half_open_probe_sent = false
       @mutex             = Mutex.new
     end
 
@@ -25,12 +26,18 @@ module OpenTrace
         when OPEN
           if Time.now - @last_failure_at >= @recovery_timeout
             @state = HALF_OPEN
+            @half_open_probe_sent = true
             true
           else
             false
           end
         when HALF_OPEN
-          false
+          if @half_open_probe_sent
+            false
+          else
+            @half_open_probe_sent = true
+            true
+          end
         end
       end
     end
@@ -38,6 +45,7 @@ module OpenTrace
     def record_success
       @mutex.synchronize do
         @failure_count = 0
+        @half_open_probe_sent = false
         @state = CLOSED
       end
     end
@@ -46,7 +54,12 @@ module OpenTrace
       @mutex.synchronize do
         @failure_count += 1
         @last_failure_at = Time.now
-        @state = OPEN if @failure_count >= @failure_threshold
+        if @state == HALF_OPEN
+          @half_open_probe_sent = false
+          @state = OPEN
+        elsif @failure_count >= @failure_threshold
+          @state = OPEN
+        end
       end
     end
 
@@ -55,6 +68,7 @@ module OpenTrace
         @state = CLOSED
         @failure_count = 0
         @last_failure_at = nil
+        @half_open_probe_sent = false
       end
     end
   end

data/lib/opentrace/client.rb

@@ -322,11 +322,20 @@ module OpenTrace
       @rate_limit_until = Time.now + retry_after
 
       # Re-enqueue batch items if space allows
+      re_enqueued = 0
       batch.each do |payload|
         break if @queue.size >= MAX_QUEUE_SIZE
-        @queue.push(payload)
-      rescue ClosedQueueError
-        break
+        begin
+          @queue.push(payload)
+          re_enqueued += 1
+        rescue ClosedQueueError
+          break
+        end
+      end
+      dropped = batch.size - re_enqueued
+      if dropped > 0
+        @stats.increment(:dropped_error, dropped)
+        fire_on_drop(dropped, :shutdown)
       end
     end
 
@@ -404,6 +413,8 @@ module OpenTrace
     end
 
     def unix_socket_send(json)
+      require "socket" unless defined?(UNIXSocket)
+
       payload = if @config.compression && json.bytesize > @config.compression_threshold
                   gzip_compress(json)
                 else
@@ -416,9 +427,14 @@ module OpenTrace
       socket.write(payload)
       socket.flush
 
-      # Read 4-byte status code response
-      response_data = socket.read(4)
-      status = response_data&.unpack1("N") || 500
+      # Read 4-byte status code response with timeout
+      if IO.select([socket], nil, nil, 5)
+        response_data = socket.read(4)
+        status = response_data&.unpack1("N") || 500
+      else
+        @stats.increment(:socket_timeouts)
+        status = 500
+      end
       socket.close
 
       UnixSocketResponse.new(status)

data/lib/opentrace/config.rb

@@ -23,7 +23,7 @@ module OpenTrace
                   :trace_propagation,
                   :log_forwarding, :view_tracking, :cache_tracking,
                   :deprecation_tracking, :detailed_request_log,
-                  :sample_rate, :sampler, :before_send,
+                  :sampler, :before_send,
                   :sql_normalization, :log_trace_injection,
                   :source_context, :before_breadcrumb,
                   :pii_scrubbing, :pii_patterns, :pii_disabled_patterns,
@@ -35,7 +35,7 @@ module OpenTrace
                 :runtime_metrics, :runtime_metrics_interval
 
     # Custom writers that invalidate caches
-    attr_reader :enabled, :min_level, :allowed_levels, :ignore_paths
+    attr_reader :enabled, :min_level, :allowed_levels, :ignore_paths, :sample_rate
 
     def enabled=(val)
       @enabled = val
@@ -56,6 +56,14 @@ module OpenTrace
       @ignore_paths = val
     end
 
+    def sample_rate=(val)
+      val = val.to_f
+      unless (0.0..1.0).cover?(val)
+        raise ArgumentError, "sample_rate must be between 0.0 and 1.0, got #{val}"
+      end
+      @sample_rate = val
+    end
+
     def initialize
       @endpoint    = nil
       @api_key     = nil

data/lib/opentrace/middleware.rb

@@ -5,6 +5,22 @@ require_relative "trace_context"
 
 module OpenTrace
   class Middleware
+    # All Fiber-local keys managed by this middleware.
+    # Centralised so cleanup never misses one.
+    FIBER_KEYS = %i[
+      opentrace_collector
+      opentrace_cached_context
+      opentrace_sql_count
+      opentrace_sql_total_ms
+      opentrace_trace_id
+      opentrace_span_id
+      opentrace_parent_span_id
+      opentrace_transaction_name
+      opentrace_breadcrumbs
+      opentrace_session_id
+      opentrace_pending_explains
+    ].freeze
+
     def initialize(app)
       @app = app
     end
@@ -65,22 +81,16 @@ module OpenTrace
         collector.memory_after = current_rss_mb
       end
 
-      Fiber[:opentrace_collector] = nil
-      Fiber[:opentrace_cached_context] = nil
-      Fiber[:opentrace_sql_count] = nil
-      Fiber[:opentrace_sql_total_ms] = nil
-      Fiber[:opentrace_trace_id] = nil
-      Fiber[:opentrace_span_id] = nil
-      Fiber[:opentrace_parent_span_id] = nil
-      Fiber[:opentrace_transaction_name] = nil
-      Fiber[:opentrace_breadcrumbs] = nil
-      Fiber[:opentrace_session_id] = nil
-      Fiber[:opentrace_pending_explains] = nil
+      cleanup_fiber_locals
       OpenTrace.current_request_id = nil
     end
 
     private
 
+    def cleanup_fiber_locals
+      FIBER_KEYS.each { |key| Fiber[key] = nil }
+    end
+
     # Extract trace context from incoming request headers.
     # Priority: W3C traceparent > X-Trace-ID > request_id > generate new
     def extract_trace_context(env)

data/lib/opentrace/payload_builder.rb

@@ -13,7 +13,9 @@ module OpenTrace
       elsif entry.is_a?(Hash)
         entry # legacy direct payload
       end
-    rescue StandardError
+    rescue StandardError => e
+      OpenTrace.stats.increment(:payload_build_errors) if OpenTrace.respond_to?(:stats)
+      $stderr.puts "[OpenTrace] PayloadBuilder error: #{e.class}: #{e.message}" if OpenTrace.respond_to?(:config) && OpenTrace.config.respond_to?(:debug) && OpenTrace.config.debug
       nil
     end
 

data/lib/opentrace/stats.rb

@@ -26,10 +26,8 @@ module OpenTrace
       @started_at = Time.now
     end
 
-    # Hot path: no mutex. Under CRuby's GIL, Hash#[]= with integer
-    # increment is effectively atomic for single operations.
     def increment(counter, amount = 1)
-      @counters[counter] += amount
+      @mutex.synchronize { @counters[counter] += amount }
     end
 
     def get(counter)

data/lib/opentrace/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module OpenTrace
-  VERSION = "0.14.1"
+  VERSION = "0.15.0"
 end

data/lib/opentrace.rb

@@ -25,13 +25,17 @@ module OpenTrace
   # Null object for when OpenTrace is not configured.
   # All methods are no-ops, avoiding nil checks on the hot path.
   class NilClient
+    def initialize
+      @nil_stats = NilStats.new
+    end
+
     def enqueue(_) = nil
     def shutdown(timeout: 5) = nil
     def queue_size = 0
     def circuit_state = :closed
     def auth_suspended? = false
     def stats_snapshot = { queue_size: 0, circuit_state: :closed, auth_suspended: false }
-    def stats = NilStats.new
+    def stats = @nil_stats
     def supports?(_) = false
   end
 

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: opentrace
 version: !ruby/object:Gem::Version
-  version: 0.14.1
+  version: 0.15.0
 platform: ruby
 authors:
 - OpenTrace