diff options
author | Sheogorath | 2018-11-16 11:42:52 +0100 |
---|---|---|
committer | Sheogorath | 2018-11-16 11:49:39 +0100 |
commit | bdeb05339764d1db64f45c56860d33929b6776e9 (patch) | |
tree | 859acc8024d08bc7c63c7f2a2d22f3109038700a /lib/web/utils.js | |
parent | f1367ba2702100a420477151d90e6a0b1999ffb3 (diff) |
Fix streaming for winston
During the upgrade of winston in
c3584770f24205d84b9399abd9535cb27dc7b00c a the class extension for
streaming was removed.
This caused silent crashes. Somehow winston simply called
`process.exit(1)` whenever `logger.write()` was called. This is really
bad and only easy to debug because of the testing right after upgrading.
However, reimplementing the stream interface as it was, didn't work, due
to the fact that `logger.write()` is already implemented and causes the
mentioned problem. So we extent the object with an `stream` object that
implements `write()` for streams and pass that to morgan.
So this patch fixes unexpected exiting for streaming towards our logging
module.
References:
https://www.digitalocean.com/community/tutorials/how-to-use-winston-to-log-node-js-applications
https://github.com/hackmdio/codimd/commit/c3584770f24205d84b9399abd9535cb27dc7b00c
https://stackoverflow.com/a/28824464
Signed-off-by: Sheogorath <sheogorath@shivering-isles.com>
Diffstat (limited to '')
0 files changed, 0 insertions, 0 deletions