From 321433420a4859a94d30ed3c6bc8149fc3cdd822 Mon Sep 17 00:00:00 2001 From: Sainan <63328889+Sainan@users.noreply.github.com> Date: Tue, 22 Jul 2025 17:30:37 +0200 Subject: [PATCH 1/2] fix: getUsernameFromEmail returning wrong value --- src/services/loginService.ts | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/services/loginService.ts b/src/services/loginService.ts index 338f43db..e929fce3 100644 --- a/src/services/loginService.ts +++ b/src/services/loginService.ts @@ -32,7 +32,7 @@ export const getUsernameFromEmail = async (email: string): Promise => { name = nameFromEmail + suffix; } while (await isNameTaken(name)); } - return nameFromEmail; + return name; }; export const createAccount = async (accountData: IDatabaseAccountRequiredFields): Promise => { -- 2.47.2 From f9bbb17f007f9538a6c5a58df39eb40d856f37af Mon Sep 17 00:00:00 2001 From: Sainan <63328889+Sainan@users.noreply.github.com> Date: Tue, 22 Jul 2025 17:30:52 +0200 Subject: [PATCH 2/2] Revert "fix(webui): handle name already being taken (#2530)" This reverts commit 304af514e28e4d6395a6fa6ac83ba2d49c54d5eb. --- src/services/wsService.ts | 26 +++++++++----------------- 1 file changed, 9 insertions(+), 17 deletions(-) diff --git a/src/services/wsService.ts b/src/services/wsService.ts index 01ebe7aa..fb4bbee3 100644 --- a/src/services/wsService.ts +++ b/src/services/wsService.ts @@ -2,13 +2,7 @@ import http from "http"; import https from "https"; import ws from "ws"; import { Account } from "@/src/models/loginModel"; -import { - createAccount, - createNonce, - getUsernameFromEmail, - isCorrectPassword, - isNameTaken -} from "@/src/services/loginService"; +import { createAccount, createNonce, getUsernameFromEmail, isCorrectPassword } from "@/src/services/loginService"; import { IDatabaseAccountJson } from "@/src/types/loginTypes"; import { HydratedDocument } from "mongoose"; import { logError } from "@/src/utils/logger"; @@ -111,16 +105,14 @@ const wsOnConnect = (ws: ws, req: http.IncomingMessage): void => { } } else if (data.auth.isRegister) { const name = await getUsernameFromEmail(data.auth.email); - if (!(await isNameTaken(name))) { - account = await createAccount({ - email: data.auth.email, - password: data.auth.password, - ClientType: "webui", - LastLogin: new Date(), - DisplayName: name, - Nonce: createNonce() - }); - } + account = await createAccount({ + email: data.auth.email, + password: data.auth.password, + ClientType: "webui", + LastLogin: new Date(), + DisplayName: name, + Nonce: createNonce() + }); } if (account) { (ws as IWsCustomData).accountId = account.id; -- 2.47.2