Issue #7: Lay out and explain file structure

- Move auth code to middleware folder - The auth code runs as middleware
codebru · Jul 18, 2019 · 7d6aa26 · 7d6aa26
1 parent facf99d
commit 7d6aa26
Show file tree

Hide file tree

Showing 4 changed files with 3 additions and 2 deletions.
diff --git a/Components/User/index.js b/Components/User/index.js
@@ -1,7 +1,7 @@
 /* INCLUDES ****************************** */
 const express = require('express');
 const passport = require('passport');
-const { isAlreadyAuthenticated } = require('./../../Utils/Passport/auth');
+const { isAlreadyAuthenticated } = require('../../Middleware/Passport/auth');
 /* **************************************** */
 
 /* ROUTES ********************************* */

diff --git a/Utils/Passport/auth.js → Middleware/Passport/auth.js b/Utils/Passport/auth.js → Middleware/Passport/auth.js
@@ -1,5 +1,6 @@
 /* AUTH FUNCTIONS ************************* */
 function isAuthenticated(req, res, next) {
+// TODO change names of these funtions as they clash with the req.blah name
   if (req.isAuthenticated()) {
     next();
     return;

diff --git a/Utils/Passport/config.js → Middleware/Passport/config.js b/Utils/Passport/config.js → Middleware/Passport/config.js
diff --git a/index.js b/index.js
@@ -5,7 +5,7 @@ const passport = require('passport');
 const flash = require('flash');
 const bodyParser = require('body-parser');
 const dotenv = require('dotenv');
-const passportConfig = require('./Utils/Passport/config');
+const passportConfig = require('./Middleware/Passport/config');
 /* **************************************** */
 
 /* INCLUDE CONFIGS ************************ */