feat(sync-v2): Watchdog to detect stale syncing

Author: msbrogli

hathor/p2p/sync_v2/agent.py

@@ -148,6 +148,10 @@ def __init__(self, protocol: 'HathorProtocol', reactor: Optional[Reactor] = None
         self._lc_run = LoopingCall(self.run_sync)
         self._lc_run.clock = self.reactor
         self._is_running = False
+        self._sync_started_at: float = 0
+
+        # Maximum running time to consider a sync stale.
+        self.max_running_time: int = 30 * 60  # seconds
 
         # Whether we propagate transactions or not
         self._is_relaying = False
@@ -262,6 +266,16 @@ def handle_error(self, payload: str) -> None:
     def update_synced(self, synced: bool) -> None:
         self._synced = synced
 
+    def watchdog(self) -> None:
+        """Close connection if sync is stale."""
+        if not self._is_running:
+            return
+
+        dt = self.reactor.seconds() - self._sync_started_at
+        if dt > self.max_running_time:
+            self.log.warn('stale syncing detected, closing connection')
+            self.protocol.send_error_and_close_connection('stale syncing')
+
     @inlineCallbacks
     def run_sync(self) -> Generator[Any, Any, None]:
         """ Async step of the sync algorithm.
@@ -274,8 +288,10 @@ def run_sync(self) -> Generator[Any, Any, None]:
         if self._is_running:
             # Already running...
             self.log.debug('already running')
+            self.watchdog()
             return
         self._is_running = True
+        self._sync_started_at = self.reactor.seconds()
         try:
             yield self._run_sync()
         except Exception: