use set instead of list in memorypool_update
[electrum-server.git] / processor.py
index 9fa9378..0bdb994 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,15 +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.idlock = threading.Lock()
-        self.sessions = []
+        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()
@@ -100,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.idlock:
-            return self.internal_ids.pop(internal_id)
-
-    def store_session_id(self, session, msgid):
-        with self.idlock:
-            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:
@@ -125,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):
@@ -141,9 +135,6 @@ class RequestDispatcher(threading.Thread):
             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]
@@ -163,34 +154,32 @@ class RequestDispatcher(threading.Thread):
 
     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 minimize lock contention
+    def remove_session(self, session):
+        key = session.key()
         with self.lock:
-            sessions = self.sessions[:]
-
-        active_sessions = []
-        for session in sessions:
-            if not session.stopped():
-                # If session is still alive then re-add it back
-                # to our internal register
-                active_sessions.append(session)
+            self.sessions.pop(key)
 
-        with self.lock:
-            self.sessions = active_sessions[:]
+    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 = []
@@ -201,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:
@@ -219,34 +213,62 @@ 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 (method, params) not in self.subscriptions:
+                self.subscriptions.append((method,params))
+
+
+    def stop_subscriptions(self):
+        bp = self.bp
 
-    def contains_subscription(self, subdesc):
         with self.lock:
-            return subdesc in self.subscriptions
+            s = self.subscriptions[:]
+
+        for method, params in s:
+            with bp.watch_lock:
+                if method == 'blockchain.numblocks.subscribe':
+                    if self in bp.watch_blocks:
+                        bp.watch_blocks.remove(self)
+                elif method == 'blockchain.headers.subscribe':
+                    if self in bp.watch_headers:
+                        bp.watch_headers.remove(self)
+                elif method == "blockchain.address.subscribe":
+                    addr = params[0]
+                    l = bp.watched_addresses.get(addr)
+                    if not l:
+                        continue
+                    if self in l:
+                        l.remove(self)
+                    if self in l:
+                        print "error rc!!"
+                        bp.shared.stop()
+
+                    if l == []:
+                        bp.watched_addresses.pop(addr)
+
+        with self.lock:
+            self.subscriptions = []
 
 
 class ResponseDispatcher(threading.Thread):
@@ -259,48 +281,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 )