From ef276e8770e19c66d14462aa325b9cb241121bb6 Mon Sep 17 00:00:00 2001
From: Erik Johnston <erik@matrix.org>
Date: Wed, 11 Feb 2015 16:48:05 +0000
Subject: [PATCH] Fix so timing out connections to actually work.

---
 synapse/federation/replication.py |  2 ++
 synapse/util/__init__.py          | 52 +++++++++++++++++++++++++------
 2 files changed, 45 insertions(+), 9 deletions(-)

diff --git a/synapse/federation/replication.py b/synapse/federation/replication.py
index e442c6c5d..54a0c7ad8 100644
--- a/synapse/federation/replication.py
+++ b/synapse/federation/replication.py
@@ -72,5 +72,7 @@ class ReplicationLayer(FederationClient, FederationServer):
 
         self._order = 0
 
+        self.hs = hs
+
     def __str__(self):
         return "<ReplicationLayer(%s)>" % self.server_name
diff --git a/synapse/util/__init__.py b/synapse/util/__init__.py
index 29f1344c5..e77eba90a 100644
--- a/synapse/util/__init__.py
+++ b/synapse/util/__init__.py
@@ -18,6 +18,9 @@ from synapse.util.logcontext import LoggingContext
 from twisted.internet import defer, reactor, task
 
 import time
+import logging
+
+logger = logging.getLogger(__name__)
 
 
 class Clock(object):
@@ -55,20 +58,51 @@ class Clock(object):
         timer.cancel()
 
     def time_bound_deferred(self, given_deferred, time_out):
+        if given_deferred.called:
+            return given_deferred
+
         ret_deferred = defer.Deferred()
 
-        def timed_out():
-            if not given_deferred.called:
-                given_deferred.cancel()
+        def timed_out_fn():
+            try:
                 ret_deferred.errback(RuntimeError("Timed out"))
+            except:
+                pass
 
-        timer = self.call_later(time_out, timed_out)
+            try:
+                given_deferred.cancel()
+            except:
+                pass
 
-        def succeed(result):
-            self.cancel_call_later(timer)
-            ret_deferred.callback(result)
+        timer = None
 
-        given_deferred.addCallback(succeed)
-        given_deferred.addErrback(ret_deferred.errback)
+        def cancel(res):
+            try:
+                self.cancel_call_later(timer)
+            except:
+                pass
+            return res
+
+        ret_deferred.addBoth(cancel)
+
+        def sucess(res):
+            try:
+                ret_deferred.callback(res)
+            except:
+                pass
+
+            return res
+
+        def err(res):
+            try:
+                ret_deferred.errback(res)
+            except:
+                pass
+
+            return res
+
+        given_deferred.addCallbacks(callback=sucess, errback=err)
+
+        timer = self.call_later(time_out, timed_out_fn)
 
         return ret_deferred