Problem with error objects in meta (regression v2.3.0 -> v2.3.1)

Hi,

we see a problem with winston@2.3.1. Here is the gist reproducing it: https://gist.github.com/OrKoN/9c5b212058563043a94fa144d460d1d4

When run with winston 2.3.1, it outputs:

{"error":{},"@timestamp":"2017-04-12T07:19:26.456Z","level":"info","message":"test error in the key"}

When run with winston 2.3.0, it outputs:

{"error":{"message":"test","stack":"Error: test\n    at Object.<anonymous> (/Users/rudenol/winston-test/index.js:29:10)\n    at Module._compile (module.js:570:32)\n    at Object.Module._extensions..js (module.js:579:10)\n    at Module.load (module.js:487:32)\n    at tryModuleLoad (module.js:446:12)\n    at Function.Module._load (module.js:438:3)\n    at Module.runMain (module.js:604:10)\n    at run (bootstrap_node.js:394:7)\n    at startup (bootstrap_node.js:149:9)\n    at bootstrap_node.js:509:3"},"@timestamp":"2017-04-12T07:20:52.922Z","level":"info","message":"test error in the key"}

The latter one is the expected behavior in my opinion. I tried to trace it down and it seems cycle.decycle removes the error object from the meta.

Any idea why this happening?

Also if we use the normal Consol transport, then the error is empty in both 2.3.0 and 2.3.1. My node version is v6.9.4.

[gunar]

Affects v2.4.0 as well

[indexzero]

#1188 should fix this. A maintenance release will be made tomorrow.