@mimik/request-retry 2.0.12 → 2.1.0

package/README.md

@@ -29,6 +29,10 @@ The following properties are added to the `options` object under the `retry` pro
    - *responseName*: name to associate with the response. The default is `response`or is the value is not a string or and empty string it is set to `response`,
 - timeout `{int}`: in seconds the timeout to be set for the request and retries. If the timeout is reached, a TimeoutError will be generated. The default is `50 seconds`. If the value is less than `10 seconds` the value is set to `10 seconds`, and if the value is more than `120 seconds` the value is set to `120 seconds`,
 - retryStrategy `{function}`: function describing the retry strategy. The parameters are (`nbRetry`, `err`, `options`, `correlationId`). Must return a `boolean`, if not the function strategy is ignored.
+The following properties are added to the `options` object under the 'metrics' property:
+- HTTPRequestDuration `function`: prom-client function to measure the delay of the request,
+- url: optional url to be added for labelling the metric. If not present the url of the request will be used.
+
 The `default` retryStategy is:
 
 ``` javascript

package/index.js

@@ -49,6 +49,10 @@ Promise.config({ cancellation: true });
  *    - *responseName*: name to associate with the response. The default is `response`or is the value is not a string or and empty string it is set to `response`,
  * - timeout `{int}`: in seconds the timeout to be set for the request and retries. If the timeout is reached, a TimeoutError will be generated. The default is `50 seconds`. If the value is less than `10 seconds` the value is set to `10 seconds`, and if the value is more than `120 seconds` the value is set to `120 seconds`,
  * - retryStrategy `{function}`: function describing the retry strategy. The parameters are (`nbRetry`, `err`, `options`, `correlationId`). Must return a `boolean`, if not the function strategy is ignored.
+ * The following properties are added to the `options` object under the 'metrics' property:
+ * - HTTPRequestDuration `function`: prom-client function to measure the delay of the request,
+ * - url: optional url to be added for labelling the metric. If not present the url of the request will be used.
+ *
  * The `default` retryStategy is:
  *
  *``` javascript
@@ -68,7 +72,10 @@ Promise.config({ cancellation: true });
  * - `qs` is used to assign to `params`
  */
 const rpRetry = (origOptions) => {
+  const startHrTime = process.hrtime();
   const options = origOptions;
+  const { metrics } = options;
+  const enteredUrl = options.uri || options.url;
   const correlationId = (options.headers && options.headers['x-correlation-id']) ? options.headers['x-correlation-id'] : getCorrelationId();
   const errors = [];
   const delayPromises = [];
@@ -77,13 +84,25 @@ const rpRetry = (origOptions) => {
   const logLevelLength = Object.keys(logLevel).length;
   let nbRetries = 0;
   let mainTimeout;
+  const measure = (statusCode) => {
+    if (metrics && metrics.HTTPRequestDuration) {
+      const elapsedHrTime = process.hrtime(startHrTime);
+      const elapsedTimeInMs = elapsedHrTime[0] * 1000 + elapsedHrTime[1] / 1e6;
+
+      metrics.HTTPRequestDuration
+        .labels('rpRetry', options.method, metrics.url || enteredUrl, enteredUrl.includes('?'), statusCode)
+        .observe(elapsedTimeInMs);
+    }
+  };
 
   if (!options.headers) options.headers = { 'user-agent': setUserAgent() };
   else if (!options.headers['user-agent']) options.headers['user-agent'] = setUserAgent();
+  const optionsInfo = { ...options };
+  delete optionsInfo.metrics;
 
   const retryProcess = (nbRetry) => rp(options)
     .then((response) => {
-      const info = { options, nbRetries, errors };
+      const info = { options: optionsInfo, nbRetries, errors };
 
       if (logLevel.response) {
         info[logLevel.responseName] = setResponse(response, logLevel.responseDetails);
@@ -136,7 +155,7 @@ const rpRetry = (origOptions) => {
       error = getRichError(
         'System',
         'request error response',
-        { options },
+        { options: optionsInfo },
         error,
         logLevel.error || ((logLevelLength === 0) ? DEFAULT_LOGLEVEL_ERROR : undefined),
         correlationId,
@@ -145,17 +164,19 @@ const rpRetry = (origOptions) => {
     }, criteria.timeout * 1000);
   });
 
-  if (logLevel.request) logger[logLevel.request]('making a request', { options, criteria }, correlationId);
-  else if (logLevelLength === 0) logger[DEFAULT_LOGLEVEL_REQUEST]('making a request', { options, criteria }, correlationId);
+  if (logLevel.request) logger[logLevel.request]('making a request', { options: optionsInfo, criteria }, correlationId);
+  else if (logLevelLength === 0) logger[DEFAULT_LOGLEVEL_REQUEST]('making a request', { options: optionsInfo, criteria }, correlationId);
   return Promise.race([retryPromise, mainTimeoutPromise])
     .then((result) => {
       mainTimeoutPromise.cancel('main timeout');
       clearTimeout(mainTimeout);
+      measure(200);
       return result;
     })
     .catch((err) => {
       mainTimeoutPromise.cancel(`main timeout error: ${err.message}`);
       clearTimeout(mainTimeout);
+      measure(err.statusCode || 500);
       throw err;
     });
 };

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mimik/request-retry",
-  "version": "2.0.12",
+  "version": "2.1.0",
   "description": "Request retry wrapping axios",
   "main": "index.js",
   "scripts": {
@@ -31,7 +31,7 @@
   "dependencies": {
     "@mimik/request-helper": "^1.7.7",
     "@mimik/response-helper": "^2.6.2",
-    "@mimik/sumologic-winston-logger": "^1.6.11",
+    "@mimik/sumologic-winston-logger": "^1.6.13",
     "axios": "1.2.1",
     "bluebird": "3.7.2",
     "lodash": "4.17.21"
@@ -42,7 +42,7 @@
     "acorn": "8.8.1",
     "body-parser": "1.20.1",
     "chai": "4.3.7",
-    "eslint": "8.28.0",
+    "eslint": "8.30.0",
     "eslint-config-airbnb": "19.0.4",
     "eslint-plugin-import": "2.26.0",
     "eslint-plugin-jsx-a11y": "6.6.1",
@@ -51,10 +51,10 @@
     "express": "4.18.2",
     "husky": "8.0.2",
     "jsdoc-to-markdown": "8.0.0",
-    "mocha": "10.1.0",
+    "mocha": "10.2.0",
     "mochawesome": "7.1.3",
     "nyc": "15.1.0",
-    "sinon": "15.0.0",
+    "sinon": "15.0.1",
     "verror": "1.10.1"
   }
 }