diff options
author | Joram Wilander <jwawilander@gmail.com> | 2018-04-13 10:57:22 -0400 |
---|---|---|
committer | Christopher Speller <crspeller@gmail.com> | 2018-04-13 07:57:22 -0700 |
commit | 21b1cd44e09c41ea581ee05de46db5e5f3fa1696 (patch) | |
tree | c939c4f6de8f72d6a2b7936b097163b273088921 | |
parent | ae5e324be8078927bf088bc9dae37189a6ecea6f (diff) | |
download | chat-21b1cd44e09c41ea581ee05de46db5e5f3fa1696.tar.gz chat-21b1cd44e09c41ea581ee05de46db5e5f3fa1696.tar.bz2 chat-21b1cd44e09c41ea581ee05de46db5e5f3fa1696.zip |
Fix error handling when requesting profile image for bad user ID (#8618)
-rw-r--r-- | api4/user.go | 3 | ||||
-rw-r--r-- | api4/user_test.go | 3 | ||||
-rw-r--r-- | i18n/en.json | 4 |
3 files changed, 9 insertions, 1 deletions
diff --git a/api4/user.go b/api4/user.go index 20b035f1d..9aa709db5 100644 --- a/api4/user.go +++ b/api4/user.go @@ -199,7 +199,8 @@ func getProfileImage(c *Context, w http.ResponseWriter, r *http.Request) { return } else { if len(users) == 0 { - c.Err = err + c.Err = model.NewAppError("getProfileImage", "api.user.get_profile_image.not_found.app_error", nil, "", http.StatusNotFound) + return } user := users[0] diff --git a/api4/user_test.go b/api4/user_test.go index 359756aeb..27219726b 100644 --- a/api4/user_test.go +++ b/api4/user_test.go @@ -834,6 +834,9 @@ func TestGetProfileImage(t *testing.T) { _, resp = Client.GetProfileImage("junk", "") CheckBadRequestStatus(t, resp) + _, resp = Client.GetProfileImage(model.NewId(), "") + CheckNotFoundStatus(t, resp) + Client.Logout() _, resp = Client.GetProfileImage(user.Id, "") CheckUnauthorizedStatus(t, resp) diff --git a/i18n/en.json b/i18n/en.json index 7abe67567..02ac46e70 100644 --- a/i18n/en.json +++ b/i18n/en.json @@ -3095,6 +3095,10 @@ "translation": "Could not parse multipart form" }, { + "id": "api.user.get_profile_image.not_found.app_error", + "translation": "Unable to get profile image, user not found." + }, + { "id": "api.user.upload_profile_user.storage.app_error", "translation": "Unable to upload file. Image storage is not configured." }, |