Remove redundant Warn log in HTTP codec. (#18699) (#18702)

This line was introduced in efcf2e5, but it is flooding the log outputs of Envoy users, so this PR just deletes the line. For detail, please refer to #18643. Fixes #18643 Signed-off-by: Abhay Narayan Katare <[email protected]> Co-authored-by: Takeshi Yoneda <[email protected]>
envoyproxy · Oct 29, 2021 · 9d581d2 · 9d581d2
1 parent 614ac4f
commit 9d581d2
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/docs/root/version_history/current.rst b/docs/root/version_history/current.rst
@@ -13,6 +13,7 @@ Bug Fixes
 ---------
 *Changes expected to improve the state of the world and are unlikely to have negative effects*
 
+* http: remove redundant Warn log in HTTP codec.
 * listener: fix a crash when updating any listener that does not bind to port.
 * listener: listener add can reuse the listener socket of a draining filter chain listener and fix the request lost.
 

diff --git a/source/common/http/codec_client.cc b/source/common/http/codec_client.cc
@@ -117,9 +117,6 @@ void CodecClient::onEvent(Network::ConnectionEvent event) {
               StreamInfo::ResponseFlag::UpstreamProtocolError);
         }
       }
-    } else {
-      ENVOY_CONN_LOG(warn, "Connection is closed by {} during connecting.", *connection_,
-                     (event == Network::ConnectionEvent::RemoteClose ? "peer" : "self"));
     }
     while (!active_requests_.empty()) {
       // Fake resetting all active streams so that reset() callbacks get invoked.