pino - npm Package Compare versions

Comparing version 2.12.0 to 2.12.1

package.json

 {
   "name": "pino",
-  "version": "2.12.0",
+  "version": "2.12.1",
   "description": "fast and simple logger",
@@ -5,0 +5,0 @@ "main": "pino.js",

pino.js

@@ -241,5 +241,5 @@ 'use strict'
         if (obj.hasOwnProperty(key) && value !== undefined) {
-          value = this.serializers[key] ? this.serializers[key](value) : value
+          value = this.stringify(this.serializers[key] ? this.serializers[key](value) : value)
           if (value !== undefined) {
-            data += ',"' + key + '":' + this.stringify(value)
+            data += ',"' + key + '":' + value
           }
@@ -246,0 +246,0 @@ }

test/basic.test.js

@@ -295,2 +295,15 @@ 'use strict'
 
+test('correctly strip undefined when returned from toJSON', function (t) {
+  t.plan(1)
+
+  var instance = pino({
+    test: 'this'
+  }, sink(function (obj, enc, cb) {
+    t.notOk('test' in obj)
+    cb()
+  }))
+
+  instance.fatal({test: {toJSON: function () { return undefined }}})
+})
+
 test('correctly support node v4+ stderr', function (t) {
@@ -297,0 +310,0 @@ t.plan(1)

New alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Fixed alerts

License Policy Violation

License

This package is not allowed per your license policy. Review the package's license to ensure compliance.

Found 1 instance in 1 package

Improved metrics

Total package byte prevSize

588577

increased by0.05%

Lines of code

2251

increased by0.45%

No dependency changes