▲ | mdaniel 12 hours ago | |
I tried booting it up and two things: - this is just evil. Pure. evil. https://github.com/colanode/colanode/blob/v0.1.3/apps/deskto... If that's the kind of error handling that you believe in, one should have religious backups of any data placed into this - It seems to actually puke if one doesn't provide it a live, TLS enabled, SMTP server[2] which (a) WTF (b) isn't present in the docker-compose Thankfully replacing .verify with return new Promise(() => true) at least let the server start 2: https://github.com/colanode/colanode/blob/v0.1.3/apps/server... | ||
▲ | hakanshehu 3 hours ago | parent [-] | |
Thank you for taking the time to test it and call these issues out. Both points slipped through our refactor/cleanup checklist. - We’ll replace the current error handling for server sync with something safer and more graceful. - We’ll make SMTP optional, expose TLS verification as a configurable setting and update the docker-compose. We’ll make these improvements soon, thanks again for the heads-up. |