fix: request logs (#1964)

* fix: request logs * chore: remove comments
specklesystems · Jan 11, 2024 · ee84900 · ee84900
1 parent 823996d
commit ee84900
Show file tree

Hide file tree

Showing 3 changed files with 44 additions and 11 deletions.
diff --git a/packages/frontend-2/plugins/002-healthMetrics.server.ts b/packages/frontend-2/plugins/002-healthMetrics.server.ts
@@ -21,7 +21,7 @@ export default defineNuxtPlugin((ctx) => {
         routeName: name,
         routePath: path
       },
-      `[{routePath}] SSR rendered in {responseTime}ms`
+      '{routePath} SSR rendered in {responseTime} ms'
     )
   })
 })
diff --git a/packages/frontend-2/server/middleware/001-logging.ts b/packages/frontend-2/server/middleware/001-logging.ts
@@ -60,16 +60,32 @@ export const LoggingMiddleware = pinoHttp({
 
     return shouldBeDebug ? 'debug' : 'info'
   },
+  customSuccessMessage() {
+    return '{requestPath} request {requestStatus} in {responseTime} ms'
+  },
 
-  customSuccessMessage(req, res) {
+  customSuccessObject(req, res, val: Record<string, unknown>) {
     const isCompleted = !req.readableAborted && res.writableEnded
-    const statusMessage = isCompleted ? 'request completed' : 'request aborted'
-
-    return `[{req.path}] ${statusMessage} in {responseTime}ms`
+    const requestStatus = isCompleted ? 'completed' : 'aborted'
+    const requestPath = req.url?.split('?')[0] || 'unknown'
+    return {
+      ...val,
+      requestStatus,
+      requestPath
+    }
   },
 
   customErrorMessage() {
-    return `[{req.path}] request errored`
+    return '{requestPath} request {requestStatus} in {responseTime} ms'
+  },
+  customErrorObject(req, res, err, val: Record<string, unknown>) {
+    const requestStatus = 'failed'
+    const requestPath = req.url?.split('?')[0] || 'unknown'
+    return {
+      ...val,
+      requestStatus,
+      requestPath
+    }
   },
 
   // we need to redact any potential sensitive data from being logged.

diff --git a/packages/server/logging/expressLogging.ts b/packages/server/logging/expressLogging.ts
@@ -34,15 +34,32 @@ export const LoggingExpressMiddleware = HttpLogger({
     return 'info'
   },
 
-  customSuccessMessage(req, res) {
-    const isCompleted = !req.readableAborted && res.writableEnded
-    const statusMessage = isCompleted ? 'request completed' : 'request aborted'
+  customSuccessMessage() {
+    return '{requestPath} request {requestStatus} in {responseTime} ms'
+  },
 
-    return `[{req.path}] ${statusMessage} in {responseTime}ms`
+  customSuccessObject(req, res, val: Record<string, unknown>) {
+    const isCompleted = !req.readableAborted && res.writableEnded
+    const requestStatus = isCompleted ? 'completed' : 'aborted'
+    const requestPath = req.url?.split('?')[0] || 'unknown'
+    return {
+      ...val,
+      requestStatus,
+      requestPath
+    }
   },
 
   customErrorMessage() {
-    return `[{req.path}] request errored`
+    return '{requestPath} request {requestStatus} in {responseTime} ms'
+  },
+  customErrorObject(req, res, err, val: Record<string, unknown>) {
+    const requestStatus = 'failed'
+    const requestPath = req.url?.split('?')[0] || 'unknown'
+    return {
+      ...val,
+      requestStatus,
+      requestPath
+    }
   },
 
   // we need to redact any potential sensitive data from being logged.