Review stderr on hopr-chat handler to allow pipeing out errors into a different stream/output/log

hoprnet / hopr-core

Main repository of HOPR - this is all you need to run a node on the HOPR network!

https://hoprnet.org

GNU General Public License v3.0

14 stars 5 forks source link

Review stderr on hopr-chat handler to allow pipeing out errors into a different stream/output/log #123

Closed 0xjjpa closed 4 years ago

0xjjpa commented 4 years ago

[ ] When error occurs in HOPR-chat, we pipe it out to a different log-file or stream to ensure it doesn't go to stdout as normal outputs.

0xjjpa commented 4 years ago

Some update here, seems like neither process.on('error') nor domain is helping us catch UnhandledPromiseRejectionWarning errors, which also seem to be more common than we think. Looking for alternatives (i.e. piping out stderr somehow).

0xjjpa commented 4 years ago

Ended up pipeing everything to an external file, which so far is working great. See changes in https://github.com/hoprnet/hopr-core/pull/139, let's keep it like this for now.