From d76d75ac1afad61f329fc2a716c43d6b06854a8b Mon Sep 17 00:00:00 2001 From: gauravsingh001-cyber Date: Mon, 30 Mar 2026 00:46:45 +0530 Subject: [PATCH] fix: use updater for username update instead of direct DB call --- .../app/lib/server/functions/setUsername.ts | 19 ++++++++++++++++--- 1 file changed, 16 insertions(+), 3 deletions(-) diff --git a/apps/meteor/app/lib/server/functions/setUsername.ts b/apps/meteor/app/lib/server/functions/setUsername.ts index 9eeff3a14a0e3..161708c1ea5b0 100644 --- a/apps/meteor/app/lib/server/functions/setUsername.ts +++ b/apps/meteor/app/lib/server/functions/setUsername.ts @@ -129,9 +129,22 @@ export const _setUsername = async function ( }, session); } // Set new username* - // TODO: use updater for setting the username and handle possible side effects in addUserToRoom - await Users.setUsername(user._id, username, { session }); - user.username = username; + +if (updater) { + await updater.updateOne( + { _id: user._id }, + { $set: { username } } + ); +} else { + + await Users.updateOne( + { _id: user._id }, + { $set: { username } }, + { session } + ); +} + +user.username = username; if (!previousUsername && settings.get('Accounts_SetDefaultAvatar') === true) { const avatarSuggestions = await getAvatarSuggestionForUser(user);