Skip to content

Commit 444f007

Browse files
committed
Fix: auth deletion
1 parent a041a0d commit 444f007

File tree

1 file changed

+3
-20
lines changed

1 file changed

+3
-20
lines changed

server/routes/users.ts

Lines changed: 3 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -68,33 +68,16 @@ usersRouter.delete("/:firebaseUid", async (req, res) => {
6868
usersRouter.delete("/email/:email", async (req, res) => {
6969
try {
7070
const { email } = req.params;
71-
console.log("email", email)
72-
73-
// try {
74-
// const userRecord = await admin.auth().getUserByEmail(email);
75-
// console.log("userRecord", userRecord)
76-
// const uid = userRecord.uid;
77-
// console.log("uid", uid)
78-
// await admin.auth().deleteUser(uid);
79-
// } catch (firebaseError) {
80-
// // If user doesn't exist in Firebase, that's okay - they might be a placeholder user
81-
// console.log(`Firebase Error: ${firebaseError}`);
82-
// }
8371

84-
const user = await db.query("SELECT * FROM users WHERE email = $1", [
72+
const user = await db.query("DELETE FROM users WHERE email = $1 RETURNING *", [
8573
email,
8674
]);
87-
75+
8876
// Only delete from Firebase if the user has a valid firebase_uid
8977
if (user[0] && user[0].firebase_uid && user[0].firebase_uid.trim() !== '') {
90-
const deletedUser = await admin.auth().deleteUser(user[0].firebase_uid);
91-
console.log("deletedUser", deletedUser)
92-
} else {
93-
console.log("No valid firebase_uid found, skipping Firebase deletion")
78+
await admin.auth().deleteUser(user[0].firebase_uid);
9479
}
9580

96-
// Only try to delete from Firebase if the user exists there
97-
9881
res.status(200).json(keysToCamel(user));
9982
} catch (err) {
10083
res.status(400).send(err.message);

0 commit comments

Comments
 (0)