Merge branch 'master' of github.com:spesmilo/electrum-server
[electrum-server.git] / processor.py
index baec5d0..b94231b 100644 (file)
@@ -34,21 +34,21 @@ class Processor(threading.Thread):
         self.dispatcher = None
         self.queue = queue.Queue()
 
-    def process(self, request):
+    def process(self, session, request):
         pass
 
-    def add_request(self, request):
-        self.queue.put(request)
+    def add_request(self, session, request):
+        self.queue.put((session, request))
 
-    def push_response(self, response):
+    def push_response(self, session, response):
         #print "response", response
-        self.dispatcher.request_dispatcher.push_response(response)
+        self.dispatcher.request_dispatcher.push_response(session, response)
 
     def run(self):
         while not self.shared.stopped():
-            request = self.queue.get(10000000000)
+            request, session = self.queue.get(10000000000)
             try:
-                self.process(request)
+                self.process(request, session)
             except:
                 traceback.print_exc(file=sys.stdout)
 
@@ -80,14 +80,13 @@ class RequestDispatcher(threading.Thread):
         self.daemon = True
         self.request_queue = queue.Queue()
         self.response_queue = queue.Queue()
-        self.internal_ids = {}
-        self.internal_id = 1
         self.lock = threading.Lock()
-        self.sessions = []
+        self.idlock = threading.Lock()
+        self.sessions = {}
         self.processors = {}
 
-    def push_response(self, item):
-        self.response_queue.put(item)
+    def push_response(self, session, item):
+        self.response_queue.put((session, item))
 
     def pop_response(self):
         return self.response_queue.get()
@@ -99,24 +98,16 @@ class RequestDispatcher(threading.Thread):
         return self.request_queue.get()
 
     def get_session_by_address(self, address):
-        for x in self.sessions:
+        for x in self.sessions.values():
             if x.address == address:
                 return x
 
-    def get_session_id(self, internal_id):
-        with self.lock:
-            return self.internal_ids.pop(internal_id)
-
-    def store_session_id(self, session, msgid):
-        with self.lock:
-            self.internal_ids[self.internal_id] = session, msgid
-            r = self.internal_id
-            self.internal_id += 1
-            return r
-
     def run(self):
         if self.shared is None:
             raise TypeError("self.shared not set in Processor")
+
+        lastgc = 0 
+
         while not self.shared.stopped():
             session, request = self.pop_request()
             try:
@@ -124,6 +115,10 @@ class RequestDispatcher(threading.Thread):
             except:
                 traceback.print_exc(file=sys.stdout)
 
+            if time.time() - lastgc > 60.0:
+                self.collect_garbage()
+                lastgc = time.time()
+
         self.stop()
 
     def stop(self):
@@ -137,13 +132,9 @@ class RequestDispatcher(threading.Thread):
         suffix = method.split('.')[-1]
 
         if session is not None:
-            is_new = session.protocol_version >= 0.5
             if suffix == 'subscribe':
                 session.subscribe_to_service(method, params)
 
-        # store session and id locally
-        request['id'] = self.store_session_id(session, request['id'])
-
         prefix = request['method'].split('.')[0]
         try:
             p = self.processors[prefix]
@@ -151,7 +142,7 @@ class RequestDispatcher(threading.Thread):
             print_log("error: no processor for", prefix)
             return
 
-        p.add_request(request)
+        p.add_request(session, request)
 
         if method in ['server.version']:
             session.version = params[0]
@@ -160,35 +151,35 @@ class RequestDispatcher(threading.Thread):
             except:
                 pass
 
-            #if session.protocol_version < 0.6:
-            #    print_log("stopping session from old client", session.protocol_version)
-            #    session.stop()
 
     def get_sessions(self):
         with self.lock:
-            r = self.sessions[:]
+            r = self.sessions.values()
         return r
 
     def add_session(self, session):
+        key = session.key()
         with self.lock:
-            self.sessions.append(session)
+            self.sessions[key] = session
 
-    def collect_garbage(self):
-        # Deep copy entire sessions list and blank it
-        # This is done to minimise lock contention
+    def remove_session(self, session):
+        key = session.key()
         with self.lock:
-            sessions = self.sessions[:]
-            self.sessions = []
-        for session in sessions:
-            if not session.stopped():
-                # If session is still alive then re-add it back
-                # to our internal register
-                self.add_session(session)
+            self.sessions.pop(key)
+
+    def collect_garbage(self):
+        now = time.time()
+        for session in self.sessions.values():
+            if (now - session.time) > session.timeout:
+                session.stop()
+
 
 
 class Session:
 
-    def __init__(self):
+    def __init__(self, dispatcher):
+        self.dispatcher = dispatcher
+        self.bp = self.dispatcher.processors['blockchain']
         self._stopped = False
         self.lock = threading.Lock()
         self.subscriptions = []
@@ -199,6 +190,11 @@ class Session:
         self.time = time.time()
         threading.Timer(2, self.info).start()
 
+
+    def key(self):
+        return self.name + self.address
+
+
     # Debugging method. Doesn't need to be threadsafe.
     def info(self):
         for sub in self.subscriptions:
@@ -217,34 +213,42 @@ class Session:
                       "%3d" % len(self.subscriptions),
                       self.version)
 
+    def stop(self):
+        with self.lock:
+            if self._stopped:
+                return
+            self._stopped = True
+
+        self.shutdown()
+        self.dispatcher.remove_session(self)
+        self.stop_subscriptions()
+
+
+    def shutdown(self):
+        pass
+
+
     def stopped(self):
         with self.lock:
             return self._stopped
 
+
     def subscribe_to_service(self, method, params):
-        subdesc = self.build_subdesc(method, params)
         with self.lock:
-            if subdesc is not None:
-                self.subscriptions.append(subdesc)
-
-    # subdesc = A subscription description
-    @staticmethod
-    def build_subdesc(method, params):
-        if method == "blockchain.numblocks.subscribe":
-            return method,
-        elif method == "blockchain.headers.subscribe":
-            return method,
-        elif method in ["blockchain.address.subscribe"]:
-            if not params:
-                return None
-            else:
-                return method, params[0]
-        else:
-            return None
+            if self._stopped:
+                return
+            if (method, params) not in self.subscriptions:
+                self.subscriptions.append((method,params))
+        self.bp.do_subscribe(method, params, self)
 
-    def contains_subscription(self, subdesc):
+
+    def stop_subscriptions(self):
+        with self.lock:
+            s = self.subscriptions[:]
+        for method, params in s:
+            self.bp.do_unsubscribe(method, params, self)
         with self.lock:
-            return subdesc in self.subscriptions
+            self.subscriptions = []
 
 
 class ResponseDispatcher(threading.Thread):
@@ -257,48 +261,5 @@ class ResponseDispatcher(threading.Thread):
 
     def run(self):
         while not self.shared.stopped():
-            self.update()
-
-    def update(self):
-        response = self.request_dispatcher.pop_response()
-        #print "pop response", response
-        internal_id = response.get('id')
-        method = response.get('method')
-        params = response.get('params')
-
-        # A notification
-        if internal_id is None:  # and method is not None and params is not None:
-            found = self.notification(method, params, response)
-            if not found and method == 'blockchain.address.subscribe':
-                request = {
-                    'id': None,
-                    'method': method.replace('.subscribe', '.unsubscribe'),
-                    'params': [self.shared.config.get('server', 'password')] + params,
-                }
-
-                self.request_dispatcher.push_request(None, request)
-        # A response
-        elif internal_id is not None:
-            self.send_response(internal_id, response)
-        else:
-            print_log("no method", response)
-
-    def notification(self, method, params, response):
-        subdesc = Session.build_subdesc(method, params)
-        found = False
-        for session in self.request_dispatcher.sessions:
-            if session.stopped():
-                continue
-            if session.contains_subscription(subdesc):
-                session.send_response(response)
-                found = True
-        # if not found: print_log("no subscriber for", subdesc)
-        return found
-
-    def send_response(self, internal_id, response):
-        session, message_id = self.request_dispatcher.get_session_id(internal_id)
-        if session:
-            response['id'] = message_id
+            session, response = self.request_dispatcher.pop_response()
             session.send_response(response)
-        #else:
-        #    print_log("send_response: no session", message_id, internal_id, response )