Merge 97d1823 into fc31921

CHANGELOG.md

@@ -4,6 +4,12 @@ All notable changes to this project will be documented in this file.
 The format is based on [Keep a Changelog](https://keepachangelog.com/en/1.0.0/),
 and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0.html).
 
+### Unreleased
+
+### Added
+
+- add optional metrics on request duration. The metrics collection is enabled by mean of the variable `ENABLE_HTTP_CLIENT_METRICS` (default: false)
+
 ## v5.1.5 - 2022-11-24
 
 ### Fixed

index.d.ts

@@ -123,6 +123,10 @@ declare namespace customPlugin {
   //
   // HTTP CLIENT
   //
+  type HttpClientMetrics = {
+    disabled: boolean
+    urlLabel: string
+  }
   interface HttpClientBaseOptions {
     headers?: http.IncomingHttpHeaders,
     timeout?: number,
@@ -136,6 +140,7 @@ declare namespace customPlugin {
   interface BaseHttpClientResponse {
     headers: http.IncomingHttpHeaders
     statusCode: number
+    duration: number
   }
   interface StreamResponse extends BaseHttpClientResponse {
     payload: NodeJS.ReadableStream
@@ -162,6 +167,7 @@ declare namespace customPlugin {
     errorMessageKey?: string;
     proxy?: HttpClientProxy;
     query?: Record<string, string>;
+    metrics?: HttpClientMetrics;
   }
 
   type RequestBody = any | Buffer | ReadableStream

index.js

@@ -38,6 +38,7 @@ const CLIENTTYPE_HEADER_KEY = 'CLIENTTYPE_HEADER_KEY'
 const BACKOFFICE_HEADER_KEY = 'BACKOFFICE_HEADER_KEY'
 const MICROSERVICE_GATEWAY_SERVICE_NAME = 'MICROSERVICE_GATEWAY_SERVICE_NAME'
 const ADDITIONAL_HEADERS_TO_PROXY = 'ADDITIONAL_HEADERS_TO_PROXY'
+const ENABLE_HTTP_CLIENT_METRICS = 'ENABLE_HTTP_CLIENT_METRICS'
 
 const baseSchema = {
   type: 'object',
@@ -85,6 +86,11 @@ const baseSchema = {
       default: '',
       description: 'comma separated list of additional headers to proxy',
     },
+    [ENABLE_HTTP_CLIENT_METRICS]: {
+      type: 'boolean',
+      default: false,
+      description: 'flag to enable the httpClient metrics',
+    },
   },
 }
 
@@ -219,11 +225,11 @@ function getHttpClientFromRequest(url, baseOptions = {}) {
   const extraHeaders = getCustomHeaders(extraHeadersKeys, requestHeaders)
   const options = getBaseOptionsDecorated(this[ADDITIONAL_HEADERS_TO_PROXY], baseOptions, requestHeaders)
   const serviceHeaders = { ...this.getMiaHeaders(), ...extraHeaders }
-  return new HttpClient(url, serviceHeaders, options)
+  return new HttpClient(url, serviceHeaders, options, this.httpClientMetrics)
 }
 
 function getHttpClient(url, baseOptions = {}) {
-  return new HttpClient(url, {}, baseOptions)
+  return new HttpClient(url, {}, baseOptions, this.httpClientMetrics)
 }
 
 function getHeadersToProxy({ isMiaHeaderInjected = true } = {}) {
@@ -241,6 +247,7 @@ function getHeadersToProxy({ isMiaHeaderInjected = true } = {}) {
 
 function decorateFastify(fastify) {
   const { config } = fastify
+  const httpClientMetrics = config[ENABLE_HTTP_CLIENT_METRICS] ? getHttpClientMetrics(fastify) : {}
 
   fastify.decorateRequest(USERID_HEADER_KEY, config[USERID_HEADER_KEY])
   fastify.decorateRequest(USER_PROPERTIES_HEADER_KEY, config[USER_PROPERTIES_HEADER_KEY])
@@ -261,6 +268,7 @@ function decorateFastify(fastify) {
   fastify.decorateRequest('getDirectServiceProxy', getDirectlyServiceBuilderFromRequest)
   fastify.decorateRequest('getServiceProxy', getServiceBuilderFromRequest)
   fastify.decorateRequest('getHttpClient', getHttpClientFromRequest)
+  fastify.decorateRequest('httpClientMetrics', httpClientMetrics)
 
   fastify.decorate(MICROSERVICE_GATEWAY_SERVICE_NAME, config[MICROSERVICE_GATEWAY_SERVICE_NAME])
   fastify.decorate('addRawCustomPlugin', addRawCustomPlugin)
@@ -270,6 +278,7 @@ function decorateFastify(fastify) {
   fastify.decorate('getDirectServiceProxy', getDirectlyServiceBuilderFromService)
   fastify.decorate('getServiceProxy', getServiceBuilderFromService)
   fastify.decorate('getHttpClient', getHttpClient)
+  fastify.decorate('httpClientMetrics', httpClientMetrics)
 }
 
 async function decorateRequestAndFastifyInstance(fastify, { asyncInitFunction, serviceOptions = {} }) {
@@ -337,6 +346,18 @@ function initCustomServiceEnvironment(envSchema = defaultSchema) {
   }
 }
 
+function getHttpClientMetrics(fastify) {
+  if (fastify.metrics?.client) {
+    const requestDuration = new fastify.metrics.client.Histogram({
+      name: 'http_request_duration_milliseconds',
+      help: 'request duration histogram',
+      labelNames: ['baseUrl', 'url', 'method', 'statusCode'],
+      buckets: [5, 10, 50, 100, 500, 1000, 5000, 10000],
+    })
+    return { requestDuration }
+  }
+}
+
 module.exports = initCustomServiceEnvironment
 module.exports.getDirectServiceProxy = getDirectlyServiceBuilderFromService
 module.exports.getServiceProxy = (microserviceGatewayServiceName, baseOptions = {}) => {

lib/httpClient.js

@@ -1,3 +1,4 @@
+/* eslint-disable max-statements */
 /*
  * Copyright 2022 Mia srl
  *
@@ -21,10 +22,11 @@ const httpsClient = require('https')
 const Pino = require('pino')
 
 class HttpClient {
-  constructor(baseUrl, requestHeadersToProxy, baseOptions = {}) {
+  constructor(baseUrl, requestHeadersToProxy, baseOptions = {}, metrics = {}) {
     this.baseURL = baseUrl
     this.requestHeadersToProxy = requestHeadersToProxy
     this.baseOptions = baseOptions
+    this.metrics = baseOptions.disableMetrics ? {} : metrics
 
     const httpsAgent = getHttpsAgent(baseOptions)
     this.axios = axios.create({
@@ -87,6 +89,7 @@ class HttpClient {
     const headers = getHeaders(options, this.requestHeadersToProxy, this.baseOptions, payload)
     const httpsAgent = getHttpsAgent(options)
     const errorMessageKey = getErrorMessageKey(options, this.baseOptions)
+    const metricsOptions = getMetricsOptions(options.metrics, url, this.metrics)
     try {
       const validateStatus = getValidateStatus(options)
       logger.trace({ baseURL: this.baseURL, url, headers, payload }, 'make call')
@@ -110,6 +113,16 @@ class HttpClient {
         duration: response.duration,
       }
       logger.trace({ url, ...responseBody }, 'response info')
+      if (metricsOptions.enabled) {
+        this.metrics.requestDuration.observe({
+          method: options.method,
+          url: metricsOptions.urlLabel,
+          baseUrl: this.baseURL,
+          statusCode: response.status,
+        },
+        response.duration,
+        )
+      }
       return responseBody
     } catch (error) {
       if (error.response) {
@@ -121,6 +134,16 @@ class HttpClient {
         responseError.duration = error.duration
         // eslint-disable-next-line id-blacklist
         logger.error({ statusCode: error.response.status, message: errorMessage }, 'response error')
+        if (metricsOptions.enabled) {
+          this.metrics.requestDuration.observe({
+            method: options.method,
+            url: metricsOptions.urlLabel,
+            baseUrl: this.baseURL,
+            statusCode: error.response.status,
+          },
+          error.duration,
+          )
+        }
         throw responseError
       }
       const errToReturn = new Error(error.message)
@@ -217,6 +240,13 @@ function getHeaders(options, miaHeaders, baseOptions, payload) {
   }
 }
 
+function getMetricsOptions(metricsOptions = {}, url, metrics) {
+  return {
+    enabled: Boolean(metrics.requestDuration && metricsOptions.disabled !== true),
+    urlLabel: metricsOptions.urlLabel ?? url,
+  }
+}
+
 function decorateResponseWithDuration(axiosInstance) {
   axiosInstance.interceptors.response.use(
     (response) => {

tests/getHttpClient.test.js

@@ -10,14 +10,22 @@ const MY_AWESOME_SERVICE_PROXY_HTTPS_URL = 'https://my-awesome-service'
 const MY_AWESOME_SERVICE_PROXY_HTTP_URL_CUSTOM_PORT = 'http://my-awesome-service:3000'
 const MY_AWESOME_SERVICE_PROXY_HTTPS_URL_CUSTOM_PORT = 'https://my-awesome-service:3001'
 
+const fastifyMock = {
+  httpClientMetrics: {
+    requestDuration: {
+      observe: () => { /* no-op*/ },
+    },
+  },
+}
+
 tap.test('getHttpClient available for testing - complete url passed', async t => {
   nock.disableNetConnect()
   t.teardown(() => {
     nock.enableNetConnect()
   })
 
   const RETURN_MESSAGE = 'OK'
-  const customProxy = getHttpClient(MY_AWESOME_SERVICE_PROXY_HTTP_URL)
+  const customProxy = getHttpClient.call(fastifyMock, MY_AWESOME_SERVICE_PROXY_HTTP_URL)
   const awesomeHttpServiceScope = nock(`${MY_AWESOME_SERVICE_PROXY_HTTP_URL}:80`)
     .get('/test-endpoint')
     .reply(200, {
@@ -38,7 +46,7 @@ tap.test('getHttpClient available for testing - timeout passed', async t => {
   })
 
   const RETURN_MESSAGE = 'OK'
-  const customProxy = getHttpClient(MY_AWESOME_SERVICE_PROXY_HTTP_URL, {
+  const customProxy = getHttpClient.call(fastifyMock, MY_AWESOME_SERVICE_PROXY_HTTP_URL, {
     timeout: 100,
   })
   const awesomeHttpServiceScope = nock(`${MY_AWESOME_SERVICE_PROXY_HTTP_URL}:80`)
@@ -65,7 +73,7 @@ tap.test('getHttpClient available for testing - https url', async t => {
   })
 
   const RETURN_MESSAGE = 'OK'
-  const customProxy = getHttpClient(MY_AWESOME_SERVICE_PROXY_HTTPS_URL)
+  const customProxy = getHttpClient.call(fastifyMock, MY_AWESOME_SERVICE_PROXY_HTTPS_URL)
   const awesomeHttpsServiceScope = nock(`${MY_AWESOME_SERVICE_PROXY_HTTPS_URL}:443`)
     .get('/test-endpoint')
     .reply(200, {
@@ -86,7 +94,7 @@ tap.test('getHttpClient available for testing - custom port 3000 - custom header
   })
 
   const RETURN_MESSAGE = 'OK'
-  const customProxy = getHttpClient(MY_AWESOME_SERVICE_PROXY_HTTP_URL_CUSTOM_PORT,
+  const customProxy = getHttpClient.call(fastifyMock, MY_AWESOME_SERVICE_PROXY_HTTP_URL_CUSTOM_PORT,
     {
       headers: {
         'test-header': 'test header works',
@@ -113,7 +121,7 @@ tap.test('getHttpClient available for testing - https url - custom port 3001', a
   })
 
   const RETURN_MESSAGE = 'OK'
-  const customProxy = getHttpClient(MY_AWESOME_SERVICE_PROXY_HTTPS_URL_CUSTOM_PORT)
+  const customProxy = getHttpClient.call(fastifyMock, MY_AWESOME_SERVICE_PROXY_HTTPS_URL_CUSTOM_PORT)
   const awesomeHttpsServiceScope = nock(`${MY_AWESOME_SERVICE_PROXY_HTTPS_URL}:3001`)
     .get('/test-endpoint')
     .reply(200, {
@@ -130,7 +138,7 @@ tap.test('getHttpClient available for testing - https url - custom port 3001', a
 tap.test('getHttpClient throws on invalid url', async t => {
   const invalidUrl = 'httpnot-a-complete-url'
   try {
-    getHttpClient(invalidUrl)
+    getHttpClient.call(fastifyMock, invalidUrl)
   } catch (error) {
     t.notOk(true, 'The function should not throw anymore if the url is not a valid one, bet return the standard proxy')
   }