]> granicus.if.org Git - pdns/commitdiff
auth: cleanup slave-renotify code
authorKees Monshouwer <mind04@monshouwer.org>
Fri, 20 Sep 2019 13:13:26 +0000 (15:13 +0200)
committermind04 <mind04@monshouwer.org>
Fri, 20 Sep 2019 13:13:26 +0000 (15:13 +0200)
pdns/slavecommunicator.cc

index c6b08fefcfbae0ed90600876b15fb859f09745c2..31ae46285fcc922d9d5385cb781b75d69d6daab3 100644 (file)
@@ -598,22 +598,20 @@ void CommunicatorClass::suck(const DNSName &domain, const ComboAddress& remote)
     di.backend->setFresh(zs.domain_id);
     purgeAuthCaches(domain.toString()+"$");
 
-
     g_log<<Logger::Error<<"AXFR done for '"<<domain<<"', zone committed with serial number "<<zs.soa_serial<<endl;
 
-    bool renotify = false;
-    if(::arg().mustDo("slave-renotify"))
-      renotify = true;
+    // Send slave re-notifications
+    bool notify;
     vector<string> meta;
-    if (B.getDomainMetadata(domain, "SLAVE-RENOTIFY", meta) && meta.size() > 0) {
-      if (meta[0] == "1") {
-        renotify = true;
-      } else {
-        renotify = false;
-      }
+    if(B.getDomainMetadata(domain, "SLAVE-RENOTIFY", meta ) && !meta.empty()) {
+      notify=(meta.front() == "1");
+    } else {
+      notify=(::arg().mustDo("slave-renotify"));
     }
-    if(renotify)
+    if(notify) {
       notifyDomain(domain, &B);
+    }
+
   }
   catch(DBException &re) {
     g_log<<Logger::Error<<"Unable to feed record during incoming AXFR of '" << domain<<"': "<<re.reason<<endl;