posthog-ruby 3.5.1 → 3.5.3

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 47d81093ca138f86158d4c964efd30df01421a1fab08c6beadb8d80781cfa39b
-  data.tar.gz: 7ca8768e6ceb39e74f6905d44c6542c76381a351bd628a6a5754a29fcf5af30c
+  metadata.gz: 9401944abdf80537015fd06feb6f1ce795bf0c011271f22216537c75a062dcb8
+  data.tar.gz: cea7ce9ff778f181f1e011657b1423ffe74ecb530cc1a5213e43b5d1663424ef
 SHA512:
-  metadata.gz: 5561844d134133bc4c948f557013cb7a4128c5848a12658311e44cd02b0d5ef033bf43ec7314e63139d9adf826ea94e2337ce77e923b7b9ef3a0ac0263b0af9b
-  data.tar.gz: 26386d521bf09da5526c2363cee3eb6ad55b58dc30f217801963e96dc30137518baf9de7550889d66b06b009ef7504f0a4abe571d3b174f64ad3dc0171cf5d4e
+  metadata.gz: a1a717bec693f592ce65145bbb2a41cf7764bbecddfa6eb8ac95aca68ac43f6a36357397106e1929b607495c681448900076e0dd0f024d0d2ee14071f23bc8f7
+  data.tar.gz: 4ff35396b4819bd246fa38096b38715863741d7419f798598f75f4e32c28388e5548e5ff9fafa17cd096e9dfb38c6a94eb20d1a58eb04868ed961d5beeaf1d80

data/lib/posthog/exception_capture.rb

@@ -67,7 +67,7 @@ module PostHog
         'platform' => 'ruby'
       }
 
-      add_context_lines(frame, file, lineno) if File.exist?(file)
+      add_context_lines(frame, file, lineno) if frame['in_app'] && File.exist?(file)
 
       frame
     end

data/lib/posthog/feature_flag.rb

@@ -3,7 +3,7 @@
 # Represents a feature flag returned by /flags v2
 module PostHog
   class FeatureFlag
-    attr_reader :key, :enabled, :variant, :reason, :metadata
+    attr_reader :key, :enabled, :variant, :reason, :metadata, :failed
 
     def initialize(json)
       json.transform_keys!(&:to_s)
@@ -12,6 +12,7 @@ module PostHog
       @variant = json['variant']
       @reason = json['reason'] ? EvaluationReason.new(json['reason']) : nil
       @metadata = json['metadata'] ? FeatureFlagMetadata.new(json['metadata'].transform_keys(&:to_s)) : nil
+      @failed = json['failed']
     end
 
     # TODO: Rename to `value` in future version

data/lib/posthog/feature_flags.rb

@@ -131,8 +131,10 @@ module PostHog
           FeatureFlag.new(flag)
         end
         flags_response[:flags] = flags_hash
-        flags_response[:featureFlags] = flags_hash.transform_values(&:get_value).transform_keys(&:to_sym)
-        flags_response[:featureFlagPayloads] = flags_hash.transform_values(&:payload).transform_keys(&:to_sym)
+        # Filter out flags that failed evaluation to avoid overwriting cached values
+        successful_flags = flags_hash.reject { |_, flag| flag.failed == true }
+        flags_response[:featureFlags] = successful_flags.transform_values(&:get_value).transform_keys(&:to_sym)
+        flags_response[:featureFlagPayloads] = successful_flags.transform_values(&:payload).transform_keys(&:to_sym)
       elsif flags_response[:featureFlags]
         # v3 format
         flags_response[:featureFlags] = flags_response[:featureFlags] || {}
@@ -309,8 +311,19 @@ module PostHog
             flags = {}
             payloads = {}
           else
-            flags = stringify_keys(flags_and_payloads[:featureFlags] || {})
-            payloads = stringify_keys(flags_and_payloads[:featureFlagPayloads] || {})
+            server_flags = stringify_keys(flags_and_payloads[:featureFlags] || {})
+            server_payloads = stringify_keys(flags_and_payloads[:featureFlagPayloads] || {})
+
+            if errors_while_computing
+              # Merge server results into locally-evaluated results so that flags which
+              # failed server-side evaluation (already filtered out by get_flags) don't
+              # overwrite valid locally-evaluated values.
+              flags = stringify_keys(flags).merge(server_flags)
+              payloads = stringify_keys(payloads).merge(server_payloads)
+            else
+              flags = server_flags
+              payloads = server_payloads
+            end
           end
         rescue StandardError => e
           @on_error.call(-1, "Error computing flag remotely: #{e}")

data/lib/posthog/send_worker.rb

@@ -43,8 +43,10 @@ module PostHog
           consume_message_from_queue! until @batch.full? || @queue.empty?
         end
 
-        res = @transport.send @api_key, @batch
-        @on_error.call(res.status, res.error) unless res.status == 200
+        unless @batch.empty?
+          res = @transport.send @api_key, @batch
+          @on_error.call(res.status, res.error) unless res.status == 200
+        end
 
         @lock.synchronize { @batch.clear }
       end

data/lib/posthog/transport.rb

@@ -50,7 +50,12 @@ module PostHog
       last_response, exception =
         retry_with_backoff(@retries) do
           status_code, body = send_request(api_key, batch)
-          error = JSON.parse(body)['error']
+          error =
+            begin
+              JSON.parse(body)['error']
+            rescue JSON::ParserError
+              body
+            end
           should_retry = should_retry_request?(status_code, body)
           logger.debug("Response status code: #{status_code}")
           logger.debug("Response error: #{error}") if error

data/lib/posthog/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module PostHog
-  VERSION = '3.5.1'
+  VERSION = '3.5.3'
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: posthog-ruby
 version: !ruby/object:Gem::Version
-  version: 3.5.1
+  version: 3.5.3
 platform: ruby
 authors:
 - ''