From ffe8375dd1f04cd3a56df7c0df5946689e8890a7 Mon Sep 17 00:00:00 2001
From: audric <audric@loki.network>
Date: Mon, 6 Sep 2021 14:13:49 +1000
Subject: [PATCH] fix build of onion path when snodes count is exactly 12

---
 ts/session/onions/onionPath.ts                 | 2 +-
 ts/session/snode_api/snodePool.ts              | 6 +++---
 ts/test/session/unit/onion/OnionErrors_test.ts | 2 +-
 3 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/ts/session/onions/onionPath.ts b/ts/session/onions/onionPath.ts
index c091afec5..789ad6701 100644
--- a/ts/session/onions/onionPath.ts
+++ b/ts/session/onions/onionPath.ts
@@ -425,7 +425,7 @@ async function buildNewOnionPathsWorker() {
   );
   // TODO: select one guard node and 2 other nodes randomly
   let otherNodes = _.differenceBy(allNodes, guardNodes, 'pubkey_ed25519');
-  if (otherNodes.length < SnodePool.minSnodePoolCount) {
+  if (otherNodes.length <= SnodePool.minSnodePoolCount) {
     window?.log?.warn(
       'LokiSnodeAPI::buildNewOnionPaths - Too few nodes to build an onion path! Refreshing pool and retrying'
     );
diff --git a/ts/session/snode_api/snodePool.ts b/ts/session/snode_api/snodePool.ts
index 1805c8825..d8af6dc88 100644
--- a/ts/session/snode_api/snodePool.ts
+++ b/ts/session/snode_api/snodePool.ts
@@ -268,7 +268,7 @@ export async function refreshRandomPool(forceRefresh = false): Promise<void> {
       if (fetchedFromDb?.length) {
         window?.log?.info(`refreshRandomPool: fetched from db ${fetchedFromDb.length} snodes.`);
         randomSnodePool = fetchedFromDb;
-        if (randomSnodePool.length < minSnodePoolCount) {
+        if (randomSnodePool.length <= minSnodePoolCount) {
           window?.log?.warn('refreshRandomPool: not enough snodes in db, going to fetch from seed');
         } else {
           return;
@@ -279,9 +279,9 @@ export async function refreshRandomPool(forceRefresh = false): Promise<void> {
     }
 
     // we don't have nodes to fetch the pool from them, so call the seed node instead.
-    if (randomSnodePool.length < minSnodePoolCount) {
+    if (randomSnodePool.length <= minSnodePoolCount) {
       window?.log?.info(
-        `refreshRandomPool: NOT enough snodes to fetch from them ${randomSnodePool.length} < ${minSnodePoolCount}, so falling back to seedNodes ${seedNodes?.length}`
+        `refreshRandomPool: NOT enough snodes to fetch from them ${randomSnodePool.length} <= ${minSnodePoolCount}, so falling back to seedNodes ${seedNodes?.length}`
       );
 
       randomSnodePool = await exports.refreshRandomPoolDetail(seedNodes);
diff --git a/ts/test/session/unit/onion/OnionErrors_test.ts b/ts/test/session/unit/onion/OnionErrors_test.ts
index 7ebd585c0..4f9e13963 100644
--- a/ts/test/session/unit/onion/OnionErrors_test.ts
+++ b/ts/test/session/unit/onion/OnionErrors_test.ts
@@ -67,7 +67,7 @@ describe('OnionPathsErrors', () => {
 
   beforeEach(async () => {
     guardPubkeys = TestUtils.generateFakePubKeys(3).map(n => n.key);
-    otherNodesPubkeys = TestUtils.generateFakePubKeys(12).map(n => n.key);
+    otherNodesPubkeys = TestUtils.generateFakePubKeys(13).map(n => n.key);
 
     SNodeAPI.Onions.resetSnodeFailureCount();