Infra queries should not use refresh mode.

When I introduced serve state in #11776, I reintroduced the mistake previously fixed in #11376.
Fixes #12078

(cherry picked from commit 28a3eea55d15d33a65b2108c1598c15d00e0173c)

diff --git a/pdns/syncres.cc b/pdns/syncres.cc
index cd018408f6605eeb0d2bcebca0a4b1db90287129..006095d83de68af6df54f41e3acadfbf62c42c6c 100644 (file)
--- a/pdns/syncres.cc
+++ b/pdns/syncres.cc
@@ -2073,9 +2073,6 @@ vector<ComboAddress> SyncRes::getAddrs(const DNSName &qname, unsigned int depth,
   if (d_serveStale) {
     flags |= MemRecursorCache::ServeStale;
   }
-  if (d_refresh) {
-    flags |= MemRecursorCache::Refresh;
-  }
   try {
     // First look for both A and AAAA in the cache
     res_t cset;
@@ -2223,9 +2220,6 @@ void SyncRes::getBestNSFromCache(const DNSName &qname, const QType qtype, vector
   if (d_serveStale) {
     flags |= MemRecursorCache::ServeStale;
   }
-  if (d_refresh) {
-    flags |= MemRecursorCache::Refresh;
-  }
   do {
     if (cutOffDomain && (subdomain == *cutOffDomain || !subdomain.isPartOf(*cutOffDomain))) {
       break;