fix: add appname to http response time metrics

Unleash · Sep 30, 2022 · 1577df8 · 1577df8
1 parent 9ba441b
commit 1577df8
Showing 2 changed files with 8 additions and 3 deletions.
diff --git a/src/lib/metrics.ts b/src/lib/metrics.ts
@@ -57,7 +57,7 @@ export default class MetricsMonitor {
         const requestDuration = new client.Summary({
             name: 'http_request_duration_milliseconds',
             help: 'App response time',
-            labelNames: ['path', 'method', 'status'],
+            labelNames: ['path', 'method', 'status', 'appName'],
             percentiles: [0.1, 0.5, 0.9, 0.95, 0.99],
             maxAgeSeconds: 600,
             ageBuckets: 5,
@@ -143,8 +143,10 @@ export default class MetricsMonitor {
 
         eventBus.on(
             events.REQUEST_TIME,
-            ({ path, method, time, statusCode }) => {
-                requestDuration.labels(path, method, statusCode).observe(time);
+            ({ path, method, time, statusCode, appName }) => {
+                requestDuration
+                    .labels(path, method, statusCode, appName)
+                    .observe(time);
             },
         );
 

diff --git a/src/lib/middleware/response-time-metrics.ts b/src/lib/middleware/response-time-metrics.ts
@@ -11,11 +11,14 @@ export function responseTimeMetrics(eventBus: EventEmitter): any {
 
         const pathname = req.route ? req.baseUrl + req.route.path : '(hidden)';
 
+        const appName = req.headers['unleash-appname'];
+
         const timingInfo = {
             path: pathname,
             method: req.method,
             statusCode,
             time,
+            appName,
         };
         eventBus.emit(REQUEST_TIME, timingInfo);
     });