From: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Date: Mon, 30 Nov 2020 16:40:31 +0000 (+0100)
Subject: Be a little less verbose telling we are looking up CNAMEs
X-Git-Tag: dnsdist-1.6.0-alpha2~13^2
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=refs%2Fpull%2F10112%2Fhead;p=thirdparty%2Fpdns.git

Be a little less verbose telling we are looking up CNAMEs
or DNAMEs. Only log when we found something
---

diff --git a/pdns/syncres.cc b/pdns/syncres.cc
index 2a4345dc47..1869c42d3e 100644
--- a/pdns/syncres.cc
+++ b/pdns/syncres.cc
@@ -1421,7 +1421,6 @@ bool SyncRes::doCNAMECacheCheck(const DNSName &qname, const QType qtype, vector<
   DNSName authZone;
   QType foundQT = QType::ENT;
 
-  LOG(prefix<<qname<<": Looking for CNAME cache hit of '"<<qname<<"|CNAME"<<"'"<<endl);
   /* we don't require auth data for forward-recurse lookups */
   if (g_recCache->get(d_now.tv_sec, qname, QType::CNAME, !wasForwardRecurse && d_requireAuthData, &cset, d_cacheRemote, d_refresh, d_routingTag, d_doDNSSEC ? &signatures : nullptr, d_doDNSSEC ? &authorityRecs : nullptr, &d_wasVariable, &state, &wasAuth, &authZone) > 0) {
     foundName = qname;
@@ -1433,7 +1432,6 @@ bool SyncRes::doCNAMECacheCheck(const DNSName &qname, const QType qtype, vector<
     auto labels = qname.getRawLabels();
     DNSName dnameName(g_rootdnsname);
 
-    LOG(prefix<<qname<<": Looking for DNAME cache hit of '"<<qname<<"|DNAME' or its ancestors"<<endl);
     do {
       dnameName.prependRawLabel(labels.back());
       labels.pop_back();
@@ -1449,7 +1447,6 @@ bool SyncRes::doCNAMECacheCheck(const DNSName &qname, const QType qtype, vector<
   }
 
   if (foundName.empty()) {
-    LOG(prefix<<qname<<": No CNAME or DNAME cache hit of '"<< qname <<"' found"<<endl);
     return false;
   }