api-health-middleware 1.0.4 → 1.0.5

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "api-health-middleware",
-  "version": "1.0.4",
+  "version": "1.0.5",
   "description": "Express middleware to report per-request API health to a monitoring backend",
   "main": "src/index.js",
   "scripts": {

package/src/middleware.js

@@ -1,23 +1,26 @@
-const INGEST_ENDPOINT = "https://devops-lite-backend.onrender.com";
+const INGEST_ENDPOINT =
+  "https://devops-lite-backend.onrender.com/logs/health";
 
 function apiHealth({ apiKey }) {
-  if (!apiKey) throw new Error("apiKey is required");
+  if (!apiKey) {
+    throw new Error("apiKey is required");
+  }
 
   return function apiHealthMiddleware(req, res, next) {
     const start = process.hrtime.bigint();
 
     res.on("finish", () => {
       try {
-        // Prevent self-logging
+        // Prevent self-logging loop
         if (req.originalUrl.includes("/logs/health")) return;
 
         const durationMs =
-          Number(process.hrtime.bigint() - start) / 1_000_000;
+          Number(process.hrtime.bigint() - start) / 1e6;
 
         const payload = {
           timestamp: new Date().toISOString(),
           method: req.method,
-          endpoint: req.route?.path || req.originalUrl,
+          endpoint: req.route?.path ?? req.path,
           fullUrl: req.originalUrl,
           statusCode: res.statusCode,
           responseTimeMs: Math.round(durationMs),
@@ -27,12 +30,13 @@ function apiHealth({ apiKey }) {
               ? res.statusMessage
               : null,
           ip: req.ip,
-          userAgent: req.headers["user-agent"] || null,
+          userAgent: req.headers["user-agent"] ?? null,
         };
 
-        // Timeout handling using AbortController
         const controller = new AbortController();
-        const timeout = setTimeout(() => controller.abort(), 2000);
+        const timeout = setTimeout(() => {
+          controller.abort();
+        }, 2000);
 
         fetch(INGEST_ENDPOINT, {
           method: "POST",
@@ -43,11 +47,14 @@ function apiHealth({ apiKey }) {
           body: JSON.stringify(payload),
           signal: controller.signal,
         })
-          .catch(() => {}) // Silent fail
-          .finally(() => clearTimeout(timeout));
-
+          .catch(() => {
+            // Silent fail — never break main request
+          })
+          .finally(() => {
+            clearTimeout(timeout);
+          });
       } catch {
-        // Never break main request
+        // Never break request lifecycle
       }
     });