kameleoon-client-ruby 3.16.1 → 3.16.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ab7859ca5fc3cf2097ba9c8d1e18e64ae4d00661a24449a85badf08ea83fecef
-  data.tar.gz: c265f043634947577b5d61f3ff064102894fff1ede4145d24e4d2a6cbe022bdf
+  metadata.gz: 602844d249763c9165d740e924c71b8d98efbfdb77c8f70ef4cd0ad361dfb86f
+  data.tar.gz: 1a9ee1ce8a3b3548f38ba19f21d3f0de86aeba513549405870bcd3150a617608
 SHA512:
-  metadata.gz: 7783e97d595ec980ecb65ac14747c5c305da7690b60c7e4190e0b3d16e2b78dd3bc93400bf478b20dd314db31dd393955bd6e72eb2d70b1a1b96fbb61e399322
-  data.tar.gz: 6308a1a932f3fd654cc52924c55610994e5cae900a982e05459a57d31224ed3719330595a69aa8f07ed6f163019f52113c8159a8b321c6fd486e00bd7c6f0b45
+  metadata.gz: 8988b718fac058005be7ac57bc7519c9cc8ad4e6dd95d03e2505e3210eea483fc619362959b8af6788d5ee5f06045f52941b27cbca57f0b11b82c45e63e92612
+  data.tar.gz: 41c1a2c1c2a7903dd352b5fe3a515d4f2639656eb05de89a19791e2d50ae0c4ef5b4336649491e2dc358dd55c51e92ab1e411132a7c7b1cd9784735c6bae3e81

data/lib/kameleoon/data/custom_data.rb

@@ -10,6 +10,8 @@ module Kameleoon
   class CustomData < DuplicationUnsafeData
     attr_reader :index, :name, :values, :overwrite
 
+    UNDEFINED_INDEX = -1
+
     def to_s
       "CustomData{index:#{@index},name:'#{@name}',values:#{@values},overwrite:#{@overwrite}}"
     end
@@ -35,13 +37,13 @@ module Kameleoon
         index = hash['index'] || hash['id']
         name = hash['name']
         if index.nil? && name.nil?
-          raise Kameleoon::Exception::NotFound.new('index/id/name'), '"index/id/name" is mandatory'
+          raise Exception::NotFound.new('index/id/name'), '"index/id/name" is mandatory'
         end
 
-        @index = index || -1
+        @index = index || CustomData::UNDEFINED_INDEX
         @name = name
         values = hash['values']
-        raise Kameleoon::Exception::NotFound.new('values'), '"values" is mandatory' if values.nil?
+        raise Exception::NotFound.new('values'), '"values" is mandatory' if values.nil?
 
         @values = values.is_a?(Array) ? values.dup : [values]
         @overwrite = hash.fetch('overwrite', true)
@@ -50,10 +52,10 @@ module Kameleoon
         when Integer
           @index = arg0
         when String
-          @index = -1
+          @index = CustomData::UNDEFINED_INDEX
           @name = arg0
         else
-          raise Kameleoon::Exception.new('Unexpected arg0 type'), 'Unexpected arg0 type'
+          raise Exception.new('Unexpected arg0 type'), 'Unexpected arg0 type'
         end
         @values = args
         @overwrite = overwrite
@@ -62,7 +64,7 @@ module Kameleoon
       return if @index.is_a?(Integer)
 
       Logging::KameleoonLogger.warning("CustomData field 'index' must be of 'Integer' type")
-      @index = @index.is_a?(String) ? @index.to_i : -1
+      @index = @index.is_a?(String) ? @index.to_i : CustomData::UNDEFINED_INDEX
     end
     # rubocop:enable Metrics/MethodLength
 
@@ -92,7 +94,7 @@ module Kameleoon
         overwrite: @overwrite,
         nonce: nonce
       }
-      Kameleoon::Network::UriHelper.encode_query(params)
+      Network::UriHelper.encode_query(params)
     end
   end
 end

data/lib/kameleoon/data/manager/data_array_storage.rb

@@ -6,13 +6,6 @@ module Kameleoon
     # DataArrayStorage is a readonly accessor to a sequntial storage of specific visitor data.
     # It is thread-safe
     class DataArrayStorage
-
-      def to_s
-        values = []
-        enumerate { |v| values << v }
-        "DataArrayStorage{values:#{values.join(',')}}"
-      end
-
       def initialize(mutex, array)
         @mutex = mutex
         @array = array
@@ -45,6 +38,12 @@ module Kameleoon
         end
         value
       end
+
+      def to_s
+        values = []
+        enumerate { |v| values << v }
+        "DataArrayStorage{values:#{values.join(',')}}"
+      end
     end
   end
 end

data/lib/kameleoon/data/manager/visitor_manager.rb

@@ -3,6 +3,8 @@
 require 'concurrent'
 require 'rufus/scheduler'
 require 'kameleoon/configuration/custom_data_info'
+require 'kameleoon/data/conversion'
+require 'kameleoon/data/custom_data'
 require 'kameleoon/data/mapping_identifier'
 require 'kameleoon/data/manager/visitor'
 require 'kameleoon/logging/kameleoon_logger'
@@ -71,15 +73,21 @@ module Kameleoon
       end
 
       def add_data(visitor_code, *args, overwrite: true)
-        Logging::KameleoonLogger.debug("CALL: VisitorManager.add_data(visitor_code: '%s', args: %s, overwrite: %s)",
-                                       visitor_code, args, overwrite)
+        Logging::KameleoonLogger.debug(
+          "CALL: VisitorManager.add_data(visitor_code: '%s', args: %s, overwrite: %s)",
+          visitor_code, args, overwrite
+        )
         visitor = get_or_create_visitor(visitor_code)
         cdi = @data_manager.data_file.custom_data_info
         args.size.times do |i|
-          custom_data = args[i]
-          if custom_data.is_a?(Kameleoon::CustomData)
-            args[i] = process_custom_data(cdi, visitor_code, visitor, custom_data)
+          data = args[i]
+          case data
+          when CustomData
+            data = process_custom_data(cdi, visitor_code, visitor, data)
+          when Conversion
+            data = process_conversion(data, cdi)
           end
+          args[i] = data
         end
         visitor.add_data(*args, overwrite: overwrite)
         Logging::KameleoonLogger.debug(
@@ -92,12 +100,13 @@ module Kameleoon
       private
 
       def process_custom_data(cdi, visitor_code, visitor, custom_data)
-        unless custom_data.name.nil?
-          cd_index = cdi.get_custom_data_index_by_name(custom_data.name)
-          return nil if cd_index.nil?
-
-          custom_data = custom_data.named_to_indexed(cd_index)
+        mapped_cd = try_map_custom_data_index_by_name(custom_data, cdi)
+        if mapped_cd.nil?
+          Logging::KameleoonLogger.error('%s is invalid and will be ignored', custom_data)
+          return nil
         end
+        custom_data = mapped_cd
+
         # We shouldn't send custom data with local only type
         custom_data.mark_as_sent if cdi.local_only?(custom_data.index)
         # If mappingIdentifier is passed, we should link anonymous visitor with real unique userId.
@@ -115,6 +124,23 @@ module Kameleoon
         custom_data
       end
 
+      def process_conversion(conversion, cdi)
+        conversion.metadata&.each_index do |i|
+          cd = try_map_custom_data_index_by_name(conversion.metadata[i], cdi)
+          Logging::KameleoonLogger.warning('Conversion metadata %s is invalid', conversion.metadata[i]) if cd.nil?
+          conversion.metadata[i] = cd
+        end
+        conversion
+      end
+
+      def try_map_custom_data_index_by_name(custom_data, cdi)
+        return custom_data if custom_data.index != CustomData::UNDEFINED_INDEX
+        return nil if custom_data.name.nil?
+
+        cd_index = cdi.get_custom_data_index_by_name(custom_data.name)
+        cd_index.nil? ? nil : custom_data.named_to_indexed(cd_index)
+      end
+
       def purge
         Logging::KameleoonLogger.debug('CALL: VisitorManager.purge')
         expired_time = Time.new.to_i - @expiration_period

data/lib/kameleoon/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: kameleoon-client-ruby
 version: !ruby/object:Gem::Version
-  version: 3.16.1
+  version: 3.16.2
 platform: ruby
 authors:
 - Kameleoon
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2025-10-16 00:00:00.000000000 Z
+date: 2025-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: em-http-request