From: Remi Gacogne <remi.gacogne@powerdns.com>
Date: Thu, 23 Jun 2022 10:36:17 +0000 (+0200)
Subject: dnsdist: Fix the number of concurrent queries on a backend TCP conn
X-Git-Tag: auth-4.8.0-alpha0~41^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=2848406e55b5a1d2e35bfa9f7c4dbb0c49989a1b;p=thirdparty%2Fpdns.git

dnsdist: Fix the number of concurrent queries on a backend TCP conn

When we are in the process of sending a query to the backend, that
query is no longer accounted in the "queued" queries nor it is in
the "queued" responses, but we need to take it into account.
Otherwise we might be sending two concurrent queries to a backend
that does not support out-of-order processing (increasing our
latency), or even worse to one that does not support pipelining.
---

diff --git a/pdns/dnsdistdist/dnsdist-tcp-downstream.hh b/pdns/dnsdistdist/dnsdist-tcp-downstream.hh
index c726775938..c257e0c124 100644
--- a/pdns/dnsdistdist/dnsdist-tcp-downstream.hh
+++ b/pdns/dnsdistdist/dnsdist-tcp-downstream.hh
@@ -239,7 +239,7 @@ public:
 
   bool reachedMaxConcurrentQueries() const override
   {
-    const size_t concurrent = d_pendingQueries.size() + d_pendingResponses.size();
+    const size_t concurrent = d_pendingQueries.size() + d_pendingResponses.size() + (d_state == State::sendingQueryToBackend ? 1 : 0);
     if (concurrent > 0 && concurrent >= d_ds->d_config.d_maxInFlightQueriesPerConn) {
       return true;
     }