fix: http response body not close on status >= 400 (close #5163)

AlistGo · Sep 5, 2023 · f2f312b · f2f312b
1 parent 6f6d20e
commit f2f312b
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/internal/net/serve.go b/internal/net/serve.go
@@ -224,9 +224,10 @@ func RequestHttp(ctx context.Context, httpMethod string, headerOverride http.Hea
 	res.Header.Del("set-cookie")
 	if res.StatusCode >= 400 {
 		all, _ := io.ReadAll(res.Body)
+		_ = res.Body.Close()
 		msg := string(all)
 		log.Debugln(msg)
-		return res, errors.New(msg)
+		return nil, fmt.Errorf("http request [%s] failure,status: %d response:%s", URL, res.StatusCode, msg)
 	}
 	return res, nil
 }