From b367b1ff4064557deacae0f7ecf77fae4624b8c6 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jes=C3=BAs=20Espino?= Date: Wed, 18 Jul 2018 10:07:00 +0200 Subject: Do not close the connection abruptly on too big file uploads (#9083) --- api4/brand.go | 4 ++++ 1 file changed, 4 insertions(+) (limited to 'api4/brand.go') diff --git a/api4/brand.go b/api4/brand.go index f182e1070..2e3081353 100644 --- a/api4/brand.go +++ b/api4/brand.go @@ -4,6 +4,8 @@ package api4 import ( + "io" + "io/ioutil" "net/http" "github.com/mattermost/mattermost-server/model" @@ -27,6 +29,8 @@ func getBrandImage(c *Context, w http.ResponseWriter, r *http.Request) { } func uploadBrandImage(c *Context, w http.ResponseWriter, r *http.Request) { + defer io.Copy(ioutil.Discard, r.Body) + if r.ContentLength > *c.App.Config().FileSettings.MaxFileSize { c.Err = model.NewAppError("uploadBrandImage", "api.admin.upload_brand_image.too_large.app_error", nil, "", http.StatusRequestEntityTooLarge) return -- cgit v1.2.3-1-g7c22