From 597a33674dacd0148e81ec24c0dceb7c52f9f872 Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 26 Apr 2026 22:29:41 +0000 Subject: [PATCH 1/3] Initial plan From e3aaa30cf49bdbc03990cdf01e5a9e8a23c47fad Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Sun, 26 Apr 2026 22:35:03 +0000 Subject: [PATCH 2/3] fix: remove duplicate HTTP request/response log statements from sendHTTPRequest Agent-Logs-Url: https://github.com/github/gh-aw-mcpg/sessions/496397d9-38d0-4f5d-8b3d-5cfbf9a58d3c Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com> --- internal/mcp/http_transport.go | 3 --- 1 file changed, 3 deletions(-) diff --git a/internal/mcp/http_transport.go b/internal/mcp/http_transport.go index 5a4cd6bd8..f6ac08265 100644 --- a/internal/mcp/http_transport.go +++ b/internal/mcp/http_transport.go @@ -643,15 +643,12 @@ func (c *Connection) sendHTTPRequest(ctx context.Context, method string, params headerModifier := c.buildSessionHeaderModifier(ctx) requestID := atomic.AddUint64(&requestIDCounter, 1) - logConn.Printf("Sending HTTP request to %s: method=%s, id=%d", c.httpURL, method, requestID) result, err := c.executeHTTPRequest(ctx, method, params, requestID, headerModifier) if err != nil { return nil, err } - logConn.Printf("Received HTTP response: status=%d, body_len=%d", result.StatusCode, len(result.ResponseBody)) - // If the backend reported that the session has expired, reconnect and retry once. if isSessionNotFoundHTTPResponse(result.StatusCode, result.ResponseBody) { logConn.Printf("Session not found from %s (serverID=%s), attempting reconnect", c.httpURL, c.serverID) From 6b00f8f8355abe0018bd47ca38ace16641dff4eb Mon Sep 17 00:00:00 2001 From: "copilot-swe-agent[bot]" <198982749+Copilot@users.noreply.github.com> Date: Mon, 27 Apr 2026 00:35:05 +0000 Subject: [PATCH 3/3] fix: guard logConn send/receive lines with !logHTTP.Enabled() to avoid duplicates Agent-Logs-Url: https://github.com/github/gh-aw-mcpg/sessions/cc765c0e-bb93-4fa3-b128-7ea8cf2150f2 Co-authored-by: lpcox <15877973+lpcox@users.noreply.github.com> --- internal/mcp/http_transport.go | 7 +++++++ 1 file changed, 7 insertions(+) diff --git a/internal/mcp/http_transport.go b/internal/mcp/http_transport.go index f6ac08265..7d7c0ad6c 100644 --- a/internal/mcp/http_transport.go +++ b/internal/mcp/http_transport.go @@ -643,12 +643,19 @@ func (c *Connection) sendHTTPRequest(ctx context.Context, method string, params headerModifier := c.buildSessionHeaderModifier(ctx) requestID := atomic.AddUint64(&requestIDCounter, 1) + if !logHTTP.Enabled() { + logConn.Printf("Sending HTTP request to %s: method=%s, id=%d", c.httpURL, method, requestID) + } result, err := c.executeHTTPRequest(ctx, method, params, requestID, headerModifier) if err != nil { return nil, err } + if !logHTTP.Enabled() { + logConn.Printf("Received HTTP response: status=%d, body_len=%d", result.StatusCode, len(result.ResponseBody)) + } + // If the backend reported that the session has expired, reconnect and retry once. if isSessionNotFoundHTTPResponse(result.StatusCode, result.ResponseBody) { logConn.Printf("Session not found from %s (serverID=%s), attempting reconnect", c.httpURL, c.serverID)