do not requeue nodes for testing from failing queue if we do not have them marked as failing anymore

3 years ago · fcba709fcb
parent 60c1b69ea3
commit fcba709fcb
1 changed files with 2 additions and 1 deletions
--- a/llarp/consensus/reachability_testing.cpp
+++ b/llarp/consensus/reachability_testing.cpp
@ -124,7 +124,8 @@ namespace llarp::consensus
      auto& [pk, retest_time, failures] = failing_queue.top();
      if (retest_time > now)
        break;
-      result.emplace_back(pk, failures);
+      if (failing.count(pk))
+        result.emplace_back(pk, failures);
      failing_queue.pop();
    }
    return result;