fix: allow anything that implements str as a context property (#353)

* fix: allow anything that implements str as a context property

Author: sighphyre

UnleashClient/__init__.py

@@ -522,7 +522,7 @@ def _safe_context_value(self, value):
             return value.isoformat()
         if isinstance(value, (int, float)):
             return str(value)
-        return value
+        return str(value)
 
     def _resolve_variant(self, feature_name: str, context: dict) -> dict:
         """

tests/unit_tests/test_client.py

@@ -1044,6 +1044,24 @@ def test_is_enabled_works_with_properties_field_in_the_context_root():
     assert unleash_client.is_enabled("customContextToggle", context)
 
 
+def test_uuids_are_valid_context_properties():
+    unleash_client = UnleashClient(
+        URL,
+        APP_NAME,
+        disable_metrics=True,
+        disable_registration=True,
+    )
+
+    context = {"userId": uuid.uuid4()}
+
+    try:
+        unleash_client.is_enabled("testFlag", context)
+    except Exception as e:
+        assert (
+            False
+        ), f"An exception was raised when passing a UUID as a context property: {e}"
+
+
 @responses.activate
 def test_identification_headers_sent_and_consistent(unleash_client):
     responses.add(responses.POST, URL + REGISTER_URL, json={}, status=202)