]> granicus.if.org Git - pdns/commitdiff
Revert "Better logging, so the operator knows where to look."
authorPeter van Dijk <peter.van.dijk@powerdns.com>
Mon, 8 Jul 2019 09:16:27 +0000 (11:16 +0200)
committerPeter van Dijk <peter.van.dijk@powerdns.com>
Mon, 8 Jul 2019 09:16:27 +0000 (11:16 +0200)
This reverts commit c6e6b0559cfcff8537ee95c4f3d7c02477fd81a3.

pdns/communicator.hh
pdns/mastercommunicator.cc
pdns/tcpreceiver.cc

index c63534dc232b5fa709eba136905c9bfcde208645..831b2cecf534bbd6c16cfe4e72e7b4b08c6808d9 100644 (file)
@@ -251,7 +251,7 @@ private:
 class FindNS
 {
 public:
-  vector<string> lookup(const DNSName &name, UeberBackend *b, const DNSName& zone)
+  vector<string> lookup(const DNSName &name, UeberBackend *b)
   {
     vector<string> addresses;
 
@@ -269,11 +269,11 @@ public:
         }
         // After an exception, b can be inconsistent so break
         catch (PDNSException &ae) {
-          g_log << Logger::Error << "Could not lookup address for nameserver " << name << " in zone " << zone << ", cannot notify: " << ae.reason << endl;
+          g_log << Logger::Error << "Skipping record(s): " << ae.reason << endl;
           break;
         }
         catch (std::exception &e) {
-          g_log << Logger::Error << "Could not lookup address for nameserver " << name << " in zone " << zone << ", cannot notify: " << e.what() << endl;
+          g_log << Logger::Error << "Skipping record(s): " << e.what() << endl;
           break;
         }
       }
index 04803474994a64e63d7294265b30260d6ed0a768..b5841132cfad6121768edc75bbd0b92acd50f5a8 100644 (file)
@@ -56,7 +56,7 @@ void CommunicatorClass::queueNotifyDomain(const DomainInfo& di, UeberBackend* B)
       nsset.insert(getRR<NSRecordContent>(rr.dr)->getNS().toString());
 
     for(set<string>::const_iterator j=nsset.begin();j!=nsset.end();++j) {
-      vector<string> nsips=fns.lookup(DNSName(*j), B, di.zone);
+      vector<string> nsips=fns.lookup(DNSName(*j), B);
       if(nsips.empty())
         g_log<<Logger::Warning<<"Unable to queue notification of domain '"<<di.zone<<"': nameservers do not resolve!"<<endl;
       else
index acdfbb1b72b2f162bed131817be42b7fd831c71e..0ea41bb88f43ff97cb59fabcefdc54f8e16f676e 100644 (file)
@@ -494,7 +494,7 @@ bool TCPNameserver::canDoAXFR(shared_ptr<DNSPacket> q)
         while(B->get(rr)) 
           nsset.insert(DNSName(rr.content));
         for(const auto & j: nsset) {
-          vector<string> nsips=fns.lookup(j, s_P->getBackend(),q->qdomain);
+          vector<string> nsips=fns.lookup(j, s_P->getBackend());
           for(vector<string>::const_iterator k=nsips.begin();k!=nsips.end();++k) {
             // cerr<<"got "<<*k<<" from AUTO-NS"<<endl;
             if(*k == q->getRemote().toString())