check server connections

[far0ouk]

if server offline check connection every 5 sec

[Rantanen]

Few comments on an initial glance:

• setInterval handle isn't stored so those intervals are leaked.
• On connect + close the Promise gets both its resolve and reject callbacks triggered, isn't this against the Promise contract?
• Quite few console.log invocations that are forced to the application console even if the application does not want these.
• It seems that a user triggered disconnect would end up propagating close event, which then results in a reconnect when the user intended to disconnect.
• I don't see how the error-callback is triggered on initial connection anymore.

On function naming

• Despite the name, reconnect might not end up reconnecting.
• Despite the name, getSocket is not a simple getter but performs reconnection as well.

The disconnect-detection should be built into the MumbleConnection class, which already has an interval for ping implementation, the same interval should be used for detecting dead sockets. That interval is handled properly in regards to closing the connection.