From 2ee8f1668871b0287fb8e70488403a854f279018 Mon Sep 17 00:00:00 2001
From: Aryan Jassal <aryan.jassal@matrix.ai>
Date: Fri, 17 Jan 2025 16:12:21 +1100
Subject: [PATCH] feat: fixed transaction conflicts in discovery

---
 src/discovery/Discovery.ts  | 1 +
 src/nodes/NodeConnection.ts | 2 +-
 2 files changed, 2 insertions(+), 1 deletion(-)

diff --git a/src/discovery/Discovery.ts b/src/discovery/Discovery.ts
index 7d32add22..a382f7419 100644
--- a/src/discovery/Discovery.ts
+++ b/src/discovery/Discovery.ts
@@ -309,6 +309,7 @@ class Discovery {
       }
     }
     await Promise.all(taskPromises);
+    this.logger.info(`Stopped all tasks for ${this.constructor.name}`);
     this.taskManager.deregisterHandler(this.discoverVertexHandlerId);
     this.taskManager.deregisterHandler(this.checkRediscoveryHandlerId);
     this.logger.info(`Stopped ${this.constructor.name}`);
diff --git a/src/nodes/NodeConnection.ts b/src/nodes/NodeConnection.ts
index 090eaa974..9ef313fbb 100644
--- a/src/nodes/NodeConnection.ts
+++ b/src/nodes/NodeConnection.ts
@@ -308,7 +308,7 @@ class NodeConnection {
       logger: logger.getChild(RPCClient.name),
     });
     if (validatedNodeId == null) {
-      never(`connection validated but no valid NodeId was returned`);
+      never('connection validated but no valid NodeId was returned');
     }
     // Obtaining remote node ID from certificate chain. It should always exist in the chain if validated.
     //  This may de different from the NodeId we validated it as if it renewed at some point.