FIX: http client read response header failed should close socket

Coldwings · Coldwings · commit 9c9f8333ac32 · 2025-04-28T16:43:32.000+08:00
diff --git a/net/http/client.cpp b/net/http/client.cpp
@@ -263,6 +263,7 @@ class ClientImpl : public Client {
             resp.reset((char *)buf, kMinimalHeadersSize, true, sock.release(), true, req.verb());
         }
         if (op->resp.receive_header(tmo.timeout()) != 0) {
+            sock->close();
             req.reset_status();
             LOG_ERROR_RETURN(0, ROUNDTRIP_NEED_RETRY, "read response header failed");
         }

Original file line number	Diff line number	Diff line change
`@@ -263,6 +263,7 @@ class ClientImpl : public Client {`
`263`	`263`	`resp.reset((char *)buf, kMinimalHeadersSize, true, sock.release(), true, req.verb());`
`264`	`264`	`}`
`265`	`265`	`if (op->resp.receive_header(tmo.timeout()) != 0) {`
	`266`	`+ sock->close();`
`266`	`267`	`req.reset_status();`
`267`	`268`	`LOG_ERROR_RETURN(0, ROUNDTRIP_NEED_RETRY, "read response header failed");`
`268`	`269`	`}`