electrum

commit c7945f4199d17229ea28ab06de600decf17efb45
parent 5714d9f4fc4baeac0aad6e4b33489fb093654572
Author: ThomasV <electrumdev@gmail.com>
Date:   Thu, 11 Jun 2015 01:36:39 +0200

Merge pull request #1287 from kyuupichan/requeue

Fix request handling.
Diffstat:
M
lib/network.py
 | 
29
++++++++++++++++++++---------

1 file changed, 20 insertions(+), 9 deletions(-)
diff --git a/lib/network.py b/lib/network.py
@@ -436,14 +436,20 @@ class Network(util.DaemonThread):
             self.response_queue.put(response)
 
     def handle_requests(self):
-        while self.is_running():
-            try:
-                request = self.requests_queue.get_nowait()
-            except Queue.Empty:
-                break
-            self.process_request(request)
+        '''Some requests require connectivity, others we handle locally in
+        process_request() and must do so in order to e.g. prevent the
+        daemon seeming unresponsive.
+        '''
+        unhandled = []
+        while not self.requests_queue.empty():
+            request = self.requests_queue.get()
+            if not self.process_request(request):
+                unhandled.append(request)
+        for request in unhandled:
+            self.requests_queue.put(request)
 
     def process_request(self, request):
+        '''Returns true if the request was processed.'''
         method = request['method']
         params = request['params']
         _id = request['id']
@@ -460,18 +466,23 @@ class Network(util.DaemonThread):
                 traceback.print_exc(file=sys.stdout)
                 self.print_error("network error", str(e))
             self.response_queue.put(out)
-            return
+            return True
 
         if method == 'blockchain.address.subscribe':
             addr = params[0]
             self.subscribed_addresses.add(addr)
             if addr in self.addr_responses:
                 self.response_queue.put({'id':_id, 'result':self.addr_responses[addr]})
-                return
+                return True
+
+        # This request needs connectivity.  If we don't have an
+        # interface, we cannot process it.
+        if not self.interface:
+            return False
 
-        # store unanswered request
         self.unanswered_requests[_id] = request
         self.interface.send_request(request)
+        return True
 
     def check_interfaces(self):
         now = time.time()