search

djcas9 / komanda

The IRC Client For Developers
http://komanda.io
MIT License
1.78k stars 75 forks source link

Handle uncaught exceptions #142

Open xqwzts opened 10 years ago

xqwzts commented 10 years ago

Instead of displaying the node stacktrace when an uncaught exception is thrown, handle all uncaught exceptions and display an error message to the user.

We can listen to the uncaughtException event and fail gracefully if it is emitted.

AlexKvazos commented 10 years ago

This can be good and bad at the same time. When we expect something to happen, we wont really know if it did with a simple console log. Maybe a modal or pop-up will be better to let the user know something has gone wrong. But yeah, this should be better than the entire thing crashing and having to start up again.

eugene-bulkin commented 10 years ago

We could also try the domain module, as @mephux suggested.