From 3461a7b20704464ee3c19a3dd31805e4c5c1fc4f Mon Sep 17 00:00:00 2001 From: Christopher Speller Date: Thu, 12 Oct 2017 08:00:53 -0700 Subject: Add back consumeAndClose functionality. (#7608) * consume bodies for action button integrations, webrtc gateway, oauth endpoint * Fixing a couple more places, switching to io.Copy to ioutil.Discard, adding a comment to help prevent future performance regressions --- app/webrtc.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'app/webrtc.go') diff --git a/app/webrtc.go b/app/webrtc.go index 65bbac7cd..2d0bb0708 100644 --- a/app/webrtc.go +++ b/app/webrtc.go @@ -62,7 +62,7 @@ func GetWebrtcToken(sessionId string) (string, *model.AppError) { if rp, err := utils.HttpClient(true).Do(rq); err != nil { return "", model.NewAppError("WebRTC.Token", "model.client.connecting.app_error", nil, err.Error(), http.StatusInternalServerError) } else if rp.StatusCode >= 300 { - defer rp.Body.Close() + defer consumeAndClose(rp) return "", model.AppErrorFromJson(rp.Body) } else { janusResponse := model.GatewayResponseFromJson(rp.Body) -- cgit v1.2.3-1-g7c22