This is an automated email from the ASF dual-hosted git repository.

cgivre pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/drill-mcp.git

commit 80cdb1fbd838111c3586d86a20ab3751151c3a44
Author: cgivre <[email protected]>
AuthorDate: Wed Aug 12 16:27:47 2026 -0400

    fix: catch DrillError alongside ConfigError in main()
    
    drill-mcp --auth kerberos without the kerberos extra died with a raw
    traceback instead of the clear message client_rest.py raises, because
    build_client's DrillError is only reachable once build_server(config)
    runs the client constructor. Widen main()'s try block to cover that
    call and its except clause to catch DrillError too.
---
 drill_mcp/server.py | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drill_mcp/server.py b/drill_mcp/server.py
index 349efe8..c4c4bf3 100644
--- a/drill_mcp/server.py
+++ b/drill_mcp/server.py
@@ -330,10 +330,10 @@ def main(argv: list[str] | None = None) -> int:
     }
     try:
         config = load_config(args.config, overrides=overrides)
-    except ConfigError as exc:
+        build_server(config).run()
+    except (ConfigError, DrillError) as exc:
         print(f"drill-mcp: configuration error: {exc}", file=sys.stderr)
         return 1
-    build_server(config).run()
     return 0
 
 

Reply via email to