Skip to content

Commit a21cadb

Browse files
committed
addresses Pamela and copilot feedback
1 parent 0f6c710 commit a21cadb

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

pyproject.toml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,3 +34,4 @@ line-length = 120
3434
target-version = "py310"
3535
lint.select = ["E", "F", "I", "UP"]
3636
lint.ignore = ["D203"]
37+
lint.isort.known-first-party = ["opentelemetry_middleware"]

servers/basic_mcp_http.py

Lines changed: 4 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -8,18 +8,17 @@
88

99
from dotenv import load_dotenv
1010
from fastmcp import FastMCP
11+
from fastmcp.server.middleware import Middleware
12+
1113
from opentelemetry_middleware import OpenTelemetryMiddleware, configure_aspire_dashboard
1214

1315
load_dotenv(override=True)
1416

1517
logging.basicConfig(level=logging.INFO, format="%(asctime)s - %(message)s")
1618
logger = logging.getLogger("ExpensesMCP")
1719

18-
19-
otel_endpoint = os.getenv("OTEL_EXPORTER_OTLP_ENDPOINT")
20-
middleware: list = []
21-
22-
if otel_endpoint:
20+
middleware: list[Middleware] = []
21+
if os.getenv("OTEL_EXPORTER_OTLP_ENDPOINT"):
2322
logger.info("Setting up Aspire Dashboard instrumentation (OTLP)")
2423
configure_aspire_dashboard(service_name="expenses-mcp")
2524
middleware = [OpenTelemetryMiddleware(tracer_name="expenses.mcp")]

0 commit comments

Comments
 (0)