Fixed Logout function, added TODO in auth.js (#296)

* Fixed logout function, added TODO in auth.js * Fixed logout function, added TODO in auth.js * Deleted switch test case in /src/utility/auth.js --------- Co-authored-by: tschiken <[email protected]>
PhlexPlexico · Dec 29, 2024 · 6ee1d26 · 6ee1d26
1 parent 0dd901f
commit 6ee1d26
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 7 deletions.
diff --git a/app.js b/app.js
@@ -174,9 +174,11 @@ app.get(
     }
   }
 );
-app.get("/logout", (req, res) => {
-  req.logout();
-  res.redirect("/");
+app.get('/logout', function(req, res, next) {
+  req.logout(function(err) {
+    if (err) { return next(err); }
+    res.redirect('/');
+  });
 });
 // END Steam API Calls.
 

diff --git a/src/utility/auth.js b/src/utility/auth.js
@@ -25,10 +25,7 @@ passport.deserializeUser((obj, done) => {
 function strategyForEnvironment() {
   let strategy;
   switch (process.env.NODE_ENV) {
-    case "test":
-      const newUser = new user();
-      strategy = new MockStrategy({ name: "steam", user: newUser, passReqToCallback: true }, returnStrategy);
-      break;
+      // #TODO: Fix crashing while on dev environment. "cb is not a function" error. Discord @tshiken
     default:
       strategy = new SteamStrategy(
         {