Skip to content

Commit baa1cdb

Browse files
KN4CK3Rzeripath
authored andcommitted
Redirect if user does not exist on admin pages (go-gitea#20981)
Backport go-gitea#20981 When on /admin/users/ endpoints if the user is no longer in the DB, redirect instead of causing a http 500.
1 parent 0ecbb71 commit baa1cdb

File tree

1 file changed

+5
-1
lines changed

1 file changed

+5
-1
lines changed

routers/web/admin/users.go

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -209,7 +209,11 @@ func NewUserPost(ctx *context.Context) {
209209
func prepareUserInfo(ctx *context.Context) *user_model.User {
210210
u, err := user_model.GetUserByID(ctx.ParamsInt64(":userid"))
211211
if err != nil {
212-
ctx.ServerError("GetUserByID", err)
212+
if user_model.IsErrUserNotExist(err) {
213+
ctx.Redirect(setting.AppSubURL + "/admin/users")
214+
} else {
215+
ctx.ServerError("GetUserByID", err)
216+
}
213217
return nil
214218
}
215219
ctx.Data["User"] = u

0 commit comments

Comments
 (0)