server: fix bug in ListUsers to handle no users
This commit is contained in:
+1
-1
@@ -70,7 +70,7 @@ func dbGetAllUsers() ([]*User, error) {
|
|||||||
}
|
}
|
||||||
if len(users) == 0 {
|
if len(users) == 0 {
|
||||||
slog.Debug("db: no users found")
|
slog.Debug("db: no users found")
|
||||||
return nil, errors.New("no users found")
|
return nil, ErrUserNotFound
|
||||||
}
|
}
|
||||||
|
|
||||||
slog.Debug("db: user list returned")
|
slog.Debug("db: user list returned")
|
||||||
|
|||||||
@@ -52,8 +52,12 @@ func ListUsers(w http.ResponseWriter, r *http.Request) {
|
|||||||
slog.Debug("user: entering ListUsers handler")
|
slog.Debug("user: entering ListUsers handler")
|
||||||
dbUsers, err := dbGetAllUsers()
|
dbUsers, err := dbGetAllUsers()
|
||||||
if err != nil {
|
if err != nil {
|
||||||
|
if errors.Is(err, ErrUserNotFound) {
|
||||||
|
render.Render(w, r, ErrNotFound)
|
||||||
|
} else {
|
||||||
slog.Error("user: failed to fetch users", "error", err)
|
slog.Error("user: failed to fetch users", "error", err)
|
||||||
render.Render(w, r, ErrInternal(err))
|
render.Render(w, r, ErrInternal(err))
|
||||||
|
}
|
||||||
return
|
return
|
||||||
}
|
}
|
||||||
|
|
||||||
|
|||||||
Reference in New Issue
Block a user