From dbcf9d93ca17a44cc6cf3133dd7a0beacdc9aa5e Mon Sep 17 00:00:00 2001 From: Chris Coutinho Date: Fri, 17 Oct 2025 04:37:25 +0200 Subject: [PATCH] chore: Improve RequestError message details Show exception type and cause when str(e) is empty for better debugging --- nextcloud_mcp_server/server/cookbook.py | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/nextcloud_mcp_server/server/cookbook.py b/nextcloud_mcp_server/server/cookbook.py index 8561b01..fdbcc43 100644 --- a/nextcloud_mcp_server/server/cookbook.py +++ b/nextcloud_mcp_server/server/cookbook.py @@ -84,10 +84,15 @@ def configure_cookbook_tools(mcp: FastMCP): recipe_id=recipe.id or "unknown", ) except RequestError as e: + # RequestError can have empty str() - get details from exception attributes + error_detail = ( + str(e) + or f"{type(e).__name__}: {getattr(e, '__cause__', 'unknown cause')}" + ) raise McpError( ErrorData( code=-1, - message=f"Network error importing recipe from {url}: {str(e)}", + message=f"Network error importing recipe from {url}: {error_detail}", ) ) except HTTPStatusError as e: