Fix LOG_IGNORE_PATH to also skip Morgan access logs

The LOG_IGNORE_PATH environment variable was only preventing the JSON
echo output from being logged, but Morgan HTTP access logs were still
being written for ignored paths. This caused the test to fail because
it expected no log output for requests to /ping when LOG_IGNORE_PATH=/ping.

This fix adds a skip function to the Morgan middleware that checks if
the request path matches LOG_IGNORE_PATH before logging.

Co-Authored-By: Willy Guggenheim <willy@gugcorp.com>

Author: willyguggenheim

index.js

@@ -243,7 +243,15 @@ if(PROMETHEUS_ENABLED === 'true') {
 }
 
 if(process.env.DISABLE_REQUEST_LOGS !== 'true'){
-  app.use(morgan('combined'));
+  app.use(morgan('combined', {
+    skip: function (req, res) {
+      // Skip logging for paths matching LOG_IGNORE_PATH
+      if (process.env.LOG_IGNORE_PATH && new RegExp(process.env.LOG_IGNORE_PATH).test(req.path)) {
+        return true;
+      }
+      return false;
+    }
+  }));
 }
 
 app.use(function(req, res, next){