Merge pull request #591 from splitio/development

Release 10.4.0

Author: chillaq

CHANGES.txt

@@ -1,3 +1,6 @@
+10.4.0 (Aug 4, 2025)
+- Added a new optional argument to the client `getTreatment` methods to allow passing additional evaluation options, such as a map of properties to append to the generated impressions sent to Split backend. Read more in our docs.
+
 10.3.0 (Jun 17, 2025)
 - Added support for rule-based segments. These segments determine membership at runtime by evaluating their configured rules against the user attributes provided to the SDK.
 - Added support for feature flag prerequisites. This allows customers to define dependency conditions between flags, which are evaluated before any allowlists or targeting rules.

splitio/api/impressions.py

@@ -30,15 +30,7 @@ def _build_bulk(impressions):
             {
                 'f': test_name,
                 'i': [
-                    {
-                        'k': impression.matching_key,
-                        't': impression.treatment,
-                        'm': impression.time,
-                        'c': impression.change_number,
-                        'r': impression.label,
-                        'b': impression.bucketing_key,
-                        'pt': impression.previous_time
-                    }
+                    ImpressionsAPIBase._filter_out_null_prop(impression)
                     for impression in imps
                 ]
             }
@@ -48,6 +40,30 @@ def _build_bulk(impressions):
             )
         ]
 
+    @staticmethod
+    def _filter_out_null_prop(impression):
+        if impression.properties == None:
+            return {
+                    'k': impression.matching_key,
+                    't': impression.treatment,
+                    'm': impression.time,
+                    'c': impression.change_number,
+                    'r': impression.label,
+                    'b': impression.bucketing_key,
+                    'pt': impression.previous_time
+                    }
+            
+        return {
+                'k': impression.matching_key,
+                't': impression.treatment,
+                'm': impression.time,
+                'c': impression.change_number,
+                'r': impression.label,
+                'b': impression.bucketing_key,
+                'pt': impression.previous_time,
+                'properties': impression.properties
+                }
+
     @staticmethod
     def _build_counters(counters):
         """

splitio/client/client.py

@@ -6,6 +6,7 @@
 import inspect
 
 from splitio.client.key import Key
+from splitio.client import client
 from splitio.engine.evaluator import CONTROL
 
 
@@ -538,6 +539,15 @@ def validate_attributes(attributes, method_name):
 
     return True
 
+def validate_evaluation_options(evaluation_options, method_name):
+    if evaluation_options == None:
+        return None
+    
+    if not isinstance(evaluation_options, client.EvaluationOptions):
+        _LOGGER.error("%s: evaluation options should be an instance of EvaluationOptions. Setting its value to None.", method_name)
+        return None
+    
+    return evaluation_options    
 
 class _ApiLogFilter(logging.Filter):  # pylint: disable=too-few-public-methods
     def filter(self, record):
@@ -564,7 +574,7 @@ def validate_factory_instantiation(sdk_key):
     return True
 
 
-def valid_properties(properties):
+def valid_properties(properties, source):
     """
     Check if properties is a valid dict and returns the properties
     that will be sent to the track method, avoiding unexpected types.
@@ -580,7 +590,7 @@ def valid_properties(properties):
         return True, None, size
 
     if not isinstance(properties, dict):
-        _LOGGER.error('track: properties must be of type dictionary.')
+        _LOGGER.error('%s: properties must be of type dictionary.', source)
         return False, None, 0
 
     valid_properties = dict()
@@ -595,9 +605,8 @@ def valid_properties(properties):
         if element is None:
             continue
 
-        if not isinstance(element, str) and not isinstance(element, Number) \
-           and not isinstance(element, bool):
-            _LOGGER.warning('Property %s is of invalid type. Setting value to None', element)
+        if not _check_element_type(element):
+            _LOGGER.warning('%s: Property %s is of invalid type. Setting value to None', source, element)
             element = None
 
         valid_properties[property] = element
@@ -607,16 +616,22 @@ def valid_properties(properties):
 
         if size > MAX_PROPERTIES_LENGTH_BYTES:
             _LOGGER.error(
-                'The maximum size allowed for the properties is 32768 bytes. ' +
-                'Current one is ' + str(size) + ' bytes. Event not queued'
-            )
+                '%s: The maximum size allowed for the properties is 32768 bytes. ' +
+                'Current one is ' + str(size) + ' bytes. Event not queued', source)
             return False, None, size
 
     if len(valid_properties.keys()) > 300:
-        _LOGGER.warning('Event has more than 300 properties. Some of them will be trimmed' +
-                        ' when processed')
+        _LOGGER.warning('%s: Event has more than 300 properties. Some of them will be trimmed' +
+                        ' when processed', source)
     return True, valid_properties if len(valid_properties) else None, size
 
+def _check_element_type(element):
+    if not isinstance(element, str) and not isinstance(element, Number) \
+        and not isinstance(element, bool):
+            return False
+    
+    return True
+    
 def validate_pluggable_adapter(config):
     """
     Check if pluggable adapter contains the expected method signature

splitio/client/input_validator.py

@@ -89,6 +89,9 @@ async def record_unique_keys(self, uniques):
         :param uniques: unique keys disctionary
         :type uniques: Dictionary {'feature_flag1': set(), 'feature_flag2': set(), .. }
         """
+        if len(uniques) == 0:
+            return
+
         await self._telemtry_http_client.record_unique_keys({'keys': self._uniques_formatter(uniques)})
 
 
@@ -184,6 +187,9 @@ async def record_unique_keys(self, uniques):
         :param uniques: unique keys disctionary
         :type uniques: Dictionary {'feature_flag1': set(), 'feature_flag2': set(), .. }
         """
+        if len(uniques) == 0:
+            return True
+
         bulk_mtks = _uniques_formatter(uniques)
         try:
             inserted = await self._redis_client.rpush(_MTK_QUEUE_KEY, *bulk_mtks)
@@ -202,6 +208,9 @@ async def flush_counters(self, to_send):
         :param to_send: unique keys disctionary
         :type to_send: Dictionary {'feature_flag1': set(), 'feature_flag2': set(), .. }
         """
+        if len(to_send) == 0:
+            return True
+
         try:
             resulted = 0
             counted = 0
@@ -277,6 +286,7 @@ def flush_counters(self, to_send):
         """
         if len(to_send) == 0:
             return
+
         try:
             resulted = 0
             for pf_count in to_send:
@@ -325,6 +335,9 @@ async def record_unique_keys(self, uniques):
         :param uniques: unique keys disctionary
         :type uniques: Dictionary {'feature_flag1': set(), 'feature_flag2': set(), .. }
         """
+        if len(uniques) == 0:
+            return True
+        
         bulk_mtks = _uniques_formatter(uniques)
         try:
             inserted = await self._adapter_client.push_items(self._prefix + _MTK_QUEUE_KEY, *bulk_mtks)
@@ -343,6 +356,9 @@ async def flush_counters(self, to_send):
         :param to_send: unique keys disctionary
         :type to_send: Dictionary {'feature_flag1': set(), 'feature_flag2': set(), .. }
         """
+        if len(to_send) == 0:
+            return True
+
         try:
             resulted = 0
             for pf_count in to_send:

splitio/engine/impressions/adapters.py

@@ -38,7 +38,14 @@ def process_impressions(self, impressions):
         :returns: Tuple of to be stored, observed and counted impressions, and unique keys tuple
         :rtype: list[tuple[splitio.models.impression.Impression, dict]], list[], list[], list[]
         """
-        imps = [(self._observer.test_and_set(imp), attrs) for imp, attrs in impressions]
+        imps = []
+        for imp, attrs in impressions:
+            if imp.properties is not None:
+                imps.append((imp, attrs))
+                continue
+            
+            imps.append((self._observer.test_and_set(imp), attrs))
+            
         return [i for i, _ in imps], imps, [], []
 
 class StrategyNoneMode(BaseStrategy):
@@ -85,7 +92,14 @@ def process_impressions(self, impressions):
         :returns: Tuple of to be stored, observed and counted impressions, and unique keys tuple
         :rtype: list[tuple[splitio.models.impression.Impression, dict]], list[splitio.models.impression.Impression], list[splitio.models.impression.Impression], list[]
         """
-        imps = [(self._observer.test_and_set(imp), attrs) for imp, attrs in impressions]
+        imps = []
+        for imp, attrs in impressions:
+            if imp.properties is not None:
+                imps.append((imp, attrs))
+                continue
+
+            imps.append((self._observer.test_and_set(imp), attrs))
+        
         counter_imps = [imp for imp, _ in imps if imp.previous_time != None]
         this_hour = truncate_time(utctime_ms())
         return [i for i, _ in imps if i.previous_time is None or i.previous_time < this_hour], imps, counter_imps, []

splitio/engine/impressions/strategies.py

@@ -12,7 +12,8 @@
         'change_number',
         'bucketing_key',
         'time',
-        'previous_time'
+        'previous_time',
+        'properties'
     ]
 )
 

splitio/models/impressions.py

@@ -1231,6 +1231,7 @@ def _wrap_impressions(self, impressions):
                         'r': impression.label,
                         'c': impression.change_number,
                         'm': impression.time,
+                        'properties': impression.properties
                     }
                 }
             bulk_impressions.append(json.dumps(to_store))

splitio/storage/pluggable.py

@@ -1100,6 +1100,7 @@ def _wrap_impressions(self, impressions):
                         'r': impression.label,
                         'c': impression.change_number,
                         'm': impression.time,
+                        'properties': impression.properties
                     }
                 }
             bulk_impressions.append(json.dumps(to_store))

splitio/storage/redis.py

@@ -1 +1 @@
-__version__ = '10.3.0'
+__version__ = '10.4.0'