@platformatic/metrics 2.69.0 → 2.70.0

package/index.js

@@ -12,9 +12,6 @@ async function collectMetrics (serviceId, workerId, metricsConfig = {}, registry
     registry = new Registry()
   }
 
-  const httpRequestCallbacks = []
-  const httpResponseCallbacks = []
-
   const labels = { ...metricsConfig.labels, serviceId }
   if (workerId >= 0) {
     labels.workerId = workerId
@@ -28,51 +25,25 @@ async function collectMetrics (serviceId, workerId, metricsConfig = {}, registry
   }
 
   if (metricsConfig.httpMetrics) {
-    {
-      const { startTimer, endTimer } = collectHttpMetrics(registry, {
-        customLabels: ['telemetry_id'],
-        getCustomLabels: req => {
-          const telemetryId = req.headers?.['x-plt-telemetry-id'] ?? 'unknown'
-          return { telemetry_id: telemetryId }
-        }
-      })
-      httpRequestCallbacks.push(startTimer)
-      httpResponseCallbacks.push(endTimer)
-    }
-
-    {
-      // TODO: check if it's a nodejs environment
-      // Needed for the Meraki metrics
-      const { startTimer, endTimer } = collectHttpMetrics(registry, {
-        customLabels: ['telemetry_id'],
-        getCustomLabels: req => {
-          const telemetryId = req.headers?.['x-plt-telemetry-id'] ?? 'unknown'
-          return { telemetry_id: telemetryId }
-        },
-        histogram: {
-          name: 'http_request_all_duration_seconds',
-          help: 'request duration in seconds summary for all requests',
-          collect: function () {
-            process.nextTick(() => this.reset())
-          }
-        },
-        summary: {
-          name: 'http_request_all_summary_seconds',
-          help: 'request duration in seconds histogram for all requests',
-          collect: function () {
-            process.nextTick(() => this.reset())
-          }
-        }
-      })
-      httpRequestCallbacks.push(startTimer)
-      httpResponseCallbacks.push(endTimer)
-    }
+    collectHttpMetrics(registry, {
+      customLabels: ['telemetry_id'],
+      getCustomLabels: req => {
+        const telemetryId = req.headers?.['x-plt-telemetry-id'] ?? 'unknown'
+        return { telemetry_id: telemetryId }
+      },
+      histogram: {
+        name: 'http_request_all_duration_seconds',
+        help: 'request duration in seconds summary for all requests'
+      },
+      summary: {
+        name: 'http_request_all_summary_seconds',
+        help: 'request duration in seconds histogram for all requests'
+      }
+    })
   }
 
   return {
     registry,
-    startHttpTimer: options => httpRequestCallbacks.forEach(cb => cb(options)),
-    endHttpTimer: options => httpResponseCallbacks.forEach(cb => cb(options))
   }
 }
 

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@platformatic/metrics",
-  "version": "2.69.0",
+  "version": "2.70.0",
   "description": "Platformatic Stackable Metrics",
   "main": "index.js",
   "type": "commonjs",

package/test/helper.js

@@ -124,15 +124,7 @@ const expectedMetrics = [
   {
     name: 'http_request_all_duration_seconds',
     type: 'histogram'
-  },
-  {
-    name: 'http_request_duration_seconds',
-    type: 'histogram'
-  },
-  {
-    name: 'http_request_summary_seconds',
-    type: 'summary'
-  },
+  }
 ]
 
 function assertMetric (metrics, metric) {
@@ -148,31 +140,31 @@ function assertMetric (metrics, metric) {
 
 function assertSummary (metrics, metric) {
   assert.ok(metrics.includes(`${metric.name}{quantile="0.01"`))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.01"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.05"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.5"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.9"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.95"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.99"'))
-  assert.ok(metrics.includes('http_request_summary_seconds{quantile="0.999"'))
-  assert.ok(metrics.includes('http_request_summary_seconds_sum{'))
-  assert.ok(metrics.includes('http_request_summary_seconds_count{'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.01"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.05"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.5"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.9"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.95"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.99"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds{quantile="0.999"'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds_sum{'))
+  assert.ok(metrics.includes('http_request_all_summary_seconds_count{'))
 }
 
 function assertHistogram (metrics, metric) {
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.005"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.01"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.025"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.05"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.1"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.25"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="0.5"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="1"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="2.5"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="5"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_bucket{le="10"'))
-  assert.ok(metrics.includes('http_request_duration_seconds_sum{'))
-  assert.ok(metrics.includes('http_request_duration_seconds_count{'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.005"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.01"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.025"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.05"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.1"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.25"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="0.5"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="1"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="2.5"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="5"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_bucket{le="10"'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_sum{'))
+  assert.ok(metrics.includes('http_request_all_duration_seconds_count{'))
 }
 
 module.exports = {

package/test/index.test.js

@@ -8,8 +8,6 @@ test('returns expected structure', async () => {
   const result = await collectMetrics('test-service', 1, {})
 
   assert.ok(result.registry)
-  assert.equal(typeof result.startHttpTimer, 'function')
-  assert.equal(typeof result.endHttpTimer, 'function')
 })
 
 test('accepts custom registry', async () => {