Merge pull request #7 from Mastercard/feature/fix_multiple_encryption_paths

Feature/fix multiple encryption paths

Author: ech0s7r

lib/mcapi/fle/field-level-encryption.js

@@ -134,13 +134,15 @@ function decrypt(response) {
  */
 function encryptBody(path, body) {
   let elem = elemFromPath(path.element, body);
-  let encryptedData = this.crypto.encryptData({data: elem.node});
-  utils.mutateObjectProperty(path.obj,
-    encryptedData,
-    body);
-  // delete encrypted field if not overridden
-  if (path.element !== path.obj + "." + this.config.encryptedValueFieldName) {
-    utils.deleteNode(path.element, body);
+  if (elem && elem.node){
+    let encryptedData = this.crypto.encryptData({data: elem.node});
+    utils.mutateObjectProperty(path.obj,
+      encryptedData,
+      body);
+    // delete encrypted field if not overridden
+    if (path.element !== path.obj + "." + this.config.encryptedValueFieldName) {
+      utils.deleteNode(path.element, body);
+    }
   }
 }
 

test/payload-encryption.test.js

@@ -58,6 +58,25 @@ describe("Payload encryption", () => {
       assert.ok(body.encryptedItems.oaepHashingAlgorithm);
     });
 
+    it("elem not found", () => {
+      let body = {
+        itemsToEncrypt: {
+          first: "first",
+          second: "second"
+        },
+        dontEncrypt: {
+          text: "just text..."
+        }
+      };
+      encryptBody.call(fle, {
+        element: "not.found",
+        obj: "encryptedItems"
+      }, body);
+      assert.ok(body.dontEncrypt);
+      assert.ok(body.itemsToEncrypt);
+      assert.ok(!body.encryptedItems);
+    });
+
     it("nested object to encrypt", () => {
       let body = {
         path: {