-
Notifications
You must be signed in to change notification settings - Fork 314
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: add Winston logging #780
Conversation
Also, some off-top thing: with |
hey @tombeynon did you have a chance to look into it? |
@freak12techno so sorry for the delay. This is epic, thanks so much! I'm going to test it on my install for a day then we'll get it released. On your other comments - totally agree, both about .mjs and Typescript. In all honesty this needs a complete re-write, I could explain the decisions/history that led to the current implementation but it's all excuses 😂 It's not a particularly complicated script especially if we abstract the signing logic, there's just a lot of technical debt in this version from the initial build. In the meantime these changes are perfect and we'll use this logging implementation in the rewrite whenever it happens. |
This is working great @freak12techno - I'm going to merge and release now. This is an awesome base for the new logging implementation, once it's merged we can take a look at some of the logs and see if they can be improved and are tagged correctly. Thanks so much for all your work on this. Let me know if there's anything else you want to tackle, or raise an issues you want me to look at. |
I'm all in for rewriting it in TS, but that'd require more work lol. (Seems like it'll totally benefit everybody though) |
Is there anything I can do in the meantime to make that easier? E.g updating the node packages, maybe changing those .mjs files? |
I think no. Can you create an issue on that and assign me to it? |
Fixes #779.
timeStamp()
functiontimeStamp()
occurrences with Winston callshere's how it looks like when I build and start it:


I already see some logs that do not make sense here (like this healthcheck start emoji and
...batch
(as it's unclear what this batch is referring to), but not sure how to better update it.Any suggestions on how to make it even better are welcome.
@tombeynon mind taking a look?