Merge pull request #1 from RycCheen/master

add resp.Body.Close() to release response body resource
CatchZeng · May 12, 2020 · c10a8a5 · c10a8a5
2 parents 69c914d + 3940133
commit c10a8a5
Showing 1 changed file with 2 additions and 1 deletion.
diff --git a/client/client.go b/client/client.go
@@ -40,7 +40,7 @@ func (d *DingTalk) Send(message message.Message) (Response, error) {
 		return res, err
 	}
 
-	req, err := http.NewRequest("POST", pushURL, bytes.NewReader(reqBytes))
+	req, err := http.NewRequest(http.MethodPost, pushURL, bytes.NewReader(reqBytes))
 	if err != nil {
 		return res, err
 	}
@@ -53,6 +53,7 @@ func (d *DingTalk) Send(message message.Message) (Response, error) {
 	if err != nil {
 		return res, err
 	}
+	defer resp.Body.Close()
 
 	resultByte, err := ioutil.ReadAll(resp.Body)
 	if err != nil {