close connection after logging

Signed-off-by: Joey Muia <[email protected]>
envoyproxy · May 29, 2020 · a81e1b9 · a81e1b9
1 parent 2a28a40
commit a81e1b9
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/source/common/upstream/health_checker_impl.cc b/source/common/upstream/health_checker_impl.cc
@@ -711,14 +711,14 @@ void GrpcHealthCheckerImpl::GrpcActiveHealthCheckSession::onResetStream(Http::St
     return;
   }
 
+  ENVOY_CONN_LOG(debug, "connection/stream error health_flags={}", *client_,
+                 HostUtility::healthFlagsToString(*host_));
+
   if (!parent_.reuse_connection_) {
     // Stream reset was unexpected, so we haven't closed the connection yet.
     client_->close();
   }
 
-  ENVOY_CONN_LOG(debug, "connection/stream error health_flags={}", *client_,
-                 HostUtility::healthFlagsToString(*host_));
-
   // TODO(baranov1ch): according to all HTTP standards, we should check if reason is one of
   // Http::StreamResetReason::RemoteRefusedStreamReset (which may mean GOAWAY),
   // Http::StreamResetReason::RemoteReset or Http::StreamResetReason::ConnectionTermination (both