Support key signing

Signed-off-by: Mathias L. Baumann <[email protected]>

Author: Marenz

RELEASE_NOTES.md

@@ -6,14 +6,14 @@
 
 ## Upgrading
 
-<!-- Here goes notes on how to upgrade from previous versions, including deprecations and what they should be replaced with -->
+* The `key` parameter in the `Dispatcher` constructor is now deprecated. Use `sign_secret` instead.
 
 ## New Features
 
-<!-- Here goes the main new features and examples or instructions on how to use them -->
+* Two new parmeters where added to the `Dispatcher` constructor:
+  * `sign_secret`: A secret key used for signing messages.
+  * `auth_key`: An authentication key for the Dispatch API.
 
 ## Bug Fixes
 
-* The merge by type class now uses the correct logger path.
-* The merge by type was made more robust under heavy load, making sure to use the same `now` for all dispatches that are checked.
-* Fix that the merge filter was using an outdated dispatches dict once fetch() ran.
+<!-- Here goes notable bug fixes that are worth a special mention or explanation -->

pyproject.toml

@@ -40,8 +40,7 @@ dependencies = [
   # plugins.mkdocstrings.handlers.python.import)
   "frequenz-sdk >= 1.0.0-rc2100, < 1.0.0-rc2200",
   "frequenz-channels >= 1.6.1, < 2.0.0",
-  "frequenz-client-dispatch >= 0.11.1, < 0.12.0",
-  "frequenz-client-common >= 0.3.2, < 0.4.0",
+  "frequenz-client-dispatch >= 0.11.2, < 0.12.0",
   "frequenz-client-base >= 0.11.0, < 0.12.0",
 ]
 dynamic = ["version"]
@@ -54,7 +53,7 @@ email = "[email protected]"
 dev-flake8 = [
   "flake8 == 7.3.0",
   "flake8-docstrings == 1.7.0",
-  "flake8-pyproject == 1.2.3", # For reading the flake8 config from pyproject.toml
+  "flake8-pyproject == 1.2.3",  # For reading the flake8 config from pyproject.toml
   "pydoclint == 0.6.6",
   "pydocstyle == 6.3.0",
 ]

src/frequenz/dispatch/_dispatcher.py

@@ -64,7 +64,7 @@ async def run():
             async with Dispatcher(
                 microgrid_id=microgrid_id,
                 server_url=url,
-                key=key
+                auth_key=key
             ) as dispatcher:
                 dispatcher.start_managing(
                     dispatch_type="DISPATCH_TYPE",
@@ -90,7 +90,7 @@ async def run():
             async with Dispatcher(
                 microgrid_id=microgrid_id,
                 server_url=url,
-                key=key
+                auth_key=key
             ) as dispatcher:
                 actor = MagicMock() # replace with your actor
 
@@ -135,7 +135,7 @@ async def run():
             async with Dispatcher(
                 microgrid_id=microgrid_id,
                 server_url=url,
-                key=key,
+                auth_key=key,
             ) as dispatcher:
                 events_receiver = dispatcher.new_lifecycle_events_receiver("DISPATCH_TYPE")
 
@@ -172,7 +172,7 @@ async def run():
             async with Dispatcher(
                 microgrid_id=microgrid_id,
                 server_url=url,
-                key=key,
+                auth_key=key,
             ) as dispatcher:
                 # Create a new dispatch
                 new_dispatch = await dispatcher.client.create(
@@ -205,7 +205,9 @@ def __init__(
         *,
         microgrid_id: MicrogridId,
         server_url: str,
-        key: str,
+        key: str | None = None,
+        auth_key: str | None = None,
+        sign_secret: str | None = None,
         call_timeout: timedelta = timedelta(seconds=60),
         stream_timeout: timedelta = timedelta(minutes=5),
     ):
@@ -214,15 +216,29 @@ def __init__(
         Args:
             microgrid_id: The microgrid id.
             server_url: The URL of the dispatch service.
-            key: The key to access the service.
+            key: The key to access the service, deprecated, use `auth_key` instead.
+            auth_key: The authentication key to access the service.
+            sign_secret: The secret to sign the requests, optional
             call_timeout: The timeout for API calls.
             stream_timeout: The timeout for streaming API calls.
+
+        Raises:
+            ValueError: If both `key` and `auth_key` are provided.
         """
         super().__init__()
 
+        if key is not None and auth_key is not None:
+            raise ValueError(
+                "Both 'key' and 'auth_key' are provided, please use only 'auth_key'."
+            )
+
+        if key is not None:
+            auth_key = key
+
         self._client = DispatchApiClient(
             server_url=server_url,
-            key=key,
+            auth_key=key,
+            sign_secret=sign_secret,
             call_timeout=call_timeout,
             stream_timeout=stream_timeout,
         )

tests/test_managing_actor.py

@@ -335,20 +335,25 @@ async def test_manage_abstraction(
     class MyFakeClient(FakeClient):
         """Fake client for testing."""
 
+        # pylint: disable=too-many-arguments,unused-argument
         def __init__(
             self,
             *,
             server_url: str,
-            key: str,
+            auth_key: str | None = None,
+            key: str | None = None,
+            sign_secret: str | None = None,
             call_timeout: timedelta,
             stream_timeout: timedelta,
         ) -> None:
             assert server_url
-            assert key
+            assert key is None or auth_key is None
             assert call_timeout
             assert stream_timeout
             super().__init__()
 
+        # pylint: enable=too-many-arguments,unused-argument
+
     mid = MicrogridId(1)
 
     # Patch `Client` class in Dispatcher with MyFakeClient