Update app.py

app.py

@@ -3,12 +3,27 @@ import httpx
 from fastapi import FastAPI, Request, HTTPException
 from fastmcp import FastMCP
 
-# Create the MCP server (FastMCP creates its own FastAPI app internally)
+# Create separate FastAPI app for our endpoints
+app = FastAPI()
+
+# Create MCP server
 mcp = FastMCP(name="SAP MCP Proxy", version="0.1")
 
 API_GATEWAY_KEY = os.getenv("API_GATEWAY_KEY")
 
-# Define the MCP tool
+# Authentication middleware
+@app.middleware("http")
+async def check_api_key(request: Request, call_next):
+    # Allow unauthenticated access to health check and docs
+    if request.url.path in ["/", "/docs", "/redoc", "/routes"]:
+        return await call_next(request)
+    token = request.headers.get("X-API-Key")
+    if not token or token != API_GATEWAY_KEY:
+        raise HTTPException(status_code=401, detail="Invalid or missing API key")
+    response = await call_next(request)
+    return response
+
+# Define MCP tool
 @mcp.tool(description="Fetch 50 supplier invoices from SAP S/4HANA OData API")
 async def get_supplier_invoices():
     api_key = os.getenv("API_KEY")
@@ -27,33 +42,12 @@ async def get_supplier_invoices():
         else:
             return {"error": resp.status_code, "body": resp.text}
 
-# Create HTTP ASGI app from MCP server for mounting
-mcp_app = mcp.http_app(path='/mcp')
-
-# Create a new FastAPI app and combine routes
-app = FastAPI(
-    title="SAP MCP Proxy",
-    routes=[*mcp_app.routes],  # Include MCP routes
-    lifespan=mcp_app.lifespan  # Important: pass lifespan for MCP initialization
-)
-
-# Add custom middleware for authentication
-@app.middleware("http")
-async def check_api_key(request: Request, call_next):
-    if request.url.path in ["/", "/docs", "/redoc", "/routes"]:
-        return await call_next(request)
-    token = request.headers.get("X-API-Key")
-    if not token or token != API_GATEWAY_KEY:
-        raise HTTPException(status_code=401, detail="Invalid or missing API key")
-    response = await call_next(request)
-    return response
-
 # Health check endpoint
 @app.get("/")
 def root():
     return {"message": "SAP MCP Proxy is running"}
 
-# Debug endpoint to list routes
+# Debug endpoint
 @app.get("/routes")
 def get_routes():
     return [route.path for route in app.routes]