From cd27b3384975befd987b590bf33e4940fe006f2c Mon Sep 17 00:00:00 2001 From: Jeff Becker Date: Fri, 7 Feb 2020 13:19:00 -0500 Subject: [PATCH 1/2] dont flood dht with requests, if we know of a router don't look it up --- llarp/dht/explorenetworkjob.cpp | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/llarp/dht/explorenetworkjob.cpp b/llarp/dht/explorenetworkjob.cpp index 364429528..6b5d8a6ce 100644 --- a/llarp/dht/explorenetworkjob.cpp +++ b/llarp/dht/explorenetworkjob.cpp @@ -4,6 +4,8 @@ #include #include +#include + namespace llarp { namespace dht @@ -24,6 +26,8 @@ namespace llarp for(const auto &pk : valuesFound) { // lookup router + if(router->nodedb()->Has(pk)) + continue; parent->LookupRouter( pk, std::bind(&AbstractRouter::HandleDHTLookupForExplore, router, pk, From a9331a143184fa5d22e8ce0c15abd3f7770b345e Mon Sep 17 00:00:00 2001 From: Jeff Becker Date: Fri, 7 Feb 2020 13:48:56 -0500 Subject: [PATCH 2/2] fix unit test --- llarp/dht/explorenetworkjob.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/llarp/dht/explorenetworkjob.cpp b/llarp/dht/explorenetworkjob.cpp index 6b5d8a6ce..5da2eb591 100644 --- a/llarp/dht/explorenetworkjob.cpp +++ b/llarp/dht/explorenetworkjob.cpp @@ -26,7 +26,7 @@ namespace llarp for(const auto &pk : valuesFound) { // lookup router - if(router->nodedb()->Has(pk)) + if(router and router->nodedb()->Has(pk)) continue; parent->LookupRouter( pk,