From 8d4a47dcc9c4dc70caeadf7e6ac12d3dde63735c Mon Sep 17 00:00:00 2001
From: Fred Morcos <fred.morcos@open-xchange.com>
Date: Thu, 26 Oct 2023 14:50:20 +0200
Subject: [PATCH] Refactor UeberBackend::getAuth: foundTarget

---
 pdns/ueberbackend.cc | 22 ++++++++++++++++------
 1 file changed, 16 insertions(+), 6 deletions(-)

diff --git a/pdns/ueberbackend.cc b/pdns/ueberbackend.cc
index 58dda6cf68..17d8178613 100644
--- a/pdns/ueberbackend.cc
+++ b/pdns/ueberbackend.cc
@@ -433,6 +433,19 @@ static std::vector<DNSBackend*>::iterator findBestMatchingBackend(std::vector<DN
   return backend;
 }
 
+static bool foundTarget(const DNSName& target, const DNSName& shorter, const QType& qtype, SOAData* soaData, const bool found)
+{
+  auto name = soaData->qname;
+
+  if (found == (qtype == QType::DS) || target != shorter) {
+    DLOG(g_log << Logger::Error << "found: " << name << endl);
+    return true;
+  }
+
+  DLOG(g_log << Logger::Error << "chasing next: " << name << endl);
+  return false;
+}
+
 bool UeberBackend::getAuth(const DNSName& target, const QType& qtype, SOAData* soaData, bool cachedOk)
 {
   // A backend can respond to our authority request with the 'best' match it
@@ -511,14 +524,11 @@ bool UeberBackend::getAuth(const DNSName& target, const QType& qtype, SOAData* s
     }
 
   found:
-    if (found == (qtype == QType::DS) || target != shorter) {
-      DLOG(g_log << Logger::Error << "found: " << soaData->qname << endl);
+    if (foundTarget(target, shorter, qtype, soaData, found)) {
       return true;
     }
-    else {
-      DLOG(g_log << Logger::Error << "chasing next: " << soaData->qname << endl);
-      found = true;
-    }
+
+    found = true;
   } while (shorter.chopOff());
 
   return found;
-- 
2.47.2