FileTransport not compatible with Node 0.10

[akoenig]

I'm not sure but I think that the new Streams2 API in Node 0.10.* has an impact on winston. If I try to log with the FileTransport it only writes one statement into the log file and stops then.

Means, every following logging transaction doesn't appear in the log file.

An alternative would be to use an own WriteableStream and push it into the "stream" property, but letting winston do that by defining a filename doesn't work.

Works in

• 0.8.22

Doesn't work in

• 0.10.0
• 0.10.1

I created a little test case: winston-streams2.js - Checked it with the latest version of winston (0.6.2).

[mahnunchik]

I also faced with this error.
This happens because the event 'drain' is not called.
This is article about changes in stream2 http://blog.nodejs.org/2012/12/21/streams2/

[1o1brian]

+1 I'd love to see this fixed so I could move on to node 0.10.x. Thanks.

[dhendo]

+1

[akoenig]

Fixed in #229. Thanks for merging :)

[1o1brian]

Can we get an npm release or am I missing something?

Thanks.

[vlamic]

Any news on when npm release will be available?

[nicokruger]

+1 on this

[3rd-Eden]

There are still a couple of tests failing on 0.10.x after these are resolved, there will be another release.

[victoruiz]

I'm testing with node 0.10.21 and seeing some tests still failing.

For instance:

✗ Errored » Asynchronous Error
in winston/transports/daily-rotate-file
in test/transports/daily-rotate-file-test.js
✗ Errored » callback not fired
in An instance of the Daily Rotate File Transport the stream() method using no options
in winston/transports/daily-rotate-file
in test/transports/daily-rotate-file-test.js

The final result of "npm test" was: ✗ Errored » 122 honored ∙ 3 errored

Sorry for not giving further details on what is specifically crashing in there, just wanted to mention this in case it's helpful.

[davis]

is this working yet?

[pose]

Hi @davis, have you tried with winston@0.8.0?