Improve reliability of SSL connections
authorMichael Marley <michael@michaelmarley.com>
Wed, 11 Dec 2013 21:15:58 +0000 (21:15 +0000)
committerManuel Nickschas <sputnick@quassel-irc.org>
Mon, 13 Jan 2014 23:13:02 +0000 (00:13 +0100)
commit2cf99193317f0c970422f6be61ac6efa658b0006
treeebc87cba7e1fcb5ec192eb8a71155cf711562141
parentd39da7e4336117bc7a87056dd7159a6e0ee1767c
Improve reliability of SSL connections

First, this fixes a bug similar to Bug #1249 except for SSL
connections.  If the TCP connection fails but isn't
actually closed before the SSL handshake is complete,
Quassel will become stuck and never attempt to reconnect.
The solution is to start the ping timeout even earlier,
before the connection is established at all.

Second, this fixes the issue where multiple presses of the
"Disconnect" button and long waits were required to close
certain broken SSL connections (where SSL negotiation was
not yet complete, as above).  The solution is to call
socket.abort() instead of socket.disconnectFromHost(),
which will ensure that the socket is closed immediately.
Additionally, in case that fails for some reason, the
socketCloseTimer is no longer a single-shot timer, so
if the first abort doesn't work, it will keep trying.

(Probably?) fixes bug #1005
src/core/corenetwork.cpp
src/core/corenetwork.h