process.stdout.write does not output to DevTools console?

[krohrsb]

The app i was testing with uses bunyan which logs to the console. However when running with iron-node the console/stdout is suppressed for some reason.

[s-a]

Did you checked the devtools console?

[krohrsb]

a normal console.log in the app does output to the devtools console. However using bunyan with (bunyan-format)[https://github.com/thlorenz/bunyan-format] as the output stream does not. Though of course bunyan normally does log the output.

[s-a]

Not very beautiful but I got output to terminal console

I have noticed some other things regarding to that as described at electron/electron#2390

[s-a]

@BlackBarn do you see any output like on my console? Otherweise I would like to change the issue title.

[s-a]

My guess is that the isbrowser method does not work correct within electron environment.

https://github.com/trentm/node-bunyan/blob/master/lib/bunyan.js#L407 can you clarify this?

[krohrsb]

Sorry, I was preoccupied there for awhile. I'm glad it seems to be working and sorry for causing any problems. When I have time i'll dig into my implementation a bit more to see if I can find my issue. Thanks for all your work on the project!

[s-a]

Absolutely no problem, feel free to submit issues! Well I am not shure it isjust a guess. So if you want. Would be nice to keep us up-to-date here.

[Bargs]

I'm having the same issue. Logs that are piped to process.stdout do not show up in my terminal or Devtools console. I do successfully get logs when using the regular node process. Any ideas what might be going on?

[s-a]

Can you publish a reproducable code example?

[Bargs]

If you clone the Kibana repo (https://github.com/elastic/kibana) and run node ./src/cli you'll see some log output. If you run iron-node ./src/cli there will be no log output in the terminal or Devtools console.

Note that these logs are ultimately getting sent to process.stdout in ./src/server/logging/LogReporter.js

[s-a]

Ty, I will check this out this weekend .
Am 12.11.2015 00:12 schrieb "Matt Bargar" notifications@github.com:

If you clone the Kibana repo (https://github.com/elastic/kibana) and run node
./src/cli you'll see some log output. If you run iron-node ./src/cli
there will be no log output in the terminal or Devtools console.

—
Reply to this email directly or view it on GitHub
#30 (comment).

[s-a]

Sir @BlackBarn I will not find time this we again :\ pls can u try #60 (comment). there where a lot of improvements by the creators of Electron regarding console log. Hope this will help u. You can try to upgrade electron within iron -node

[Bargs]

Hi @s-a , I tried upgrading Electron, unfortunately it didn't make a difference.

[s-a]

@Bargs ty for the attempt. Could you please open an issue at the electron repo?

[Bargs]

Yes I'll try to get to that this weekend. Thanks!

[s-a]

@Bargs I plan a new release this weekend. Any news about this topic?

[Bargs]

Hi @s-a. I have to admit this slipped my mind. I haven't been working with iron node much recently, if I get back into it I'll make sure to follow up.

[willhoag]

Hi! I'm having the same issue. Any updates on this? I'm happy to make an issue on the electron repo, but haven't used electron and wouldn't know how to properly describe what we think is happening.

[s-a]

I believe you experience this issue when using console output log streams. I never used this. But it seems you have reproducable code there?

[willhoag]

Yeah, when using streams process.stdout like this:

process.stdout.write("doesn't work")
console.log('works')

[s-a]

Ok. Could you plz send an issue to the creator s of electron?

[willhoag]

Per electron/electron#3979 (comment) to get output in the console via process.stdout.write it sounds like this ought to do it:

process.stdout.write = console.log.bind(console);

Can you confirm?

[s-a]

@willhoag thanks for clarify! Yes it sounds very reasonable. You can test if this works when you put this code into your startup script. I will implement the code into iron-node core if this works for you. I would test it but realy never used process.stdout.write.

[s-a]

just tested this with kibana by @Bargs 👍

Will publish this pice of code with the next release. Ty @willhoag

[s-a]

Please use https://github.com/s-a/iron-node/releases/tag/v2.1.4 for further testings.

[willhoag]

Confirming that the update produces output using process.stdout.write in the console. Thx!