Skip to content

Commit 67716f0

Browse files
authored
fix(auth.service.js): fixes deprecated error callback in mongoose save method (#323)
1 parent e56d90e commit 67716f0

File tree

1 file changed

+12
-15
lines changed

1 file changed

+12
-15
lines changed

api/server/services/auth.service.js

Lines changed: 12 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -24,25 +24,22 @@ const loginUser = async (user) => {
2424
};
2525

2626
const logoutUser = async (user, refreshToken) => {
27-
User.findById(user._id).then((user) => {
28-
const tokenIndex = user.refreshToken.findIndex((item) => item.refreshToken === refreshToken);
27+
try {
28+
const userFound = await User.findById(user._id);
29+
const tokenIndex = userFound.refreshToken.findIndex((item) => item.refreshToken === refreshToken);
2930

3031
if (tokenIndex !== -1) {
31-
user.refreshToken.id(user.refreshToken[tokenIndex]._id).remove();
32+
userFound.refreshToken.id(userFound.refreshToken[tokenIndex]._id).remove();
3233
}
3334

34-
user.save((err) => {
35-
if (err) {
36-
return { status: 500, message: err.message };
37-
} else {
38-
//res.clearCookie('refreshToken', COOKIE_OPTIONS);
39-
// removeTokenCookie(res);
40-
return { status: 200, message: 'Logout successful' };
41-
}
42-
});
43-
});
44-
return { status: 200, message: 'Logout successful' };
45-
};
35+
await userFound.save();
36+
//res.clearCookie('refreshToken', COOKIE_OPTIONS);
37+
// removeTokenCookie(res);
38+
return { status: 200, message: 'Logout successful' };
39+
} catch (err) {
40+
return { status: 500, message: err.message };
41+
}
42+
}
4643

4744
const registerUser = async (user) => {
4845
let response = {};

0 commit comments

Comments
 (0)