refactor: move token revoke call to other auth util functions

This commit is contained in:
Miodec 2024-05-28 19:02:08 +02:00
parent a279744ef3
commit 90c96a2bd7
2 changed files with 3 additions and 3 deletions

View file

@ -200,7 +200,6 @@ export async function deleteUser(
//delete user from
await AuthUtil.deleteUser(uid);
await AuthUtil.revokeTokensByUid(uid);
void Logger.logToDb(
"user_deleted",
@ -326,7 +325,6 @@ export async function updateEmail(
try {
await AuthUtil.updateUserEmail(uid, newEmail);
await UserDAL.updateEmail(uid, newEmail);
await AuthUtil.revokeTokensByUid(uid);
} catch (e) {
if (e.code === "auth/email-already-exists") {
throw new MonkeyError(
@ -367,7 +365,6 @@ export async function updatePassword(
const { newPassword } = req.body;
await AuthUtil.updateUserPassword(uid, newPassword);
await AuthUtil.revokeTokensByUid(uid);
return new MonkeyResponse("Password updated");
}

View file

@ -53,6 +53,7 @@ export async function updateUserEmail(
uid: string,
email: string
): Promise<UserRecord> {
await revokeTokensByUid(uid);
return await FirebaseAdmin().auth().updateUser(uid, {
email,
emailVerified: false,
@ -63,12 +64,14 @@ export async function updateUserPassword(
uid: string,
password: string
): Promise<UserRecord> {
await revokeTokensByUid(uid);
return await FirebaseAdmin().auth().updateUser(uid, {
password,
});
}
export async function deleteUser(uid: string): Promise<void> {
await revokeTokensByUid(uid);
await FirebaseAdmin().auth().deleteUser(uid);
}