Merge pull request #19 from pinojs/handle-undefined-fields

Do not crash with a property that has value undefined

index.js

@@ -196,6 +196,10 @@ module.exports = function prettyFactory (options) {
     return line
 
     function joinLinesWithIndentation (value) {
+      if (!value) {
+        return
+      }
+
       const lines = value.split(/\r?\n/)
       for (var i = 1; i < lines.length; i++) {
         lines[i] = IDENT + lines[i]

test/basic.test.js

@@ -22,7 +22,6 @@ test('basic prettifier tests', (t) => {
   t.afterEach((done) => {
     Date.now = Date.originalNow
     delete Date.originalNow
-
     done()
   })
 
@@ -433,5 +432,25 @@ test('basic prettifier tests', (t) => {
     t.is(formatted, `[${epoch}] INFO: hello world\n`)
   })
 
+  t.test('formats a line with an undefined field', (t) => {
+    t.plan(1)
+    const pretty = prettyFactory()
+    const log = pino({}, new Writable({
+      write (chunk, enc, cb) {
+        const obj = JSON.parse(chunk.toString())
+        // weird hack, but we should not crash
+        obj.a = undefined
+        const formatted = pretty(obj)
+        t.is(
+          formatted,
+          `[${epoch}] INFO (${pid} on ${hostname}): foo
+    a: undefined\n`
+        )
+        cb()
+      }
+    }))
+    log.info('foo')
+  })
+
   t.end()
 })