From 59f44e825a242ea639efe5d230113c6bcf656ddf Mon Sep 17 00:00:00 2001 From: emad-salah Date: Thu, 23 Jul 2020 16:43:54 +0100 Subject: [PATCH] Added a .env toggle for enabling/disabling detailed route logging --- src/routes.js | 2 -- src/server.js | 47 +++++++++++++++++++++++++---------------------- 2 files changed, 25 insertions(+), 24 deletions(-) diff --git a/src/routes.js b/src/routes.js index 22e163e7..8b4dbb2a 100644 --- a/src/routes.js +++ b/src/routes.js @@ -272,8 +272,6 @@ module.exports = async ( app.use(async (req, res, next) => { try { - logger.info("Route:", req.path) - if (unprotectedRoutes[req.method][req.path]) { next(); return; diff --git a/src/server.js b/src/server.js index 20b4b3c7..978aa2e6 100644 --- a/src/server.js +++ b/src/server.js @@ -195,31 +195,34 @@ const server = program => { }) app.use((req, res, next) => { - if (sensitiveRoutes[req.method][req.path]) { - logger.info( - JSON.stringify({ - time: new Date(), - ip: req.ip, - method: req.method, - path: req.path, - sessionId: req.sessionId - }) - ) - } else { - logger.info( - JSON.stringify({ - time: new Date(), - ip: req.ip, - method: req.method, - path: req.path, - body: req.body, - query: req.query, - sessionId: req.sessionId - }) - ) + if (process.env.ROUTE_LOGGING === 'true') { + if (sensitiveRoutes[req.method][req.path]) { + logger.info( + JSON.stringify({ + time: new Date(), + ip: req.ip, + method: req.method, + path: req.path, + sessionId: req.sessionId + }) + ) + } else { + logger.info( + JSON.stringify({ + time: new Date(), + ip: req.ip, + method: req.method, + path: req.path, + body: req.body, + query: req.query, + sessionId: req.sessionId + }) + ) + } } next() }) + app.use( session({ secret: defaults.sessionSecret,