Skip to content

Commit 54dad60

Browse files
Copilotstephentoub
andcommitted
Apply code review feedback: inline duration calculation and remove logging test
Co-authored-by: stephentoub <[email protected]>
1 parent b9c408e commit 54dad60

File tree

2 files changed

+1
-123
lines changed

2 files changed

+1
-123
lines changed

src/ModelContextProtocol.Core/McpSessionHandler.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -357,8 +357,7 @@ private void HandleMessageWithId(JsonRpcMessage message, JsonRpcMessageWithId me
357357
LogRequestHandlerCalled(EndpointName, request.Method);
358358
long startingTimestamp = Stopwatch.GetTimestamp();
359359
JsonNode? result = await handler(request, cancellationToken).ConfigureAwait(false);
360-
TimeSpan elapsed = GetElapsed(startingTimestamp);
361-
LogRequestHandlerCompleted(EndpointName, request.Method, elapsed.TotalMilliseconds);
360+
LogRequestHandlerCompleted(EndpointName, request.Method, GetElapsed(startingTimestamp).TotalMilliseconds);
362361

363362
await SendMessageAsync(new JsonRpcResponse
364363
{

tests/ModelContextProtocol.Tests/Server/McpServerRequestDurationLoggingTests.cs

Lines changed: 0 additions & 121 deletions
This file was deleted.

0 commit comments

Comments
 (0)