From 2b052d9d98633797ac8d1bb9a66fb995ebe4822e Mon Sep 17 00:00:00 2001 From: Marcelo Trylesinski Date: Thu, 22 Jan 2026 09:38:37 +0100 Subject: [PATCH] refactor: drop unused logger/logging --- .../clients/sse-polling-client/mcp_sse_polling_client/main.py | 2 -- .../simple-auth/mcp_simple_auth/simple_auth_provider.py | 3 --- src/mcp/client/auth/utils.py | 3 --- src/mcp/client/client.py | 3 --- src/mcp/server/websocket.py | 3 --- tests/server/test_streamable_http_security.py | 2 -- 6 files changed, 16 deletions(-) diff --git a/examples/clients/sse-polling-client/mcp_sse_polling_client/main.py b/examples/clients/sse-polling-client/mcp_sse_polling_client/main.py index 533fce3789..3b3171205e 100644 --- a/examples/clients/sse-polling-client/mcp_sse_polling_client/main.py +++ b/examples/clients/sse-polling-client/mcp_sse_polling_client/main.py @@ -21,8 +21,6 @@ from mcp import ClientSession from mcp.client.streamable_http import streamable_http_client -logger = logging.getLogger(__name__) - async def run_demo(url: str, items: int, checkpoint_every: int) -> None: """Run the SSE polling demo.""" diff --git a/examples/servers/simple-auth/mcp_simple_auth/simple_auth_provider.py b/examples/servers/simple-auth/mcp_simple_auth/simple_auth_provider.py index e244e0fd94..3a3895cc57 100644 --- a/examples/servers/simple-auth/mcp_simple_auth/simple_auth_provider.py +++ b/examples/servers/simple-auth/mcp_simple_auth/simple_auth_provider.py @@ -8,7 +8,6 @@ """ -import logging import secrets import time from typing import Any @@ -29,8 +28,6 @@ ) from mcp.shared.auth import OAuthClientInformationFull, OAuthToken -logger = logging.getLogger(__name__) - class SimpleAuthSettings(BaseSettings): """Simple OAuth settings for demo purposes.""" diff --git a/src/mcp/client/auth/utils.py b/src/mcp/client/auth/utils.py index 1ce57818dc..1aa960b9ce 100644 --- a/src/mcp/client/auth/utils.py +++ b/src/mcp/client/auth/utils.py @@ -1,4 +1,3 @@ -import logging import re from urllib.parse import urljoin, urlparse @@ -16,8 +15,6 @@ ) from mcp.types import LATEST_PROTOCOL_VERSION -logger = logging.getLogger(__name__) - def extract_field_from_www_auth(response: Response, field_name: str) -> str | None: """Extract field from WWW-Authenticate header. diff --git a/src/mcp/client/client.py b/src/mcp/client/client.py index 6eafb794a3..72d4048cc3 100644 --- a/src/mcp/client/client.py +++ b/src/mcp/client/client.py @@ -2,7 +2,6 @@ from __future__ import annotations -import logging from contextlib import AsyncExitStack from typing import Any @@ -22,8 +21,6 @@ from mcp.server.fastmcp import FastMCP from mcp.shared.session import ProgressFnT -logger = logging.getLogger(__name__) - class Client: """A high-level MCP client for connecting to MCP servers. diff --git a/src/mcp/server/websocket.py b/src/mcp/server/websocket.py index 9df3e25c87..61d314c28b 100644 --- a/src/mcp/server/websocket.py +++ b/src/mcp/server/websocket.py @@ -1,4 +1,3 @@ -import logging from contextlib import asynccontextmanager import anyio @@ -10,8 +9,6 @@ import mcp.types as types from mcp.shared.message import SessionMessage -logger = logging.getLogger(__name__) - @asynccontextmanager # pragma: no cover async def websocket_server(scope: Scope, receive: Receive, send: Send): diff --git a/tests/server/test_streamable_http_security.py b/tests/server/test_streamable_http_security.py index a637b1dce0..897555353e 100644 --- a/tests/server/test_streamable_http_security.py +++ b/tests/server/test_streamable_http_security.py @@ -1,6 +1,5 @@ """Tests for StreamableHTTP server DNS rebinding protection.""" -import logging import multiprocessing import socket from collections.abc import AsyncGenerator @@ -19,7 +18,6 @@ from mcp.types import Tool from tests.test_helpers import wait_for_server -logger = logging.getLogger(__name__) SERVER_NAME = "test_streamable_http_security_server"