Merge pull request #158 from Flagsmith/release/3.3.1

Release 3.3.1

Author: matthewelwell

flag_engine/segments/models.py

@@ -56,7 +56,10 @@ def evaluate_not_contains(self, trait_value: typing.Iterable) -> bool:
         return self.value not in trait_value
 
     def evaluate_regex(self, trait_value: str) -> bool:
-        return re.compile(str(self.value)).match(trait_value) is not None
+        return (
+            trait_value is not None
+            and re.compile(str(self.value)).match(str(trait_value)) is not None
+        )
 
     def evaluate_modulo(self, trait_value: typing.Union[str, int, float, bool]) -> bool:
         if type(trait_value) not in (int, float):

setup.py

@@ -2,7 +2,7 @@
 
 setup(
     name="flagsmith-flag-engine",
-    version="3.3.0",
+    version="3.3.1",
     author="Flagsmith",
     author_email="[email protected]",
     packages=find_packages(include=["flag_engine", "flag_engine.*"]),

tests/unit/segments/test_segments_models.py

@@ -60,6 +60,8 @@
         (constants.REGEX, "foo", r"[a-z]+", True),
         (constants.REGEX, "FOO", r"[a-z]+", False),
         (constants.REGEX, "1.2.3", r"\d", True),
+        (constants.REGEX, 1, r"\d", True),
+        (constants.REGEX, None, r"[a-z]", False),
         (constants.IN, "foo", "", False),
         (constants.IN, "foo", "foo, bar", True),
         (constants.IN, "foo", "foo", True),