ShogunIsHere/spotify-manager
1
0
Fork 0
mirror of https://github.com/20kaushik02/spotify-manager.git synced 2025-12-06 08:04:06 +00:00
Code Issues Packages Projects Releases Wiki Activity

session cookie cleared on client side for logout/deauth

Browse Source
This commit is contained in:
Kaushik Narayan R 2024-01-08 20:09:10 -07:00
parent 143391507e
commit 2225f5db49
2 changed files with 8 additions and 4 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
controllers/auth.js
View File

@ -135,10 +135,12 @@ const logout = async (req, res) => {
const delSession = req.session.destroy((err) => {
if (Object.keys(err).length) {
logger.error("Error while logging out", { err });
return res.sendStatus(500);
} else {
logger.info("Logged out.", { sessionID: delSession.id });
res.clearCookie("connect.sid");
return res.sendStatus(200);
}
return res.sendStatus(200);
})
} catch (error) {
logger.error('logout', { error });

8
middleware/authCheck.js
View File

@ -13,12 +13,14 @@ const isAuthenticated = (req, res, next) => {
next()
} else {
const delSession = req.session.destroy((err) => {
if (err) {
if (Object.keys(err).length) {
logger.error("Error while destroying session.", { err });
return res.status(500).send("Server error, try later.");
} else {
logger.info("Session destroyed.", { sessionID: delSession.id });
logger.info("Session invalid, destroyed.", { sessionID: delSession.id });
res.clearCookie("connect.sid");
return res.sendStatus(401);
}
return res.sendStatus(401);
});
}
}