Merge pull request #66 from BonnierNews/feature/add-express-route-log…

…ging Add express route logging
BonnierNews · Jun 10, 2024 · 6e0c46b · 6e0c46b
2 parents f88f181 + 84e08fb
commit 6e0c46b
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 5 deletions.
diff --git a/lib/middleware.js b/lib/middleware.js
@@ -16,10 +16,11 @@ function accessLog(req, res, next) {
     res.removeListener("close", afterResponse);
 
     if (!isHealthOrMetricRequest(req)) {
+      const meta = { ...req.debugMeta, ...(req.route && { route: req.route.path, methods: req.route.methods }) };
       if (!res.finished) {
-        logger.info(`"${req.method} ${req.originalUrl}" NO RESPONSE SENT ${new Date() - time} ms`, req.debugMeta);
+        logger.info(`"${req.method} ${req.originalUrl}" NO RESPONSE SENT ${new Date() - time} ms`, meta);
       } else {
-        logger.info(`"${req.method} ${req.originalUrl}" ${res.statusCode} ${new Date() - time} ms`, req.debugMeta);
+        logger.info(`"${req.method} ${req.originalUrl}" ${res.statusCode} ${new Date() - time} ms`, meta);
       }
     }
   }

diff --git a/package-lock.json b/package-lock.json
diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "lu-server",
-  "version": "7.0.1",
+  "version": "7.0.2",
   "engines": {
     "node": ">=18 <=20",
     "yarn": "please use npm"