Merge pull request #557 from MiguelNazMor/add_support_for_arrays

Add lodash cloneDeep to support arrays in omitPaths

packages/input-output-logger/__tests__/index.js

@@ -49,5 +49,23 @@ describe('📦 Middleware Input Output Logger', () => {
       expect(logger).toHaveBeenCalledWith({ event: { foo: 'bar', fuu: 'baz' } })
       expect(logger).toHaveBeenCalledWith({ response: 'yo' })
     })
+
+    test('Skipped parts should be present in the response', async () => {
+      const logger = jest.fn()
+
+      const handler = middy((event, context, cb) => {
+        cb(null, { foo: [{ foo: 'bar', fuu: 'baz' }] })
+      })
+
+      handler
+        .use(inputOutputLogger({ logger, omitPaths: ['event.zooloo', 'event.foo.hoo', 'response.foo[0].foo'] }))
+
+      const response = await invoke(handler, { foo: 'bar', fuu: 'baz' })
+
+      expect(logger).toHaveBeenCalledWith({ event: { foo: 'bar', fuu: 'baz' } })
+      expect(logger).toHaveBeenCalledWith({ response: { foo: [{ fuu: 'baz' }] } })
+
+      expect(response).toMatchObject({ foo: [{ foo: 'bar', fuu: 'baz' }] })
+    })
   })
 })

packages/input-output-logger/index.js

@@ -8,8 +8,11 @@ module.exports = (opts) => {
 
   const { logger, omitPaths } = Object.assign({}, defaults, opts)
 
+  const cloneMessage = message => JSON.parse(JSON.stringify(message))
+
   const omitAndLog = message => {
-    const redactedMessage = omit(message, omitPaths)
+    const messageClone = cloneMessage(message)
+    const redactedMessage = omit(messageClone, omitPaths)
     logger(redactedMessage)
   }