Do not create missing nested attributes

Author: simonratner

lib/encrypted-attr.js

@@ -33,7 +33,10 @@ function EncryptedAttributes (attributes, options) {
 
   function encryptAll (obj) {
     for (let attr of attributes) {
-      set(obj, attr, encryptAttribute(obj, get(obj, attr)))
+      let val = get(obj, attr)
+      if (val != null) {
+        set(obj, attr, encryptAttribute(obj, val))
+      }
     }
     return obj
   }
@@ -65,7 +68,10 @@ function EncryptedAttributes (attributes, options) {
 
   function decryptAll (obj) {
     for (let attr of attributes) {
-      set(obj, attr, decryptAttribute(obj, get(obj, attr)))
+      let val = get(obj, attr)
+      if (val != null) {
+        set(obj, attr, decryptAttribute(obj, val))
+      }
     }
     return obj
   }

test/encrypted-attr.spec.js

@@ -40,6 +40,13 @@ describe('encrypted attributes', function () {
     expect(obj.secret.dataLength, 'to equal', 5)
   })
 
+  it('should not create missing nested attributes', function () {
+    let enc = EncryptedAttributes(['secret.data'], this.options)
+    let obj = {id: 1}
+    enc.encryptAll(obj)
+    expect(obj.secret, 'to be undefined')
+  })
+
   it('should not encrypt null', function () {
     let enc = EncryptedAttributes(['secret'], this.options)
     let obj = {id: 1}
@@ -99,6 +106,13 @@ describe('encrypted attributes', function () {
     expect(obj.secret.dataLength, 'to equal', 5)
   })
 
+  it('should not create missing nested attributes', function () {
+    let enc = EncryptedAttributes(['secret.data'], this.options)
+    let obj = {id: 1}
+    enc.decryptAll(obj)
+    expect(obj.secret, 'to be undefined')
+  })
+
   it('should not decrypt null', function () {
     let enc = EncryptedAttributes(['secret'], this.options)
     let obj = {id: 1}