diff options
-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." }, |