bitfab 0.17.1 → 0.18.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 9dd280a2cd6afdf47b2bbbf62508dd4918dccf42c83ed276845f592497168601
-  data.tar.gz: 224fc7cfe39e847f3bfd56e9876151ea356f7a8ad7bff66b529f9704ab360141
+  metadata.gz: 4401b0c7aa47516909e636557ab23f325fb0cbbd4ca8564f3d8864078e58f975
+  data.tar.gz: 2b11fbfee500e0e03ec7b4b61772d623e2c236479965441b659aafbed4a687a6
 SHA512:
-  metadata.gz: 4cba57762a9ae927ba34ac70fdd1d750d118a9f8f3cc47402f30d8a09c336337f27cef601f8b36e63140e01acaa410d1a58450f160cb1c3d7e7c2c5b1a198cb3
-  data.tar.gz: eaa0a0b2eed0c0f7c0e24a73e2128540c53a504e9a3dd4d6858355e8076aa2fa8a4913ea170b278ac1579c5e407d4333759e1bae345dbe575b9fa91b666275a1
+  metadata.gz: e720bb8dd79d572f4671b5f891cb1528dac2b610d2357d1f16ec5a7e04656fff0969f434e5e683be991d8745865bacbbe1e37993380e1a09d2cc9377e221eda5
+  data.tar.gz: c197da185d7966cd9861213c49f0db4e3246a5eab0bdb70e800d8a4a5a4e8f174b5294d833511232555d43f0ca026a7988297fa3660743a8fef94b05b17a7ca9

data/lib/bitfab/http_client.rb

@@ -5,6 +5,7 @@ require "json"
 require "uri"
 
 require_relative "constants"
+require_relative "serialize"
 require_relative "version"
 
 module Bitfab
@@ -32,7 +33,7 @@ module Bitfab
         http.read_timeout = request_timeout
 
         req = Net::HTTP::Post.new(uri.path, headers)
-        req.body = JSON.generate(payload)
+        req.body = safe_generate(payload, endpoint)
 
         response = http.request(req)
 
@@ -166,6 +167,46 @@ module Bitfab
 
     private
 
+    # JSON-encode a request body without ever raising on a stray value.
+    #
+    # Upstream serialization (Serialize.serialize_value) should already have
+    # flattened user data. This is the boundary backstop: if anything
+    # non-serializable still slips through, it is run through serialize_value
+    # (which never raises and stubs strays) instead of letting JSON.generate
+    # raise and drop the whole span/trace silently. A degraded payload warns
+    # loudly so the trace isn't quietly left incomplete or not replayable.
+    def safe_generate(payload, endpoint)
+      JSON.generate(payload)
+    rescue => e
+      begin
+        warn "Bitfab: request body to #{endpoint} held a non-serializable " \
+             "value (#{e.message}); it was stubbed so the span still sends, " \
+             "but the trace may be incomplete or not replayable. Capture a " \
+             "JSON-safe projection of this input to make it replayable."
+      rescue
+        # Never crash the host app over a log line.
+      end
+
+      begin
+        JSON.generate(sanitize_payload(payload))
+      rescue
+        # Truly pathological. Still never drop silently: send a marker body.
+        JSON.generate({"error" => "payload_serialize_failed"})
+      end
+    end
+
+    # Serialize each top-level value so a bad or oversize value is stubbed in
+    # place while the payload keeps its object shape. Running serialize_value on
+    # the whole payload could collapse the entire body to a single stub string
+    # (oversize/cyclic), sending a JSON string instead of a span object.
+    def sanitize_payload(payload)
+      return {"error" => "payload_serialize_failed"} unless payload.is_a?(Hash)
+
+      payload.each_with_object({}) do |(k, v), acc|
+        acc[k.to_s] = Serialize.serialize_value(v)
+      end
+    end
+
     # Normalize each entry to a hash with stable string keys, accepting either
     # symbol-keyed or string-keyed hashes from callers.
     def normalize_code_change_files(files)

data/lib/bitfab/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module Bitfab
-  VERSION = "0.17.1"
+  VERSION = "0.18.0"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: bitfab
 version: !ruby/object:Gem::Version
-  version: 0.17.1
+  version: 0.18.0
 platform: ruby
 authors:
 - Harvest Team