From e93eb9d302fa8c8e089fbfd7a7790918242feddf Mon Sep 17 00:00:00 2001
From: Chris Coutinho <chris@coutinho.io>
Date: Sun, 25 May 2025 11:46:41 +0200
Subject: [PATCH] fix: Configure logging

---
 Dockerfile                     |  6 +-----
 nextcloud_mcp_server/client.py |  3 ---
 nextcloud_mcp_server/config.py |  5 +----
 nextcloud_mcp_server/server.py | 10 ++++------
 4 files changed, 6 insertions(+), 18 deletions(-)

diff --git a/Dockerfile b/Dockerfile
index b5e3fc8..df3e2d0 100644
--- a/Dockerfile
+++ b/Dockerfile
@@ -6,8 +6,4 @@ COPY . .
 
 RUN uv sync --locked
 
-ENV VIRTUAL_ENV=/app/.venv
-ENV PATH="$VIRTUAL_ENV/bin:$PATH"
-ENV FASTMCP_LOG_LEVEL=DEBUG
-
-CMD ["mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"]
+CMD ["uv", "run", "mcp", "run", "--transport", "sse", "nextcloud_mcp_server/server.py:mcp"]
diff --git a/nextcloud_mcp_server/client.py b/nextcloud_mcp_server/client.py
index d891a31..916ac2f 100644
--- a/nextcloud_mcp_server/client.py
+++ b/nextcloud_mcp_server/client.py
@@ -1,12 +1,9 @@
 import os
-import time  # Import time for sleep
 import mimetypes
-from io import BytesIO
 from httpx import (
     Client,
     Auth,
     BasicAuth,
-    Headers,
     Request,
     Response,
     HTTPStatusError,
diff --git a/nextcloud_mcp_server/config.py b/nextcloud_mcp_server/config.py
index 44e5ce1..5f5e978 100644
--- a/nextcloud_mcp_server/config.py
+++ b/nextcloud_mcp_server/config.py
@@ -4,11 +4,8 @@ LOGGING_CONFIG = {
     "version": 1,
     "handlers": {
         "default": {
-            "class": "logging.FileHandler",
+            "class": "logging.StreamHandler",
             "formatter": "http",
-            # "stream": "ext://sys.stderr"
-            "filename": "/tmp/nextcloud-mcp-server.log",
-            "mode": "a",
         }
     },
     "formatters": {
diff --git a/nextcloud_mcp_server/server.py b/nextcloud_mcp_server/server.py
index 1f0457c..abbc4ae 100644
--- a/nextcloud_mcp_server/server.py
+++ b/nextcloud_mcp_server/server.py
@@ -11,8 +11,6 @@ import asyncio  # Import asyncio
 
 setup_logging()
 
-logger = logging.getLogger(__name__)
-
 
 @dataclass
 class AppContext:
@@ -23,11 +21,9 @@ class AppContext:
 async def app_lifespan(server: FastMCP) -> AsyncIterator[AppContext]:
     """Manage application lifecycle with type-safe context"""
     # Initialize on startup
-    logger.info("Creating Nextcloud client")
+    logging.info("Creating Nextcloud client")
     client = NextcloudClient.from_env()
-    # Add a small delay to allow client initialization to complete
-    logger.info("Waiting 2 seconds for client initialization...")
-    logger.info("Client initialization wait complete.")
+    logging.info("Client initialization wait complete.")
     try:
         yield AppContext(client=client)
     finally:
@@ -38,6 +34,8 @@ async def app_lifespan(server: FastMCP) -> AsyncIterator[AppContext]:
 # Create an MCP server
 mcp = FastMCP("Nextcloud MCP", lifespan=app_lifespan)
 
+logger = logging.getLogger(__name__)
+
 
 @mcp.resource("nc://capabilities")
 def nc_get_capabilities():