Skip to content

Commit e463798

Browse files
committed
Adjust logging level for frequent / low-level events
1 parent 609cdc7 commit e463798

File tree

2 files changed

+16
-16
lines changed

2 files changed

+16
-16
lines changed

Sources/MCP/Base/Transports/HTTPClientTransport.swift

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -131,7 +131,7 @@ public actor HTTPClientTransport: Transport {
131131
if let continuation = self.initialSessionIDContinuation {
132132
continuation.resume()
133133
self.initialSessionIDContinuation = nil // Consume the continuation
134-
logger.debug("Initial session ID signal triggered for SSE task.")
134+
logger.trace("Initial session ID signal triggered for SSE task.")
135135
}
136136
}
137137

@@ -251,7 +251,7 @@ public actor HTTPClientTransport: Transport {
251251
logger.warning("SSE responses aren't fully supported on Linux")
252252
messageContinuation.yield(data)
253253
} else if contentType.contains("application/json") {
254-
logger.debug("Received JSON response", metadata: ["size": "\(data.count)"])
254+
logger.trace("Received JSON response", metadata: ["size": "\(data.count)"])
255255
messageContinuation.yield(data)
256256
} else {
257257
logger.warning("Unexpected content type: \(contentType)")
@@ -285,15 +285,15 @@ public actor HTTPClientTransport: Transport {
285285

286286
if contentType.contains("text/event-stream") {
287287
// For SSE, processing happens via the stream
288-
logger.debug("Received SSE response, processing in streaming task")
288+
logger.trace("Received SSE response, processing in streaming task")
289289
try await self.processSSE(stream)
290290
} else if contentType.contains("application/json") {
291291
// For JSON responses, collect and deliver the data
292292
var buffer = Data()
293293
for try await byte in stream {
294294
buffer.append(byte)
295295
}
296-
logger.debug("Received JSON response", metadata: ["size": "\(buffer.count)"])
296+
logger.trace("Received JSON response", metadata: ["size": "\(buffer.count)"])
297297
messageContinuation.yield(buffer)
298298
} else {
299299
logger.warning("Unexpected content type: \(contentType)")
@@ -390,7 +390,7 @@ public actor HTTPClientTransport: Transport {
390390

391391
// Wait for the initial session ID signal, but only if sessionID isn't already set
392392
if self.sessionID == nil, let signalTask = self.initialSessionIDSignalTask {
393-
logger.debug("SSE streaming task waiting for initial sessionID signal...")
393+
logger.trace("SSE streaming task waiting for initial sessionID signal...")
394394

395395
// Race the signalTask against a timeout
396396
let timeoutTask = Task {
@@ -429,16 +429,16 @@ public actor HTTPClientTransport: Transport {
429429
timeoutTask.cancel()
430430

431431
if signalReceived {
432-
logger.debug("SSE streaming task proceeding after initial sessionID signal.")
432+
logger.trace("SSE streaming task proceeding after initial sessionID signal.")
433433
} else {
434434
logger.warning(
435435
"Timeout waiting for initial sessionID signal. SSE stream will proceed (sessionID might be nil)."
436436
)
437437
}
438438
} else if self.sessionID != nil {
439-
logger.debug(
440-
"Initial sessionID already available. Proceeding with SSE streaming task immediately."
441-
)
439+
logger.trace(
440+
"Initial sessionID already available. Proceeding with SSE streaming task immediately."
441+
)
442442
} else {
443443
logger.info(
444444
"Proceeding with SSE connection attempt; sessionID is nil. This might be expected for stateless servers or if initialize hasn't provided one yet."
@@ -525,7 +525,7 @@ public actor HTTPClientTransport: Transport {
525525
// Check if task has been cancelled
526526
if Task.isCancelled { break }
527527

528-
logger.debug(
528+
logger.trace(
529529
"SSE event received",
530530
metadata: [
531531
"type": "\(event.event ?? "message")",

Sources/MCP/Base/Transports/NetworkTransport.swift

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -309,11 +309,11 @@ import Logging
309309
case .cancelled:
310310
await self.handleConnectionCancelled(continuation: continuation)
311311
case .waiting(let error):
312-
self.logger.debug("Connection waiting: \(error)")
312+
self.logger.trace("Connection waiting: \(error)")
313313
case .preparing:
314-
self.logger.debug("Connection preparing...")
314+
self.logger.trace("Connection preparing...")
315315
case .setup:
316-
self.logger.debug("Connection setup...")
316+
self.logger.trace("Connection setup...")
317317
@unknown default:
318318
self.logger.warning("Unknown connection state")
319319
}
@@ -408,7 +408,7 @@ import Logging
408408
})
409409
}
410410

411-
logger.debug("Heartbeat sent")
411+
logger.trace("Heartbeat sent")
412412
}
413413

414414
/// Handles connection failure
@@ -638,11 +638,11 @@ import Logging
638638

639639
// Check if this is a heartbeat message
640640
if Heartbeat.isHeartbeat(newData) {
641-
logger.debug("Received heartbeat from peer")
641+
logger.trace("Received heartbeat from peer")
642642

643643
// Extract timestamp if available
644644
if let heartbeat = Heartbeat.from(data: newData) {
645-
logger.debug("Heartbeat timestamp: \(heartbeat.timestamp)")
645+
logger.trace("Heartbeat timestamp: \(heartbeat.timestamp)")
646646
}
647647

648648
// Reset the counter since we got valid data

0 commit comments

Comments
 (0)