More robust connection failure detection. There should be absolutely no more hanged clients.

Author: AMorporkian

server.py

@@ -441,6 +441,8 @@ def die(self):
         except AttributeError:
             pass
 
+    def connectionFailed(self, *args, **kwargs):
+        self.connectionLost()
 
 class ClientProtocol(Protocol):
     """
@@ -503,7 +505,6 @@ def dataReceived(self, data):
         else:
             self.packet_stream += data
 
-
 class StarryPyServerFactory(ServerFactory):
     """
     Factory which creates `StarryPyServerProtocol` instances.
@@ -572,9 +573,9 @@ def reap_dead_protocols(self):
                 protocol.connectionLost()
                 count += 1
         if count == 1:
-            logger.debug("1 connection reaped.")
+            logger.info("1 connection reaped.")
         elif count > 1:
-            logger.debug("%d connections reaped.")
+            logger.info("%d connections reaped.")
         else:
             logger.debug("No connections reaped.")
 
@@ -622,8 +623,6 @@ def buildProtocol(self, address):
             config.upstream_hostname, config.upstream_port))
             logger.critical(
                 "Please ensure that you are running starbound_server on the correct port and that is reflected in the StarryPy configuration.")
-            sock.shutdown()
-            sock.close()
             sys.exit()
         sock.shutdown(SHUT_RDWR)
         sock.close()