summaryrefslogtreecommitdiffstats
path: root/api/post.go
diff options
context:
space:
mode:
authorJoram Wilander <jwawilander@gmail.com>2016-06-02 15:40:34 -0400
committerCorey Hulen <corey@hulen.com>2016-06-02 12:40:34 -0700
commit0788e836475722123c36835489396d65c6f9b252 (patch)
treeff18b2e5042e3023f87eb3efffd8b8e7f8f2e75e /api/post.go
parent848a8663ed7f856aee3f801a62b82e87b20de0ea (diff)
downloadchat-0788e836475722123c36835489396d65c6f9b252.tar.gz
chat-0788e836475722123c36835489396d65c6f9b252.tar.bz2
chat-0788e836475722123c36835489396d65c6f9b252.zip
Close HTTP response bodies properly in golang driver (and some api functions) (#3217)
Diffstat (limited to 'api/post.go')
-rw-r--r--api/post.go5
1 files changed, 5 insertions, 0 deletions
diff --git a/api/post.go b/api/post.go
index 875f30ba5..831591784 100644
--- a/api/post.go
+++ b/api/post.go
@@ -8,6 +8,7 @@ import (
"fmt"
"html/template"
"io"
+ "io/ioutil"
"net/http"
"net/url"
"path/filepath"
@@ -428,6 +429,10 @@ func handleWebhookEvents(c *Context, post *model.Post, team *model.Team, channel
if resp, err := client.Do(req); err != nil {
l4g.Error(utils.T("api.post.handle_webhook_events_and_forget.event_post.error"), err.Error())
} else {
+ defer func() {
+ ioutil.ReadAll(resp.Body)
+ resp.Body.Close()
+ }()
respProps := model.MapFromJson(resp.Body)
// copy the context and create a mock session for posting the message