Merge pull request #2562 from rajatmohan22/logoutfix2

lindapaiste · web-flow · commit a175a0498ac6 · 2023-11-25T14:44:07.000-06:00
Fixes to destroySession
diff --git a/server/controllers/session.controller.js b/server/controllers/session.controller.js
@@ -36,6 +36,12 @@ export function destroySession(req, res, next) {
       next(err);
       return;
     }
-    res.json({ success: true });
+    req.session.destroy((error) => {
+      if (error) {
+        next(error);
+        return;
+      }
+      res.json({ success: true });
+    });
   });
 }

Original file line number	Diff line number	Diff line change
`@@ -36,6 +36,12 @@ export function destroySession(req, res, next) {`
`36`	`36`	`next(err);`
`37`	`37`	`return;`
`38`	`38`	`}`
`39`		`- res.json({ success: true });`
	`39`	`+ req.session.destroy((error) => {`
	`40`	`+ if (error) {`
	`41`	`+ next(error);`
	`42`	`+ return;`
	`43`	`+ }`
	`44`	`+ res.json({ success: true });`
	`45`	`+ });`
`40`	`46`	`});`
`41`	`47`	`}`