From e019402a9e848c770e4d5a36accbef6d60c8b6e4 Mon Sep 17 00:00:00 2001 From: Dylan Hurd Date: Tue, 30 Jun 2026 16:40:21 -0700 Subject: [PATCH] [codex] Backport websocket trace fix to release/0.142 (#30771) ## Summary Backports #30757 to `release/0.142`. Removes the trace-level log that emitted the full Responses WebSocket request text. ## Why The full request payload was still being logged by this trace statement and was not covered by the existing request-log filtering. ## Impact Prevents full WebSocket request contents from being written to traces on the 0.142 release branch. This does not change request behavior or the app-server API. ## Testing - `just fmt` - `just test -p codex-api` (130 passed) Cherry-picked from `db887d03e1f907467e33271572dffb73bceecd6b` with `-x`. --- codex-rs/codex-api/src/endpoint/responses_websocket.rs | 3 --- 1 file changed, 3 deletions(-) diff --git a/codex-rs/codex-api/src/endpoint/responses_websocket.rs b/codex-rs/codex-api/src/endpoint/responses_websocket.rs index ea7565ef57..03f3befd9e 100644 --- a/codex-rs/codex-api/src/endpoint/responses_websocket.rs +++ b/codex-rs/codex-api/src/endpoint/responses_websocket.rs @@ -43,7 +43,6 @@ use tracing::debug; use tracing::error; use tracing::info; use tracing::instrument; -use tracing::trace; use tungstenite::extensions::ExtensionsConfig; use tungstenite::extensions::compression::deflate::DeflateConfig; use tungstenite::protocol::WebSocketConfig; @@ -782,8 +781,6 @@ async fn send_websocket_request( telemetry: Option<&Arc>, connection_reused: bool, ) -> Result<(), ApiError> { - trace!("websocket request: {request_text}"); - let request_start = Instant::now(); let result = tokio::time::timeout( idle_timeout,