Ability to disable the ExpressJS request log lines

mendhak · mendhak · commit 88b14fe549a6 · 2022-01-03T21:36:20.000Z
diff --git a/index.js b/index.js
@@ -12,7 +12,9 @@ const sleep = promisify(setTimeout);
 app.set('json spaces', 2);
 app.set('trust proxy', ['loopback', 'linklocal', 'uniquelocal']);
 
-app.use(morgan('combined'));
+if(process.env.DISABLE_REQUEST_LOGS !== 'true'){
+  app.use(morgan('combined'));
+}
 
 app.use(function(req, res, next){
   req.pipe(concat(function(data){
diff --git a/tests.sh b/tests.sh
@@ -213,6 +213,23 @@ else
 fi
 
 
+message " Stop containers "
+docker stop http-echo-tests
+
+message " Start container with DISABLE_REQUEST_LOGS "
+docker run -d --rm -e DISABLE_REQUEST_LOGS=true --name http-echo-tests -p 8080:8080 -p 8443:8443 -t mendhak/http-https-echo
+sleep 5
+curl -s -k -X GET https://localhost:8443/strawberry > /dev/null
+if  [ $(docker logs http-echo-tests | grep -c "GET /strawberry HTTP/1.1") -eq 0 ]
+then
+    passed "DISABLE_REQUEST_LOGS disabled Express HTTP logging"
+else
+    failed "DISABLE_REQUEST_LOGS failed"
+    docker logs http-echo-tests
+    exit 1
+fi
+
+
 message " Stop containers "
 docker stop http-echo-tests
 
