haha okay

Author: dmarticus

posthog/client.py

@@ -714,6 +714,9 @@ def get_feature_flag(
         if self.disabled:
             return None
 
+        # Normalize key to lowercase
+        key = str(key).lower()
+
         person_properties, group_properties = self._add_local_person_and_group_properties(
             distinct_id, groups, person_properties, group_properties
         )
@@ -724,7 +727,8 @@ def get_feature_flag(
 
         if self.feature_flags:
             for flag in self.feature_flags:
-                if flag["key"] == key:
+                # Compare lowercase keys
+                if str(flag["key"]).lower() == key:
                     try:
                         response = self._compute_flag_locally(
                             flag,
@@ -855,10 +859,18 @@ def _compute_payload_locally(self, key, match_value):
         if self.feature_flags_by_key is None:
             return payload
 
-        flag_definition = self.feature_flags_by_key.get(key) or {}
-        flag_filters = flag_definition.get("filters") or {}
-        flag_payloads = flag_filters.get("payloads") or {}
-        payload = flag_payloads.get(str(match_value).lower(), None)
+        # Normalize key to lowercase when looking up flag definition
+        key = str(key).lower()
+        flag_definition = None
+        for flag_key, flag in self.feature_flags_by_key.items():
+            if str(flag_key).lower() == key:
+                flag_definition = flag
+                break
+
+        if flag_definition:
+            flag_filters = flag_definition.get("filters") or {}
+            flag_payloads = flag_filters.get("payloads") or {}
+            payload = flag_payloads.get(str(match_value).lower(), None)
         return payload
 
     def get_all_flags(

posthog/test/test_feature_flags.py

@@ -4640,3 +4640,47 @@ def test_multivariate_flag_consistency(self, patch_get):
                 self.assertEqual(feature_flag_match, results[i])
             else:
                 self.assertFalse(feature_flag_match)
+
+    @mock.patch("posthog.client.decide")
+    def test_feature_flag_case_insensitive(self, mock_decide):
+        client = Client(api_key=FAKE_TEST_API_KEY, personal_api_key=FAKE_TEST_API_KEY)
+        client.feature_flags = [
+            {
+                "id": 1,
+                "key": "Beta-Feature",
+                "active": True,
+                "filters": {
+                    "groups": [{"properties": [], "rollout_percentage": 100}],
+                },
+            }
+        ]
+
+        # Test that flag evaluation works regardless of case
+        self.assertTrue(client.feature_enabled("Beta-Feature", "user1"))
+        self.assertTrue(client.feature_enabled("beta-feature", "user1"))
+        self.assertTrue(client.feature_enabled("BETA-FEATURE", "user1"))
+
+    @mock.patch("posthog.client.decide")
+    def test_feature_flag_mixed_case_consistency(self, mock_decide):
+        client = Client(api_key=FAKE_TEST_API_KEY, personal_api_key=FAKE_TEST_API_KEY)
+        client.feature_flags = [
+            {
+                "id": 1,
+                "key": "Beta-Feature",
+                "active": True,
+                "filters": {
+                    "groups": [{"properties": [], "rollout_percentage": 100}],
+                    "payloads": {
+                        "true": {"some": "value"},
+                    }
+                },
+            }
+        ]
+
+        # Test that flag evaluation and payload retrieval are consistent
+        # regardless of the case used
+        test_cases = ["Beta-Feature", "beta-feature", "BETA-FEATURE", "bEtA-FeAtUrE"]
+        
+        for case in test_cases:
+            # Both the flag evaluation and payload retrieval should work
+            self.assertTrue(client.feature_enabled(case, "user1"))