@mojaloop/sdk-scheme-adapter 16.0.0 → 17.0.0

package/CHANGELOG.md

@@ -1,4 +1,21 @@
 # Changelog: [mojaloop/thirdparty-api-svc](https://github.com/mojaloop/thirdparty-api-svc)
+## [17.0.0](https://github.com/mojaloop/sdk-scheme-adapter/compare/v15.0.1...v17.0.0) (2022-07-04)
+
+
+### ⚠ BREAKING CHANGES
+
+* add dummy pr to major version bump due to unsquashed title (#327)
+
+### Features
+
+* suppress health check logs ([#328](https://github.com/mojaloop/sdk-scheme-adapter/issues/328)) ([64fadde](https://github.com/mojaloop/sdk-scheme-adapter/commit/64faddea8307aa0c19d56466d0afe8f8208c4d66))
+
+
+### Chore
+
+* add dummy pr to major version bump due to unsquashed title ([#327](https://github.com/mojaloop/sdk-scheme-adapter/issues/327)) ([ff0f29b](https://github.com/mojaloop/sdk-scheme-adapter/commit/ff0f29b7ce560565ee21cf6ae84118e1b391b5b5))
+* **release:** 16.0.0 [skip ci] ([0071c65](https://github.com/mojaloop/sdk-scheme-adapter/commit/0071c65c3665239d0a482e76498f9eec5c288043))
+
 ## [16.0.0](https://github.com/mojaloop/sdk-scheme-adapter/compare/v15.0.1...v16.0.0) (2022-07-01)
 
 

package/audit-resolve.json

@@ -95,31 +95,6 @@
     "1068310|widdershins>markdown-it>yargs>yargs-parser": {
       "decision": "ignore",
       "madeAt": 1656681883038
-    },
-    "1070030|@mojaloop/central-services-shared>@mojaloop/event-sdk>grpc>protobufjs>widdershins>markdown-it": {
-      "decision": "ignore",
-      "madeAt": 1656691882275,
-      "expiresAt": 1659283873047
-    },
-    "1070030|shins>markdown-it": {
-      "decision": "ignore",
-      "madeAt": 1656691882275,
-      "expiresAt": 1659283873047
-    },
-    "1068310|@mojaloop/central-services-shared>@mojaloop/event-sdk>grpc>protobufjs>widdershins>markdown-it>yargs>yargs-parser": {
-      "decision": "ignore",
-      "madeAt": 1656691883964,
-      "expiresAt": 1659283873047
-    },
-    "1068155|shins>markdown-it>sanitize-html": {
-      "decision": "ignore",
-      "madeAt": 1656691885077,
-      "expiresAt": 1659283873047
-    },
-    "1070260|shins>markdown-it>sanitize-html": {
-      "decision": "ignore",
-      "madeAt": 1656691886045,
-      "expiresAt": 1659283873047
     }
   },
   "rules": {},

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "@mojaloop/sdk-scheme-adapter",
-  "version": "16.0.0",
+  "version": "17.0.0",
   "description": "An adapter for connecting to Mojaloop API enabled switches.",
   "main": "src/index.js",
   "types": "src/index.d.ts",

package/src/InboundServer/index.js

@@ -23,6 +23,8 @@ const router = require('../lib/router');
 const handlers = require('./handlers');
 const middlewares = require('./middlewares');
 
+const logExcludePaths = ['/'];
+
 class InboundApi extends EventEmitter {
     constructor(conf, logger, cache, validator, wso2) {
         super({ captureExceptions: true });
@@ -99,7 +101,7 @@ class InboundApi extends EventEmitter {
             api.use(middlewares.createJwsValidator(logger, jwsVerificationKeys, jwsExclusions));
         }
 
-        api.use(middlewares.applyState({ cache, wso2, conf }));
+        api.use(middlewares.applyState({ cache, wso2, conf, logExcludePaths }));
         api.use(middlewares.createLogger(logger));
         api.use(middlewares.createRequestValidator(validator));
         api.use(middlewares.assignFspiopIdentifier());
@@ -133,7 +135,7 @@ class InboundServer extends EventEmitter {
     constructor(conf, logger, cache, wso2) {
         super({ captureExceptions: true });
         this._conf = conf;
-        this._validator = new Validate();
+        this._validator = new Validate({ logExcludePaths });
         this._logger = logger;
         this._api = new InboundApi(
             conf,

package/src/InboundServer/middlewares.js

@@ -353,13 +353,17 @@ const createLogger = (logger) => async (ctx, next) => {
         path: ctx.path,
         method: ctx.method
     }});
-    ctx.state.logger.push({ body: ctx.request.body }).log('Request received');
+    if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+        ctx.state.logger.push({body: ctx.request.body}).log('Request received');
+    }
     try {
         await next();
     } catch (err) {
         ctx.state.logger.push(err).log('Error');
     }
-    await ctx.state.logger.log('Request processed');
+    if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+        await ctx.state.logger.log('Request processed');
+    }
 };
 
 
@@ -369,10 +373,14 @@ const createLogger = (logger) => async (ctx, next) => {
  * @return {Function}
  */
 const createRequestValidator = (validator) => async (ctx, next) => {
-    ctx.state.logger.log('Validating request');
+    if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+        ctx.state.logger.log('Validating request');
+    }
     try {
         ctx.state.path = validator.validateRequest(ctx, ctx.state.logger);
-        ctx.state.logger.log('Request passed validation');
+        if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+            ctx.state.logger.log('Request passed validation');
+        }
         await next();
     } catch (err) {
         ctx.state.logger.push({ err }).log('Request failed validation.');

package/src/OutboundServer/index.js

@@ -23,6 +23,7 @@ const handlers = require('./handlers');
 const middlewares = require('./middlewares');
 
 const endpointRegex = /\/.*/g;
+const logExcludePaths = ['/'];
 
 class OutboundApi extends EventEmitter {
     constructor(conf, logger, cache, validator, metricsClient, wso2) {
@@ -36,7 +37,7 @@ class OutboundApi extends EventEmitter {
         this._api.use(middlewares.createErrorHandler(this._logger));
         this._api.use(middlewares.createRequestIdGenerator());
         this._api.use(koaBody()); // outbound always expects application/json
-        this._api.use(middlewares.applyState({ cache, wso2, conf, metricsClient }));
+        this._api.use(middlewares.applyState({ cache, wso2, conf, metricsClient, logExcludePaths }));
         this._api.use(middlewares.createLogger(this._logger));
 
         //Note that we strip off any path on peerEndpoint config after the origin.
@@ -69,7 +70,7 @@ class OutboundApi extends EventEmitter {
 class OutboundServer extends EventEmitter {
     constructor(conf, logger, cache, metricsClient, wso2) {
         super({ captureExceptions: true });
-        this._validator = new Validate();
+        this._validator = new Validate({ logExcludePaths });
         this._conf = conf;
         this._logger = logger;
         this._server = null;

package/src/OutboundServer/middlewares.js

@@ -20,10 +20,14 @@ const { applyState, createErrorHandler, createLogger, createRequestIdGenerator }
  * @return {Function}
  */
 const createRequestValidator = (validator) => async (ctx, next) => {
-    ctx.state.logger.log('Validating request');
+    if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+        ctx.state.logger.log('Validating request');
+    }
     try {
         ctx.state.path = validator.validateRequest(ctx, ctx.state.logger);
-        ctx.state.logger.log('Request passed validation');
+        if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+            ctx.state.logger.log('Request passed validation');
+        }
         await next();
     } catch (err) {
         ctx.state.logger.push({ err }).log('Request failed validation.');

package/src/TestServer/index.js

@@ -21,6 +21,8 @@ const router = require('../lib/router');
 const handlers = require('./handlers');
 const middlewares = require('../InboundServer/middlewares');
 
+const logExcludePaths = ['/'];
+
 const getWsIp = (req) => [
     req.socket.remoteAddress,
     ...(
@@ -36,7 +38,7 @@ class TestApi {
 
         this._api.use(middlewares.createErrorHandler(logger));
         this._api.use(middlewares.createRequestIdGenerator());
-        this._api.use(middlewares.applyState({ cache }));
+        this._api.use(middlewares.applyState({ cache, logExcludePaths }));
         this._api.use(middlewares.createLogger(logger));
 
         this._api.use(middlewares.createRequestValidator(validator));
@@ -174,7 +176,7 @@ class TestServer {
     constructor({ port, logger, cache }) {
         this._port = port;
         this._logger = logger;
-        this._validator = new Validate();
+        this._validator = new Validate({ logExcludePaths });
         this._api = new TestApi(this._logger.push({ component: 'api' }), this._validator, cache);
         this._server = http.createServer(this._api.callback());
         // TODO: why does this appear to need to be called after creating this._server (try reorder

package/src/lib/metrics.js

@@ -131,8 +131,6 @@ class MetricsServer {
 
         result.use(koaBody());
         result.use(async ctx => {
-            this._logger.log('Metrics request received');
-
             ctx.response.set('Content-Type', this._prometheusRegister.contentType);
             ctx.response.body = await this._prometheusRegister.metrics();
         });

package/src/lib/router.js

@@ -21,7 +21,9 @@ module.exports = (handlerMap) => async (ctx, next) => {
         ctx.response.body = { statusCode: 404, message: 'Not found' };
     }
     else {
-        ctx.state.logger.push({ handler }).log('Found handler');
+        if (!ctx.state.logExcludePaths.includes(ctx.path)) {
+            ctx.state.logger.push({handler}).log('Found handler');
+        }
         await handler(ctx);
     }
     await next();

package/src/lib/validate.js

@@ -125,6 +125,13 @@ const transformApiDoc = apiDoc => ({
 });
 
 class Validator {
+    /**
+   * @param {{logExcludePaths: string[]}} [opts]
+   */
+
+    constructor(opts) {
+        this.logExcludePaths = opts?.logExcludePaths || [];
+    }
     // apiDoc
     //   POJO representing apiDoc API spec. Example:
     //   const v = new Validator(require('./apiDoc.json'));
@@ -163,7 +170,9 @@ class Validator {
         }
         result.params = Object.assign({}, ...path.match(result.matcher.regex).slice(1).map((m, i) => ({ [result.matcher.params[i]]: m})));
 
-        logger.push({ path, result }).log('Matched path');
+        if (!this.logExcludePaths.includes(path)) {
+            logger.push({path, result}).log('Matched path');
+        }
         return result;
     }