From 77bd54154638423139dc2097043c297fcc508b9a Mon Sep 17 00:00:00 2001 From: Amy Gale Ruth Bowersox Date: Sat, 7 Feb 2026 22:07:03 -0700 Subject: [PATCH] Fixed invalid return in AmGetUserByName when user does not exist. Closes issue #1. --- database/user.go | 2 ++ 1 file changed, 2 insertions(+) diff --git a/database/user.go b/database/user.go index d0644a4..bde09e3 100644 --- a/database/user.go +++ b/database/user.go @@ -443,6 +443,8 @@ func AmGetUserByName(ctx context.Context, name string, tx *sqlx.Tx) (*User, erro } if len(dbdata) > 1 { return nil, fmt.Errorf("AmGetUserByName(\"%s\"): too many responses(%d)", name, len(dbdata)) + } else if len(dbdata) == 0 { + return nil, errors.New("user not found") } getUserMutex.Lock() rc, ok := userCache.Get(dbdata[0].Uid)