From 98a8ccff4a3eac703d0fa27f27a3b85fcb49a473 Mon Sep 17 00:00:00 2001 From: Simon Zehetner <48550834+SimonZehetner@users.noreply.github.com> Date: Fri, 6 Feb 2026 19:46:23 +0100 Subject: [PATCH] fix: Remove logging initialization from `MCPServer.__init__` --- src/mcp/server/mcpserver/server.py | 5 +--- src/mcp/server/mcpserver/utilities/logging.py | 24 ------------------- 2 files changed, 1 insertion(+), 28 deletions(-) diff --git a/src/mcp/server/mcpserver/server.py b/src/mcp/server/mcpserver/server.py index 8c1fc342b..ea94a11af 100644 --- a/src/mcp/server/mcpserver/server.py +++ b/src/mcp/server/mcpserver/server.py @@ -36,7 +36,7 @@ from mcp.server.mcpserver.resources import FunctionResource, Resource, ResourceManager from mcp.server.mcpserver.tools import Tool, ToolManager from mcp.server.mcpserver.utilities.context_injection import find_context_parameter -from mcp.server.mcpserver.utilities.logging import configure_logging, get_logger +from mcp.server.mcpserver.utilities.logging import get_logger from mcp.server.sse import SseServerTransport from mcp.server.stdio import stdio_server from mcp.server.streamable_http import EventStore @@ -167,9 +167,6 @@ def __init__( # Set up MCP protocol handlers self._setup_handlers() - # Configure logging - configure_logging(self.settings.log_level) - @property def name(self) -> str: return self._lowlevel_server.name diff --git a/src/mcp/server/mcpserver/utilities/logging.py b/src/mcp/server/mcpserver/utilities/logging.py index c394f2bfa..c8d2e5005 100644 --- a/src/mcp/server/mcpserver/utilities/logging.py +++ b/src/mcp/server/mcpserver/utilities/logging.py @@ -1,7 +1,6 @@ """Logging utilities for MCPServer.""" import logging -from typing import Literal def get_logger(name: str) -> logging.Logger: @@ -14,26 +13,3 @@ def get_logger(name: str) -> logging.Logger: a configured logger instance """ return logging.getLogger(name) - - -def configure_logging( - level: Literal["DEBUG", "INFO", "WARNING", "ERROR", "CRITICAL"] = "INFO", -) -> None: - """Configure logging for MCP. - - Args: - level: the log level to use - """ - handlers: list[logging.Handler] = [] - try: - from rich.console import Console - from rich.logging import RichHandler - - handlers.append(RichHandler(console=Console(stderr=True), rich_tracebacks=True)) - except ImportError: # pragma: no cover - pass - - if not handlers: # pragma: no cover - handlers.append(logging.StreamHandler()) - - logging.basicConfig(level=level, format="%(message)s", handlers=handlers)