RanvierMUD / neuro

Websocket MUD client for the Ranvier MUD game engine
http://ranviermud.com
MIT License
66 stars 30 forks source link

Quitting neuro triggers uncaught exception in Ranvier websocket-networking #8

Open stennie opened 3 years ago

stennie commented 3 years ago

Expected behaviour: Quitting neuro should gracefully disconnect from Ranvier before shutting down.

Actual behaviour: Quitting neuro drops the connection and triggers an uncaught server exception due to lack of error handling in websocket-networking (which would be fixed via https://github.com/RanvierMUD/websocket-networking/pull/2).

The onus should probably be on the server in this case, but Neuro could also call disconnect for a more graceful shutdown.