From abbb43632f8a6358cc01ae22d4ec53c4ea303e97 Mon Sep 17 00:00:00 2001 From: CREDO23 Date: Mon, 6 Jul 2026 03:17:25 +0200 Subject: [PATCH] fix(mcp): omit unset query params from REST requests --- surfsense_mcp/src/surfsense_mcp/core/client.py | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/surfsense_mcp/src/surfsense_mcp/core/client.py b/surfsense_mcp/src/surfsense_mcp/core/client.py index b61de5221..6635d011b 100644 --- a/surfsense_mcp/src/surfsense_mcp/core/client.py +++ b/surfsense_mcp/src/surfsense_mcp/core/client.py @@ -49,6 +49,10 @@ class SurfSenseClient: files: Any | None = None, ) -> Any: """Send a request and return the parsed body, or raise ``ToolError``.""" + # Omit unset query params: sending them empty makes the API parse "" + # as a value (e.g. int("") on folder_id) and fail. + if params is not None: + params = {key: value for key, value in params.items() if value is not None} try: response = await self._http.request( method, path, params=params, json=json, data=data, files=files