femsci/forgejo
femsci/forgejo
1
0
Fork 0
forked from NYANDEV/forgejo
Code Pull requests Activity

Return 404 when user is not found on avatar (#21476)

Browse source 
- Instead of returning a 500 Internal Server when the user wasn't found,
return 404 Not found
This commit is contained in:
Gusted 2022-10-16 18:48:17 +00:00 committed by GitHub
parent 11d3677818
commit 5d3dbffa15
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 4 additions and 0 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
routers/web/user/avatar.go
View file

@ -31,6 +31,10 @@ func AvatarByUserName(ctx *context.Context) {
if strings.ToLower(userName) != "ghost" { if strings.ToLower(userName) != "ghost" {
var err error var err error
if user, err = user_model.GetUserByName(ctx, userName); err != nil { if user, err = user_model.GetUserByName(ctx, userName); err != nil {
if user_model.IsErrUserNotExist(err) {
ctx.NotFound("GetUserByName", err)
return
}
ctx.ServerError("Invalid user: "+userName, err) ctx.ServerError("Invalid user: "+userName, err)
return return
} }