rec: take into account throttled queries when determining if we had a cachehit

Problem spotted by @zjs604381586 in #13483, though proposed fix was not correct.

diff --git a/pdns/recursordist/pdns_recursor.cc b/pdns/recursordist/pdns_recursor.cc
index 9d7d10a3d4cbf535765f507083dcd9b8754b8530..9435f555e53f771ecb47847775681868ebb9218d 100644 (file)
--- a/pdns/recursordist/pdns_recursor.cc
+++ b/pdns/recursordist/pdns_recursor.cc
@@ -1846,7 +1846,7 @@ void startDoResolve(void* arg) // NOLINT(readability-function-cognitive-complexi
     }
 
     if (comboWriter->d_mdp.d_header.opcode == static_cast<unsigned>(Opcode::Query)) {
-      if (resolver.d_outqueries != 0 || resolver.d_authzonequeries != 0) {
+      if (resolver.d_outqueries != 0 || resolver.d_throttledqueries != 0 || resolver.d_authzonequeries != 0) {
         g_recCache->incCacheMisses();
       }
       else {