kameleoon-client-ruby 3.17.0 → 3.17.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a9254260f1962a3abfa79d2a583188660b28540faea927f0eb04efc1898dc4c0
-  data.tar.gz: 231728dd53a2e4662b4b705fa63905a10a1b5598eb1875e1959a4124cf6334da
+  metadata.gz: 7fc2b3610def24ae9ceb3a3e11f8a1e6e360c12359631c47c1af18e7bdd67a2b
+  data.tar.gz: ebe8a7cd7920d0522c8cde5f09e02db0050b3ccbd81fe47ffde2452c1c40c136
 SHA512:
-  metadata.gz: d90ed8a74b0e64bd244f85c79381bdaad76adff7103cd1c1afde6578d98b4838d53f8466040b6fa6537706e91eb6589d0c558193be892b0ddb6785d93e4894de
-  data.tar.gz: 33dca1c8fcde8dcb441dd9eb001b329d4798a9900411c98a1a020331a832734d52f74d2e7b9204ffe5576f6c4cc8a0039610214e6b7fa1e9bd82ad6e12f2d8f1
+  metadata.gz: 2f4ec5f9d42e0260ad0646a837654c412b15e42583763992ab7df04167357e4d670874cfad5abfb017212cbcce224cef5828e4f7df10ae94305ee7efa1eb36ec
+  data.tar.gz: 2c562cccbdf690902f653fc13b961a3197ef493be9886253842e369943c2935d90fc50bac9e5aba98a4c1bff0171db1a6a663bf14c1b169a19bd418936d59f80

data/lib/kameleoon/kameleoon_client.rb

@@ -1058,11 +1058,19 @@ module Kameleoon
       holdout = @data_manager.data_file.holdout
       return true if holdout.nil?
 
-      in_holdout_variation_key = 'in-holdout'
       Logging::KameleoonLogger.debug(
         "CALL: KameleoonClient.visitor_not_in_holdout?(visitor, visitor_code: '%s', track: %s, save: %s, " \
         'bucketing_custom_data_index: %s)', visitor_code, track, save, bucketing_custom_data_index
       )
+
+      consent, blocking_behaviour = _get_consent_and_blocking_behaviour(visitor)
+      if consent == DataManager::LegalConsent::NOT_GIVEN &&
+         blocking_behaviour == Configuration::ConsentBlockingBehaviour::COMPLETELY_BLOCKED
+        raise FeatureEnvironmentDisabled,
+              "Evaluation for a holdout is blocked because visitor's consent was not provided."
+      end
+
+      in_holdout_variation_key = 'in-holdout'
       is_not_in_holdout = true
       code_for_hash = get_code_for_hash(visitor, visitor_code, bucketing_custom_data_index)
       variation_hash = Utils::Hasher.obtain(code_for_hash, holdout.id)
@@ -1179,9 +1187,7 @@ module Kameleoon
         visitor_code, feature_flag
       )
       visitor = @visitor_manager.get_visitor(visitor_code)
-      datafile = @data_manager.data_file
-      consent = datafile.settings.is_consent_required ?
-        visitor&.legal_consent || DataManager::LegalConsent::UNKNOWN : DataManager::LegalConsent::GIVEN
+      consent, blocking_behaviour = _get_consent_and_blocking_behaviour(visitor)
       code_for_hash = get_code_for_hash(visitor, visitor_code, feature_flag.bucketing_custom_data_index)
       # no rules -> return default_variation_key
       eval_exp = nil
@@ -1207,7 +1213,6 @@ module Kameleoon
         if hash_rule <= rule.exposition
           # Checking if the evaluation is blocked due to the consent policy
           if (consent == DataManager::LegalConsent::NOT_GIVEN) && rule.experimentation_type?
-            blocking_behaviour = datafile.settings.blocking_behaviour_if_consent_not_given
             break if blocking_behaviour == Configuration::ConsentBlockingBehaviour::PARTIALLY_BLOCKED
 
             # TODO: In the next major, create a new error type for the Completely Blocked by Consent error
@@ -1243,6 +1248,18 @@ module Kameleoon
       eval_exp
     end
 
+    def _get_consent_and_blocking_behaviour(visitor)
+      datafile = @data_manager.data_file
+      behaviour = datafile.settings.blocking_behaviour_if_consent_not_given
+
+      consent = DataManager::LegalConsent::GIVEN
+      if datafile.settings.is_consent_required
+        consent = visitor&.legal_consent || DataManager::LegalConsent::UNKNOWN
+      end
+
+      [consent, behaviour]
+    end
+
     def calculate_variation_key(eval_exp, default_value)
       Logging::KameleoonLogger.debug(
         "CALL: KameleoonClient.calculate_variation_key(eval_exp: %s, default_value: '%s') ", eval_exp, default_value

data/lib/kameleoon/managers/remote_data/remote_visitor_data.rb

@@ -24,15 +24,16 @@ module Kameleoon
           @filter = filter
           @visit_number = 0
           current_visit = hash['currentVisit']
-          parse_visit(current_visit, false) unless current_visit.nil?
+          parse_visit(current_visit, 0) unless current_visit.nil?
           previous_visits = hash['previousVisits']
           previous_visits = [] if previous_visits.nil?
 
           if previous_visits.size.positive?
             prev_visits = []
-            previous_visits.each do |visit|
+            previous_visits.each_index do |i|
+              visit = previous_visits[i]
               prev_visits.push(VisitorVisits::Visit.new(visit['timeStarted'] || 0, visit['timeLastEvent']))
-              parse_visit(visit, true)
+              parse_visit(visit, i + 1)
             end
             @visitor_visits = VisitorVisits.new(prev_visits, @visit_number)
           end
@@ -74,7 +75,7 @@ module Kameleoon
 
         private
 
-        def parse_visit(hash, is_prev_visit)
+        def parse_visit(hash, visit_offset)
           @visitor_code = hash['visitorCode'] if @visitor_code.nil?
           custom_data_events = hash['customDataEvents']
           parse_custom_data(custom_data_events) if !custom_data_events.nil? && custom_data_events.size.positive?
@@ -88,7 +89,7 @@ module Kameleoon
           @geolocation = parse_geolocation(geolocation_events) if @geolocation.nil? && !geolocation_events.nil? && \
                                                                   geolocation_events.size.positive?
           static_data_events = hash['staticDataEvent']
-          parse_static_data(static_data_events, is_prev_visit) unless static_data_events.nil?
+          parse_static_data(static_data_events, visit_offset) unless static_data_events.nil?
           personalization_events = hash['personalizationEvents']
           parse_personalizations(personalization_events) unless personalization_events.nil?
         end
@@ -146,11 +147,11 @@ module Kameleoon
           Geolocation.new(data['country'], data['region'], data['city'])
         end
 
-        def parse_static_data(static_data_event, is_prev_visit)
+        def parse_static_data(static_data_event, visit_offset)
           data = static_data_event['data']
           if @visit_number.zero?
-            @visit_number = data['visitNumber'] || 0
-            @visit_number += 1 if is_prev_visit && @visit_number.positive?
+            remote_visit_number = data['visitNumber']
+            @visit_number = remote_visit_number + visit_offset unless remote_visit_number.nil?
           end
           if @filter.device && @device.nil?
             device_type = data['deviceType']

data/lib/kameleoon/version.rb

@@ -1,6 +1,6 @@
 # frozen_string_literal: true
 
 module Kameleoon
-  SDK_VERSION = '3.17.0'
+  SDK_VERSION = '3.17.1'
   SDK_NAME = 'RUBY'
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kameleoon-client-ruby
 version: !ruby/object:Gem::Version
-  version: 3.17.0
+  version: 3.17.1
 platform: ruby
 authors:
 - Kameleoon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-11-21 00:00:00.000000000 Z
+date: 2025-12-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: em-http-request