Debugging improvements

Author: dayofpay

config/app.js

@@ -6,6 +6,13 @@ const APP_CONFIG = {
         'MESSAGES' : {
 
         }
+    },
+    'SECURITY' : {
+        PROXY_CHECK : true,
+    },
+    'DEBUG' : {
+        DEBUG_ENABLED : false,
+        DEBUG_PREFIX : '[DEBUG]',
     }
 }
 

controllers/analyticsController.js

@@ -7,6 +7,7 @@ const Site = require('../models/Sites');
 let cors = require('cors');
 const ipTools = require('../utils/ipTools');
 const security = require('../utils/security');
+const config = require('../config/app');
 let corsOptions = {
     origin: function (origin, callback) {
         if (whitelist.indexOf(origin) !== -1) {
@@ -20,10 +21,12 @@ router.post('/sites/:id', async (req, res) => {
     // Get Site URL
 
     try {
-        const checkProxy = await security.isProxy(req.ip);
-        if(checkProxy.isProxy){
-            res.status(403).send({SecurityException : 'You are not allowed to send analytics data using Proxy'});
-            return;
+        if(config.APP_CONFIG.SECURITY.PROXY_CHECK){
+            const checkProxy = await security.isProxy(req.ip);
+            if(checkProxy.isProxy){
+                res.status(403).send({SecurityException : 'You are not allowed to send analytics data using Proxy'});
+                return;
+            }
         }
         const siteUrl = await Site.findById(req.params.id).exec();
         if (siteUrl) {
@@ -58,7 +61,9 @@ router.post('/sites/:id', async (req, res) => {
                 });
             }
         } else {
-            console.log(false);
+            if(config.APP_CONFIG.DEBUG.DEBUG_ENABLED){
+                console.log(`${config.APP_CONFIG.DEBUG.DEBUG_PREFIX} This site does not exists!`);
+            }
         }
     } catch (error) {
         res.send({

controllers/authController.js

@@ -2,6 +2,7 @@
 
 const router = require("express").Router();
 
+const { APP_CONFIG } = require("../config/app");
 const middlewares = require('../middlewares/auth');
 
 const userServices = require('../services/userServices');
@@ -17,7 +18,9 @@ router.post('/login',middlewares.preventAuthenticatedUser,async (req,res) => {
     const token = await userServices.login(email,password);
 
     if(token.hasOwnProperty('error')){
-        console.log('Invalid username or password.');
+        if(APP_CONFIG.DEBUG.DEBUG_ENABLED){
+            console.log(`${APP_CONFIG.DEBUG.DEBUG_PREFIX} Invalid Username OR Password !`);
+        }
         res.render('auth/login',{hasError:true,errorMessage:token.error});
 
         return;
@@ -43,7 +46,9 @@ router.post('/register',middlewares.preventAuthenticatedUser,async(req,res) => {
         return;
     }
 
-    console.log(token);
+    if(APP_CONFIG.DEBUG.DEBUG_ENABLED){
+        console.log(`${APP_CONFIG.DEBUG.DEBUG_PREFIX} Token - ${token}`);
+    }
     res.cookie("auth", token, { httpOnly: true });
 
     res.redirect("/");

controllers/sitesController.js

@@ -6,6 +6,7 @@ const middlewares = require('../middlewares/auth');
 const siteServices = require('../services/siteServices');
 
 const notificationServices = require('../services/notificationServices');
+const { APP_CONFIG } = require("../config/app");
 router.get('/create', middlewares.protectedRoute, async (req, res) => {
 
     res.render('sites/create');
@@ -52,8 +53,9 @@ router.get('/:id/manage', middlewares.protectedRoute, async (req, res) => {
 
 
     const mapped = JSON.stringify(siteData);
-
-    console.log(siteData.browser_list);
+    if(APP_CONFIG.DEBUG.DEBUG_ENABLED){
+        console.log(`${APP_CONFIG.DEBUG.DEBUG_PREFIX} Browser List ${siteData.browser_list}`);
+    }
     if (hasAccess) {
         res.render('index', {
             siteData: JSON.parse(mapped),

middlewares/auth.js

@@ -16,7 +16,6 @@ exports.auth = async (req, res, next) => {
       res.locals.userImage = await utils.getAvatar(decodedToken.email);
       next();
     } catch (error) {
-      console.log({ error });
       res.clearCookie("auth");
       res.redirect("/auth/login");
     }

services/siteServices.js

@@ -1,5 +1,6 @@
 const User = require('../models/User');
 const Site = require('../models/Sites');
+const { APP_CONFIG } = require('../config/app');
 // const jwt = require('jsonwebtoken');
 // const config = require('../database/config');
 async function createSite(siteData,userId) {
@@ -21,8 +22,9 @@ async function createSite(siteData,userId) {
         // Add the created site to the user's sites array
         user.sites.push(site);
         await user.save();
-
-        console.log('Site Created!');
+        if(APP_CONFIG.DEBUG.DEBUG_ENABLED){
+            console.log(`${APP_CONFIG.DEBUG.DEBUG_PREFIX} Site Created!`);
+        }
         return site;
     } catch (error) {
         console.error('Error creating site:', error);
@@ -71,8 +73,9 @@ async function checkUserAccess(req,res){
             return true;
 
         } else {
-
-            console.log('User does not have access to the site');
+            if(APP_CONFIG.DEBUG.DEBUG_ENABLED){
+                console.log(`${APP_CONFIG.DEBUG.DEBUG_PREFIX} User does not have access to the site!`);
+            }
             return false;
 
         }

services/userServices.js

@@ -44,7 +44,7 @@ async function login(email,password){
       return {error:"Invalid email or password!"}
     }
 
-    console.log(isValid);
+
     const payload = {
       _id: user._id,
       email: user.email,