@itentialopensource/adapter-utils 4.44.11 → 4.45.3

package/CHANGELOG.md

@@ -1,4 +1,42 @@
 
+## 4.45.3 [12-13-2021]
+
+* move preinstall to postinstall
+
+See merge request itentialopensource/adapter-utils!229
+
+---
+
+## 4.45.2 [12-09-2021]
+
+* change for sending content-length with empty token body
+
+Closes ADAPT-1889
+
+See merge request itentialopensource/adapter-utils!228
+
+---
+
+## 4.45.1 [12-08-2021]
+
+* Modified response handling on error to use the translator
+
+Closes ADAPT-1876
+
+See merge request itentialopensource/adapter-utils!225
+
+---
+
+## 4.45.0 [12-08-2021]
+
+* changes for uriOptions and not encoding query params
+
+Closes ADAPT-1866
+
+See merge request itentialopensource/adapter-utils!227
+
+---
+
 ## 4.44.11 [10-25-2021]
 
 * change to the npm registry url

package/lib/connectorRest.js

@@ -1760,6 +1760,20 @@ function buildTokenRequest(reqPath, reqBody, callProperties, callback) {
         }
       }
 
+      // only add global options if there are global options to add
+      if (globalRequest && globalRequest.uriOptions
+          && Object.keys(globalRequest.uriOptions).length > 0) {
+        const optionString = querystring.stringify(globalRequest.uriOptions);
+
+        // if no query paramters yet - start with ?
+        if (options.path.indexOf('?') < 0) {
+          options.path += `?${optionString}`;
+        } else {
+          // if already have query parameters, add on to end
+          options.path += `&${optionString}`;
+        }
+      }
+
       // remove the path vars from the reqBody
       const actReqBody = Object.assign({}, reqBody);
       if (actReqBody && actReqBody.uriPathVars) {
@@ -1917,7 +1931,7 @@ function buildTokenRequest(reqPath, reqBody, callProperties, callback) {
 
           // if there is a body, set the content length of the body and add it to
           // the header
-          if (Object.keys(tokenEntity).length > 0) {
+          if (Object.keys(tokenEntity).length > 0 || tokenSchema.sendEmpty) {
             options.headers['Content-length'] = Buffer.byteLength(bodyString);
           }
 

package/lib/propertyUtil.js

@@ -187,6 +187,7 @@ class AdapterPropertyUtil {
       // get the path for the specific schema file
       const reqSchemaFile = path.join(this.baseDir, `/entities/${entityName}/${reqSchemaName}`);
       const respSchemaFile = path.join(this.baseDir, `/entities/${entityName}/${respSchemaName}`);
+      const errorSchemaFile = path.join(this.baseDir, '/entities/.system/errorSchema');
 
       // if the file does not exist - error
       if (!fs.existsSync(reqSchemaFile)) {
@@ -244,6 +245,16 @@ class AdapterPropertyUtil {
         throw new Error(JSON.stringify(errorObj));
       }
 
+      // if the error schema file exist - read it in
+      if (fs.existsSync(errorSchemaFile)) {
+        entitySchema.errorSchema = JSON.parse(fs.readFileSync(errorSchemaFile, 'utf-8'));
+
+        // if the error schema file is bad, warn about it but continue and use the global on!
+        if (entitySchema.errorSchema && typeof entitySchema.errorSchema !== 'object') {
+          log.warn(`${origin}: Invalid error schema, please verify file: ${errorSchemaFile}`);
+        }
+      }
+
       // Merge the information into the entity schema
       entitySchema.protocol = actionInfo.protocol;
       entitySchema.method = actionInfo.method;

package/lib/requestHandler.js

@@ -51,7 +51,7 @@ function validateProperties(properties) {
 
   try {
     // get the path for the specific action file
-    const propertyFile = path.join(__dirname, '/../propertiesSchema.json');
+    const propertyFile = path.join(__dirname, '/../schemas/propertiesSchema.json');
 
     // Read the action from the file system
     const propertySchema = JSON.parse(fs.readFileSync(propertyFile, 'utf-8'));
@@ -113,7 +113,7 @@ function walkThroughActionFiles(directory) {
 
   try {
     // Read the action schema from the file system
-    const actionSchemaFile = path.join(__dirname, '/../actionSchema.json');
+    const actionSchemaFile = path.join(__dirname, '/../schemas/actionSchema.json');
     const actionSchema = JSON.parse(fs.readFileSync(actionSchemaFile, 'utf-8'));
     const entitydir = `${directory}/entities`;
 

package/lib/restHandler.js

@@ -9,6 +9,8 @@ const jsonQuery = require('json-query');
 const jsonxml = require('jsontoxml');
 const xml2js = require('xml2js');
 
+const globalSchema = JSON.parse(require('fs').readFileSync(require('path').join(__dirname, '/../schemas/globalSchema.json')));
+
 let transUtilInst = null;
 let connectorInst = null;
 
@@ -19,6 +21,7 @@ let basepathGl = null;
 let globalRequestGl = null;
 let returnRawGl = false;
 let encodePath = true;
+let encodeUri = true;
 
 // INTERNAL FUNCTIONS
 /*
@@ -178,7 +181,13 @@ function handleRestRequest(request, entityId, entitySchema, callProperties, filt
 
         // if the return error message was JSON then return the parsed object
         if (retError !== null) {
-          retErrorObj.response = retError;
+          // if there is a local error schema in the entity use that one
+          if (Object.hasOwnProperty.call(entitySchema, 'errorSchema')) {
+            retErrorObj.response = transUtilInst.mapFromOutboundEntity(retError, entitySchema.errorSchema);
+          } else {
+            // if there is a no local error schema in the entity use that one
+            retErrorObj.response = transUtilInst.mapFromOutboundEntity(retError, globalSchema);
+          }
         }
 
         // return the error response
@@ -460,6 +469,9 @@ function handleRestRequest(request, entityId, entitySchema, callProperties, filt
         return callback(retObject);
       }
 
+      // Apply global schema
+      // retResponse = transUtilInst.mapFromOutboundEntity(retResponse, globalSchema);
+
       // added the translated response to the return Object
       retObject.response = transUtilInst.mapFromOutboundEntity(retResponse, entitySchema.responseSchema);
 
@@ -677,7 +689,23 @@ function buildRequestPath(entity, action, entitySchema, reqPath, uriPathVars, ur
           addquery = entitySchema.querykey;
         }
         if (systemQuery !== null) {
-          addquery += querystring.stringify(systemQuery);
+          // if we are encoding - use querystring since it does it all!
+          if (encodeUri === true) {
+            addquery += querystring.stringify(systemQuery);
+          } else {
+            // if not encoding we need to build
+            const qkeys = Object.keys(systemQuery);
+
+            // add each query parameter and its value
+            for (let k = 0; k < qkeys.length; k += 1) {
+              // need to add separator for everything after the first one
+              if (k > 0) {
+                addquery += '&';
+              }
+              // adds key=value
+              addquery += `${qkeys[k]}=${systemQuery[qkeys[k]]}`;
+            }
+          }
         }
       }
 
@@ -988,6 +1016,8 @@ class RestHandler {
     this.globalRequest = null;
     this.encode = properties.encode_pathvars;
     encodePath = this.encode;
+    this.encodeQ = properties.encode_queryvars;
+    encodeUri = this.encodeQ;
 
     // only need to set returnRaw if the property is true - defaults to false
     if (properties.request && properties.request.return_raw) {

package/package.json

@@ -1,9 +1,9 @@
 {
   "name": "@itentialopensource/adapter-utils",
-  "version": "4.44.11",
+  "version": "4.45.3",
   "description": "Itential Adapter Utility Libraries",
   "scripts": {
-    "preinstall": "node utils/setup.js",
+    "postinstall": "node utils/setup.js",
     "lint": "eslint . --ext .json --ext .js",
     "lint:errors": "eslint --quiet . --ext .json --ext .js",
     "test:unit": "mocha test/unit/lib/requestHandlerTest.js --LOG=error && mocha test/unit/lib/restHandlerTest.js --LOG=error && mocha test/unit/lib/propertyUtilTest.js --LOG=error && mocha test/unit/lib/translatorUtilTest.js --LOG=error && mocha test/unit/lib/dbUtilTest.js --LOG=debug",

package/schemas/globalSchema.json

@@ -0,0 +1,19 @@
+{
+    "$id": "schema.json",
+    "type": "object",
+    "schema": "http://json-schema.org/draft-07/schema#",
+    "translate": true,
+    "dynamicfields": true,
+    "properties": {
+      "id": {
+        "type": "string",
+        "description": "Restricted for: special use within mongo",
+        "encrypt": {
+          "type": "AES",
+          "key": ""
+        },
+        "external_name": "$id"
+      }
+    },
+    "definitions": {}
+  }

package/{propertiesSchema.json → schemas/propertiesSchema.json}

@@ -51,6 +51,11 @@
       "description": "When true the path variables are encoded in the url",
       "default": true
     },
+    "encode_queryvars": {
+      "type": "boolean",
+      "description": "When true the query variables are encoded in the url",
+      "default": true
+    },
     "save_metric": {
       "type": [
         "boolean",