observe-node 1.0.3 → 1.0.4

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "observe-node",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Structured logs + Prometheus metrics SDK for Node.js (Loki/Grafana ready)",
   "main": "src/index.js",
   "type": "commonjs",

package/src/http/express.js

@@ -41,12 +41,41 @@ function expressMiddleware(observe, opts = {}) {
       return _json(body);
     };
 
-    const _send = res.send.bind(res);
-    res.send = (body) => {
-      // if body is string/buffer, avoid logging full
-      capturedResponse = { truncated: true, bytes: Buffer.isBuffer(body) ? body.length : (typeof body === "string" ? Buffer.byteLength(body) : null) };
+ const _send = res.send.bind(res);
+res.send = (body) => {
+  try {
+    // If body is a JSON string, parse it and store full object
+    if (typeof body === "string") {
+      const trimmed = body.trim();
+      if (trimmed.startsWith("{") || trimmed.startsWith("[")) {
+        const parsed = JSON.parse(trimmed);
+        capturedResponse = pickResponseBody(parsed, maxResponseBytes);
+        return _send(body);
+      }
+      // non-json string => don't log full
+      capturedResponse = { truncated: true, bytes: Buffer.byteLength(body, "utf8") };
       return _send(body);
-    };
+    }
+
+    // If Buffer, don't log full
+    if (Buffer.isBuffer(body)) {
+      capturedResponse = { truncated: true, bytes: body.length };
+      return _send(body);
+    }
+
+    // If object (sometimes people do res.send({}))
+    if (typeof body === "object" && body !== null) {
+      capturedResponse = pickResponseBody(body, maxResponseBytes);
+      return _send(body);
+    }
+
+    capturedResponse = { truncated: true, bytes: null };
+    return _send(body);
+  } catch (e) {
+    capturedResponse = { truncated: true, bytes: null, parse_error: true };
+    return _send(body);
+  }
+};
 
     res.on("finish", () => {
       const duration_ms = Date.now() - start;