From e472003b6d380a5bdc360b4de9fd38aae41a343b Mon Sep 17 00:00:00 2001 From: lunar-mining Date: Wed, 3 Jan 2024 18:00:23 +0100 Subject: [PATCH] outbound_session: fetch_address() logic bug fix we need to use else if rather than chaining if statements as it gets confused and calls multiple fetch methods otherwise. --- src/net/session/outbound_session.rs | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/src/net/session/outbound_session.rs b/src/net/session/outbound_session.rs index 71502bb1a..e40df74bd 100644 --- a/src/net/session/outbound_session.rs +++ b/src/net/session/outbound_session.rs @@ -210,7 +210,7 @@ impl Slot { // Select from the whitelist // If the whitelist is empty, select from the greylist // If the greylist is empty, do peer discovery - if connect_count < white_count { + else if connect_count < white_count { debug!(target: "outbound_session::fetch_address()", "Next N connections- prefer white connections"); return hosts.whitelist_fetch_address_with_lock(self.p2p(), transports).await @@ -219,7 +219,7 @@ impl Slot { // // Select from the greylist // If the greylist is empty, do peer discovery - if connect_count < slot_count { + else if connect_count < slot_count { debug!(target: "outbound_session::fetch_address()", "All other connections- get grey connections"); return hosts.greylist_fetch_address_with_lock(self.p2p(), transports).await