summary refs log tree commit diff
path: root/src/util/entities/User.ts
diff options
context:
space:
mode:
authorPuyodead1 <puyodead@proton.me>2023-09-08 22:38:56 -0400
committerPuyodead1 <puyodead@protonmail.com>2023-12-23 16:38:16 -0500
commitd88e9273ffcc95477e560e7b1a3bd32011e24f6c (patch)
tree62d130f36ddf96ad980bac2f1a9d2a17154d65d3 /src/util/entities/User.ts
parentadd unique-username routes to unauthenticated list (diff)
downloadserver-d88e9273ffcc95477e560e7b1a3bd32011e24f6c.tar.xz
fix being able to register with taken username
Diffstat (limited to 'src/util/entities/User.ts')
-rw-r--r--src/util/entities/User.ts13
1 files changed, 8 insertions, 5 deletions
diff --git a/src/util/entities/User.ts b/src/util/entities/User.ts

index 5ec9862e..07b68b6d 100644 --- a/src/util/entities/User.ts +++ b/src/util/entities/User.ts
@@ -379,7 +379,7 @@ export class User extends BaseClass { if (uniqueUsernames) { // check if there is already an account with this username - if (!User.isUsernameAvailable(username)) + if (!(await User.isUsernameAvailable(username))) throw FieldErrors({ username: { code: "USERNAME_ALREADY_TAKEN", @@ -463,11 +463,14 @@ export class User extends BaseClass { } static async isUsernameAvailable(username: string) { - const user = await User.findOne({ - where: { username }, - select: ["id"], + // TODO: implement regex check? + const count = await User.count({ + where: { + username: username.toLowerCase(), + }, }); - return !user; + + return count === 0; } }