flagsmith 3.6.0__tar.gz → 3.8.0__tar.gz

{flagsmith-3.6.0 → flagsmith-3.8.0}/PKG-INFO

@@ -1,6 +1,6 @@
 Metadata-Version: 2.1
 Name: flagsmith
-Version: 3.6.0
+Version: 3.8.0
 Summary: Flagsmith Python SDK
 License: BSD3
 Keywords: feature,flag,flagsmith,remote,config
@@ -14,9 +14,9 @@ Classifier: Programming Language :: Python :: 3.10
 Classifier: Programming Language :: Python :: 3.11
 Classifier: Programming Language :: Python :: 3.12
 Requires-Dist: flagsmith-flag-engine (>=5.1.0,<6.0.0)
-Requires-Dist: pytz (>=2023.4,<2024.0)
-Requires-Dist: requests (>=2.27.1,<3.0.0)
-Requires-Dist: requests-futures (>=1.0.0,<2.0.0)
+Requires-Dist: pydantic (>=2,<3)
+Requires-Dist: requests (>=2.32.3,<3.0.0)
+Requires-Dist: requests-futures (>=1.0.1,<2.0.0)
 Requires-Dist: sseclient-py (>=1.8.0,<2.0.0)
 Project-URL: Documentation, https://docs.flagsmith.com
 Description-Content-Type: text/markdown
@@ -33,12 +33,12 @@ The SDK for Python applications for [https://www.flagsmith.com/](https://www.fla
 ## Adding to your project
 
 For full documentation visit
-[https://docs.flagsmith.com/clients/server-side](https://docs.flagsmith.com/clients/server-side).
+[https://docs.flagsmith.com/clients/server-side?language=python](https://docs.flagsmith.com/clients/server-side?language=python).
 
 ## Contributing
 
-Please read [CONTRIBUTING.md](https://gist.github.com/kyle-ssg/c36a03aebe492e45cbd3eefb21cb0486) for details on our code
-of conduct, and the process for submitting pull requests
+Please read [CONTRIBUTING.md](CONTRIBUTING.md) for details on our code of conduct, and the process for submitting pull
+requests
 
 ## Getting Help
 

flagsmith-3.6.0/Readme.md → flagsmith-3.8.0/README.md

@@ -10,12 +10,12 @@ The SDK for Python applications for [https://www.flagsmith.com/](https://www.fla
 ## Adding to your project
 
 For full documentation visit
-[https://docs.flagsmith.com/clients/server-side](https://docs.flagsmith.com/clients/server-side).
+[https://docs.flagsmith.com/clients/server-side?language=python](https://docs.flagsmith.com/clients/server-side?language=python).
 
 ## Contributing
 
-Please read [CONTRIBUTING.md](https://gist.github.com/kyle-ssg/c36a03aebe492e45cbd3eefb21cb0486) for details on our code
-of conduct, and the process for submitting pull requests
+Please read [CONTRIBUTING.md](CONTRIBUTING.md) for details on our code of conduct, and the process for submitting pull
+requests
 
 ## Getting Help
 

{flagsmith-3.6.0 → flagsmith-3.8.0}/flagsmith/exceptions.py

@@ -4,3 +4,7 @@ class FlagsmithClientError(Exception):
 
 class FlagsmithAPIError(FlagsmithClientError):
     pass
+
+
+class FlagsmithFeatureDoesNotExistError(FlagsmithClientError):
+    pass

{flagsmith-3.6.0 → flagsmith-3.8.0}/flagsmith/flagsmith.py

@@ -1,9 +1,8 @@
-import json
 import logging
 import typing
-from datetime import datetime
+from datetime import timezone
 
-import pytz
+import pydantic
 import requests
 from flag_engine import engine
 from flag_engine.environments.models import EnvironmentModel
@@ -20,23 +19,14 @@ from flagsmith.models import DefaultFlag, Flags, Segment
 from flagsmith.offline_handlers import BaseOfflineHandler
 from flagsmith.polling_manager import EnvironmentDataPollingManager
 from flagsmith.streaming_manager import EventStreamManager, StreamEvent
-from flagsmith.utils.identities import Identity, generate_identities_data
+from flagsmith.types import JsonType, TraitConfig, TraitMapping
+from flagsmith.utils.identities import generate_identity_data
 
 logger = logging.getLogger(__name__)
 
 DEFAULT_API_URL = "https://edge.api.flagsmith.com/api/v1/"
 DEFAULT_REALTIME_API_URL = "https://realtime.flagsmith.com/"
 
-JsonType = typing.Union[
-    None,
-    int,
-    str,
-    bool,
-    typing.List["JsonType"],
-    typing.List[typing.Mapping[str, "JsonType"]],
-    typing.Dict[str, "JsonType"],
-]
-
 
 class Flagsmith:
     """A Flagsmith client.
@@ -169,8 +159,10 @@ class Flagsmith:
                 )
 
     def _initialise_local_evaluation(self) -> None:
+        # To ensure that the environment is set before allowing subsequent
+        # method calls, update the environment manually.
+        self.update_environment()
         if self.enable_realtime_updates:
-            self.update_environment()
             if not self._environment:
                 raise ValueError("Unable to get environment from API key")
 
@@ -185,9 +177,6 @@ class Flagsmith:
             self.event_stream_thread.start()
 
         else:
-            # To ensure that the environment is set before allowing subsequent
-            # method calls, update the environment manually.
-            self.update_environment()
             self.environment_data_polling_manager_thread = (
                 EnvironmentDataPollingManager(
                     main=self,
@@ -199,30 +188,15 @@ class Flagsmith:
             self.environment_data_polling_manager_thread.start()
 
     def handle_stream_event(self, event: StreamEvent) -> None:
-        try:
-            event_data = json.loads(event.data)
-        except json.JSONDecodeError as e:
-            raise FlagsmithAPIError("Unable to get valid json from event data.") from e
-
-        try:
-            stream_updated_at = datetime.fromtimestamp(event_data.get("updated_at"))
-        except TypeError as e:
-            raise FlagsmithAPIError(
-                "Unable to get valid timestamp from event data."
-            ) from e
-
-        if stream_updated_at.tzinfo is None:
-            stream_updated_at = pytz.utc.localize(stream_updated_at)
-
         if not self._environment:
             raise ValueError(
                 "Unable to access environment. Environment should not be null"
             )
         environment_updated_at = self._environment.updated_at
         if environment_updated_at.tzinfo is None:
-            environment_updated_at = pytz.utc.localize(environment_updated_at)
+            environment_updated_at = environment_updated_at.astimezone(timezone.utc)
 
-        if stream_updated_at > environment_updated_at:
+        if event.updated_at > environment_updated_at:
             self.update_environment()
 
     def get_environment_flags(self) -> Flags:
@@ -238,7 +212,9 @@ class Flagsmith:
     def get_identity_flags(
         self,
         identifier: str,
-        traits: typing.Optional[typing.Mapping[str, TraitValue]] = None,
+        traits: typing.Optional[TraitMapping] = None,
+        *,
+        transient: bool = False,
     ) -> Flags:
         """
         Get all the flags for the current environment for a given identity. Will also
@@ -248,13 +224,20 @@ class Flagsmith:
         :param identifier: a unique identifier for the identity in the current
             environment, e.g. email address, username, uuid
         :param traits: a dictionary of traits to add / update on the identity in
-            Flagsmith, e.g. {"num_orders": 10}
+            Flagsmith, e.g. `{"num_orders": 10}`. Envelope traits you don't want persisted
+            in a dictionary with `"transient"` and `"value"` keys, e.g.
+            `{"num_orders": 10, "color": {"value": "pink", "transient": True}}`.
+        :param transient: if `True`, the identity won't get persisted
         :return: Flags object holding all the flags for the given identity.
         """
         traits = traits or {}
         if (self.offline_mode or self.enable_local_evaluation) and self._environment:
             return self._get_identity_flags_from_document(identifier, traits)
-        return self._get_identity_flags_from_api(identifier, traits)
+        return self._get_identity_flags_from_api(
+            identifier,
+            traits,
+            transient=transient,
+        )
 
     def get_identity_segments(
         self,
@@ -282,16 +265,15 @@ class Flagsmith:
         return [Segment(id=sm.id, name=sm.name) for sm in segment_models]
 
     def update_environment(self) -> None:
-        self._environment = self._get_environment_from_api()
-        self._update_overrides()
-
-    def _update_overrides(self) -> None:
-        if not self._environment:
-            return
-        if overrides := self._environment.identity_overrides:
-            self._identity_overrides_by_identifier = {
-                identity.identifier: identity for identity in overrides
-            }
+        try:
+            self._environment = self._get_environment_from_api()
+        except (FlagsmithAPIError, pydantic.ValidationError):
+            logger.exception("Error updating environment")
+        else:
+            if overrides := self._environment.identity_overrides:
+                self._identity_overrides_by_identifier = {
+                    identity.identifier: identity for identity in overrides
+                }
 
     def _get_environment_from_api(self) -> EnvironmentModel:
         environment_data = self._get_json_response(self.environment_url, method="GET")
@@ -307,7 +289,7 @@ class Flagsmith:
         )
 
     def _get_identity_flags_from_document(
-        self, identifier: str, traits: typing.Mapping[str, TraitValue]
+        self, identifier: str, traits: TraitMapping
     ) -> Flags:
         identity_model = self._get_identity_model(identifier, **traits)
         if self._environment is None:
@@ -324,9 +306,9 @@ class Flagsmith:
 
     def _get_environment_flags_from_api(self) -> Flags:
         try:
-            json_response: typing.List[
-                typing.Mapping[str, JsonType]
-            ] = self._get_json_response(url=self.environment_flags_url, method="GET")
+            json_response: typing.List[typing.Mapping[str, JsonType]] = (
+                self._get_json_response(url=self.environment_flags_url, method="GET")
+            )
             return Flags.from_api_flags(
                 api_flags=json_response,
                 analytics_processor=self._analytics_processor,
@@ -340,14 +322,24 @@ class Flagsmith:
             raise
 
     def _get_identity_flags_from_api(
-        self, identifier: str, traits: typing.Mapping[str, typing.Any]
+        self,
+        identifier: str,
+        traits: TraitMapping,
+        *,
+        transient: bool = False,
     ) -> Flags:
+        request_body = generate_identity_data(
+            identifier,
+            traits,
+            transient=transient,
+        )
         try:
-            data = generate_identities_data(identifier, traits)
-            json_response: typing.Dict[
-                str, typing.List[typing.Dict[str, JsonType]]
-            ] = self._get_json_response(
-                url=self.identities_url, method="POST", body=data
+            json_response: typing.Dict[str, typing.List[typing.Dict[str, JsonType]]] = (
+                self._get_json_response(
+                    url=self.identities_url,
+                    method="POST",
+                    body=request_body,
+                )
             )
             return Flags.from_api_flags(
                 api_flags=json_response["flags"],
@@ -365,22 +357,16 @@ class Flagsmith:
         self,
         url: str,
         method: str,
-        body: typing.Optional[
-            typing.Union[Identity, typing.Dict[str, JsonType]]
-        ] = None,
+        body: typing.Optional[JsonType] = None,
     ) -> typing.Any:
         try:
             request_method = getattr(self.session, method.lower())
             response = request_method(
                 url, json=body, timeout=self.request_timeout_seconds
             )
-            if response.status_code != 200:
-                raise FlagsmithAPIError(
-                    "Invalid request made to Flagsmith API. Response status code: %d",
-                    response.status_code,
-                )
+            response.raise_for_status()
             return response.json()
-        except (requests.ConnectionError, json.JSONDecodeError) as e:
+        except requests.RequestException as e:
             raise FlagsmithAPIError(
                 "Unable to get valid response from Flagsmith API."
             ) from e
@@ -388,7 +374,7 @@ class Flagsmith:
     def _get_identity_model(
         self,
         identifier: str,
-        **traits: TraitValue,
+        **traits: typing.Union[TraitValue, TraitConfig],
     ) -> IdentityModel:
         if not self._environment:
             raise FlagsmithClientError(
@@ -396,7 +382,10 @@ class Flagsmith:
             )
 
         trait_models = [
-            TraitModel(trait_key=key, trait_value=value)
+            TraitModel(
+                trait_key=key,
+                trait_value=value["value"] if isinstance(value, dict) else value,
+            )
             for key, value in traits.items()
         ]
 

{flagsmith-3.6.0 → flagsmith-3.8.0}/flagsmith/models.py

@@ -6,7 +6,7 @@ from dataclasses import dataclass, field
 from flag_engine.features.models import FeatureStateModel
 
 from flagsmith.analytics import AnalyticsProcessor
-from flagsmith.exceptions import FlagsmithClientError
+from flagsmith.exceptions import FlagsmithFeatureDoesNotExistError
 
 
 @dataclass
@@ -135,7 +135,9 @@ class Flags:
         except KeyError:
             if self.default_flag_handler:
                 return self.default_flag_handler(feature_name)
-            raise FlagsmithClientError("Feature does not exist: %s" % feature_name)
+            raise FlagsmithFeatureDoesNotExistError(
+                "Feature does not exist: %s" % feature_name
+            )
 
         if self._analytics_processor and hasattr(flag, "feature_name"):
             self._analytics_processor.track_feature(flag.feature_name)

{flagsmith-3.6.0 → flagsmith-3.8.0}/flagsmith/polling_manager.py

@@ -5,10 +5,6 @@ import threading
 import time
 import typing
 
-import requests
-
-from flagsmith.exceptions import FlagsmithAPIError
-
 if typing.TYPE_CHECKING:
     from flagsmith import Flagsmith
 
@@ -30,10 +26,7 @@ class EnvironmentDataPollingManager(threading.Thread):
 
     def run(self) -> None:
         while not self._stop_event.is_set():
-            try:
-                self.main.update_environment()
-            except (FlagsmithAPIError, requests.RequestException):
-                logger.exception("Failed to update environment")
+            self.main.update_environment()
             time.sleep(self.refresh_interval_seconds)
 
     def stop(self) -> None:

{flagsmith-3.6.0 → flagsmith-3.8.0}/flagsmith/streaming_manager.py

@@ -1,18 +1,17 @@
 import logging
 import threading
 import typing
-from typing import Callable, Generator, Optional, Protocol, cast
+from typing import Callable, Optional
 
+import pydantic
 import requests
 import sseclient
 
-from flagsmith.exceptions import FlagsmithAPIError
-
 logger = logging.getLogger(__name__)
 
 
-class StreamEvent(Protocol):
-    data: str
+class StreamEvent(pydantic.BaseModel):
+    updated_at: pydantic.AwareDatetime
 
 
 class EventStreamManager(threading.Thread):
@@ -22,7 +21,7 @@ class EventStreamManager(threading.Thread):
         stream_url: str,
         on_event: Callable[[StreamEvent], None],
         request_timeout_seconds: Optional[int] = None,
-        **kwargs: typing.Any
+        **kwargs: typing.Any,
     ) -> None:
         super().__init__(*args, **kwargs)
         self._stop_event = threading.Event()
@@ -39,17 +38,12 @@ class EventStreamManager(threading.Thread):
                     headers={"Accept": "application/json, text/event-stream"},
                     timeout=self.request_timeout_seconds,
                 ) as response:
-                    sse_client = sseclient.SSEClient(
-                        cast(Generator[bytes, None, None], response)
-                    )
+                    sse_client = sseclient.SSEClient(chunk for chunk in response)
                     for event in sse_client.events():
-                        self.on_event(event)
-
-            except requests.exceptions.ReadTimeout:
-                pass
+                        self.on_event(StreamEvent.model_validate_json(event.data))
 
-            except (FlagsmithAPIError, requests.RequestException):
-                logger.exception("Error handling event stream")
+            except (requests.RequestException, pydantic.ValidationError):
+                logger.exception("Error opening or reading from the event stream")
 
     def stop(self) -> None:
         self._stop_event.set()

flagsmith-3.8.0/flagsmith/types.py

@@ -0,0 +1,25 @@
+import typing
+
+from flag_engine.identities.traits.types import TraitValue
+from typing_extensions import TypeAlias
+
+_JsonScalarType: TypeAlias = typing.Union[
+    int,
+    str,
+    float,
+    bool,
+    None,
+]
+JsonType: TypeAlias = typing.Union[
+    _JsonScalarType,
+    typing.Dict[str, "JsonType"],
+    typing.List["JsonType"],
+]
+
+
+class TraitConfig(typing.TypedDict):
+    value: TraitValue
+    transient: bool
+
+
+TraitMapping: TypeAlias = typing.Mapping[str, typing.Union[TraitValue, TraitConfig]]

flagsmith-3.8.0/flagsmith/utils/identities.py

@@ -0,0 +1,26 @@
+import typing
+
+from flagsmith.types import JsonType, TraitMapping
+
+
+def generate_identity_data(
+    identifier: str,
+    traits: TraitMapping,
+    *,
+    transient: bool,
+) -> JsonType:
+    identity_data: typing.Dict[str, JsonType] = {"identifier": identifier}
+    traits_data: typing.List[JsonType] = []
+    for trait_key, trait_value in traits.items():
+        trait_data: typing.Dict[str, JsonType] = {"trait_key": trait_key}
+        if isinstance(trait_value, dict):
+            trait_data["trait_value"] = trait_value["value"]
+            if trait_value.get("transient"):
+                trait_data["transient"] = True
+        else:
+            trait_data["trait_value"] = trait_value
+        traits_data.append(trait_data)
+    identity_data["traits"] = traits_data
+    if transient:
+        identity_data["transient"] = True
+    return identity_data

{flagsmith-3.6.0 → flagsmith-3.8.0}/pyproject.toml

@@ -1,42 +1,39 @@
 [tool.poetry]
 name = "flagsmith"
-version = "3.6.0"
+version = "3.8.0"
 description = "Flagsmith Python SDK"
 authors = ["Flagsmith <support@flagsmith.com>"]
 license = "BSD3"
-readme = "Readme.md"
+readme = "README.md"
 keywords = ["feature", "flag", "flagsmith", "remote", "config"]
 documentation = "https://docs.flagsmith.com"
 packages = [{ include = "flagsmith" }]
 
 [tool.poetry.dependencies]
 python = ">=3.8.1,<4"
-requests = "^2.27.1"
-requests-futures = "^1.0.0"
+requests = "^2.32.3"
+requests-futures = "^1.0.1"
 flagsmith-flag-engine = "^5.1.0"
 sseclient-py = "^1.8.0"
-pytz = "^2023.4"
+pydantic = "^2"
 
 [tool.poetry.group.dev]
 optional = true
 
 [tool.poetry.group.dev.dependencies]
 pytest = "^7.4.0"
+pytest-cov = "^4.1.0"
 pytest-mock = "^3.6.1"
-black = "^23.3.0"
 pre-commit = "^2.17.0"
 responses = "^0.24.1"
-flake8 = "^6.1.0"
-isort = "^5.12.0"
-mypy = "^1.7.1"
-types-requests = "^2.31.0.10"
-pytest-cov = "^4.1.0"
-types-pytz = "^2024.1.0.20240203"
+types-requests = "^2.32"
 
 [tool.mypy]
 plugins = ["pydantic.mypy"]
 exclude = ["example/*"]
 
+[tool.black]
+target-version = ["py38"]
 
 [build-system]
 requires = ["poetry-core>=1.0.0"]

flagsmith-3.6.0/flagsmith/utils/identities.py

@@ -1,19 +0,0 @@
-import typing
-
-from flag_engine.identities.traits.types import TraitValue
-
-Identity = typing.TypedDict(
-    "Identity",
-    {"identifier": str, "traits": typing.List[typing.Mapping[str, TraitValue]]},
-)
-
-
-def generate_identities_data(
-    identifier: str, traits: typing.Optional[typing.Mapping[str, TraitValue]] = None
-) -> Identity:
-    return {
-        "identifier": identifier,
-        "traits": [{"trait_key": k, "trait_value": v} for k, v in traits.items()]
-        if traits
-        else [],
-    }