fix: move logging configuration from MCPServer.__init__ to run()

MCPServer.__init__() called configure_logging() which invokes
logging.basicConfig(), configuring the root logger with handlers
and level on every instantiation. This violates Python's logging
best practice for libraries: library code must never configure
logging — only application entrypoints should.

Move configure_logging() to run(), which is the actual application
entrypoint. MCPServer used as a library no longer has logging side
effects.

Github-Issue: #1656
Reported-by: SimonZehetner

Author: omar-y-abdi

src/mcp/server/mcpserver/server.py

@@ -194,9 +194,6 @@ def __init__(
             self._token_verifier = ProviderTokenVerifier(auth_server_provider)
         self._custom_starlette_routes: list[Route] = []
 
-        # Configure logging
-        configure_logging(self.settings.log_level)
-
     @property
     def name(self) -> str:
         return self._lowlevel_server.name
@@ -278,6 +275,8 @@ def run(
             transport: Transport protocol to use ("stdio", "sse", or "streamable-http")
             **kwargs: Transport-specific options (see overloads for details)
         """
+        configure_logging(self.settings.log_level)
+
         TRANSPORTS = Literal["stdio", "sse", "streamable-http"]
         if transport not in TRANSPORTS.__args__:  # type: ignore  # pragma: no cover
             raise ValueError(f"Unknown transport: {transport}")

tests/server/mcpserver/test_server.py

@@ -1481,3 +1481,17 @@ async def test_report_progress_passes_related_request_id():
         message="halfway",
         related_request_id="req-abc-123",
     )
+
+
+def test_init_does_not_configure_logging():
+    """MCPServer.__init__ must not call logging.basicConfig or add handlers.
+
+    Libraries should never configure the root logger — that is the
+    application's responsibility.  Logging should only be configured
+    when the server is run as a standalone process via ``run()``.
+
+    Regression test for https://github.com/modelcontextprotocol/python-sdk/issues/1656
+    """
+    with patch("mcp.server.mcpserver.server.configure_logging") as mock_configure:
+        MCPServer("test")
+        mock_configure.assert_not_called()