log_ignore_path env var now takes a regex

Author: mendhak

index.js

@@ -175,7 +175,7 @@ app.all('*', (req, res) => {
     }
 
     //Certain paths can be ignored in the container logs, useful to reduce noise from healthchecks
-    if (process.env.LOG_IGNORE_PATH != req.path) {
+    if (!process.env.LOG_IGNORE_PATH || !new RegExp(process.env.LOG_IGNORE_PATH).test(req.path)) {
 
       let spacer = 4;
       if(process.env.LOG_WITHOUT_NEWLINE){

tests.sh

@@ -270,7 +270,7 @@ docker stop http-echo-tests
 sleep 5
 
 
-message " Start container with LOG_IGNORE_PATH "
+message " Start container with LOG_IGNORE_PATH (normal path)"
 docker run -d --rm -e LOG_IGNORE_PATH=/ping --name http-echo-tests -p 8080:8080 -p 8443:8443 -t mendhak/http-https-echo:testing
 sleep 5
 curl -s -k -X POST -d "banana" https://localhost:8443/ping > /dev/null
@@ -285,6 +285,38 @@ else
     exit 1
 fi
 
+message " Stop containers "
+docker stop http-echo-tests
+sleep 5
+
+message " Start container with LOG_IGNORE_PATH (regex path)"
+docker run -d --rm -e LOG_IGNORE_PATH="^\/ping|^\/health|^\/metrics" --name http-echo-tests -p 8080:8080 -p 8443:8443 -t mendhak/http-https-echo:testing
+sleep 5
+curl -s -k -X POST -d "banana" https://localhost:8443/metrics > /dev/null
+
+if [ $(docker logs http-echo-tests | wc -l) == 2 ] && \
+   ! [ $(docker logs http-echo-tests | grep banana) ]
+then
+    passed "LOG_IGNORE_PATH ignored the /metrics path"
+else
+    failed "LOG_IGNORE_PATH failed"
+    docker logs http-echo-tests
+    exit 1
+fi
+
+# Test a positive case where the path is not ignored
+curl -s -k -X POST -d "strawberry" https://localhost:8443/veryvisible > /dev/null
+
+if [[ $(docker logs http-echo-tests | grep strawberry) ]]
+then
+    passed "LOG_IGNORE_PATH didn't ignore the /veryvisible path"
+else
+    failed "LOG_IGNORE_PATH failed, it should not ignore the /veryvisible path"
+    docker logs http-echo-tests
+    exit 1
+fi
+
+
 
 message " Stop containers "
 docker stop http-echo-tests