omit check_request params with None values

Signed-off-by: Sam Lock <[email protected]>
cerbos · Dec 19, 2024 · 0b3b2a1 · 0b3b2a1
1 parent da53079
commit 0b3b2a1
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 4 deletions.
diff --git a/cerbos/sdk/_async/_http.py b/cerbos/sdk/_async/_http.py
@@ -196,7 +196,11 @@ async def check_resources(
             resources=resources.resources,
             aux_data=aux_data,
         )
-        resp = await self._http.post("/api/check/resources", json=req.to_dict())
+
+        # omit keys with `None` values
+        req_params = {k: v for k, v in req.to_dict().items() if v is not None}
+
+        resp = await self._http.post("/api/check/resources", json=req_params)
         if resp.is_error:
             if self._raise_on_error:
                 raise CerbosRequestException(APIError.from_dict(resp.json()))

diff --git a/cerbos/sdk/_sync/_http.py b/cerbos/sdk/_sync/_http.py
@@ -196,7 +196,11 @@ def check_resources(
             resources=resources.resources,
             aux_data=aux_data,
         )
-        resp = self._http.post("/api/check/resources", json=req.to_dict())
+
+        # omit keys with `None` values
+        req_params = {k: v for k, v in req.to_dict().items() if v is not None}
+
+        resp = self._http.post("/api/check/resources", json=req_params)
         if resp.is_error:
             if self._raise_on_error:
                 raise CerbosRequestException(APIError.from_dict(resp.json()))

diff --git a/cerbos/sdk/model.py b/cerbos/sdk/model.py
@@ -272,7 +272,9 @@ def __init__(self, error: Optional[APIError]):
         self.error = error
 
 
-class CerbosTLSError(Exception): ...
+class CerbosTLSError(Exception):
+    ...
 
 
-class CerbosTypeError(Exception): ...
+class CerbosTypeError(Exception):
+    ...