Merge pull request #11 from AntonBazhal/deep-merge

Fixed bug with deep nested objects
AntonBazhal · Mar 6, 2017 · 33bd3e9 · 33bd3e9
2 parents cd7fe9b + f5e2fb8
commit 33bd3e9
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 3 deletions.
diff --git a/lib/handler.js b/lib/handler.js
@@ -78,7 +78,13 @@ module.exports = function(options) {
           }
 
           acc.meta.push({
-            event: _.merge({}, record, { dynamodb: parsedRecord }),
+            event: _.assign(
+              {},
+              record,
+              {
+                dynamodb: _.assign({}, record.dynamodb, parsedRecord)
+              }
+            ),
             action,
             document: doc
           });

diff --git a/package.json b/package.json
@@ -1,6 +1,6 @@
 {
   "name": "dynamo2es-lambda",
-  "version": "1.0.3",
+  "version": "1.0.4",
   "description": "Configurable AWS Lambda handler to index documents from DynamoDB Streams in Amazon Elasticsearch",
   "repository": {
     "type": "git",

diff --git a/test/handler-tests.js b/test/handler-tests.js
@@ -166,7 +166,7 @@ describe('handler', function() {
 
       let hookCalled = false;
       const testItemKeys = { id: uuid.v4() };
-      const testItemData = { data: 'some data' };
+      const testItemData = { data: 'some data', nestedData: { data: 'nested data' } };
       const testEvent = formatEvent({
         name: 'INSERT',
         keys: testItemKeys,