Merge pull request #1175 from chienfuchen32/patch-1

nickysemenza · web-flow · commit 2215630d72a3 · 2021-02-24T12:02:09.000-08:00
Add http response close in client post function
diff --git a/api/client/client.go b/api/client/client.go
@@ -157,6 +157,7 @@ func (srv *server) post(url string, jsonData []byte) (*api.Response, error) {
 		return nil, errors.Wrap(errors.APIClientError, errors.ClientHTTPError, err)
 	}
 	defer req.Body.Close()
+	defer resp.Body.Close()
 	body, err := ioutil.ReadAll(resp.Body)
 	if err != nil {
 		return nil, errors.Wrap(errors.APIClientError, errors.IOError, err)

Original file line number	Diff line number	Diff line change
`@@ -157,6 +157,7 @@ func (srv server) post(url string, jsonData []byte) (api.Response, error) {`
`157`	`157`	`return nil, errors.Wrap(errors.APIClientError, errors.ClientHTTPError, err)`
`158`	`158`	`}`
`159`	`159`	`defer req.Body.Close()`
	`160`	`+ defer resp.Body.Close()`
`160`	`161`	`body, err := ioutil.ReadAll(resp.Body)`
`161`	`162`	`if err != nil {`
`162`	`163`	`return nil, errors.Wrap(errors.APIClientError, errors.IOError, err)`