dnsdist: Fix clang-tidy warnings

diff --git a/pdns/dnsdistdist/dnsdist-doh-common.cc b/pdns/dnsdistdist/dnsdist-doh-common.cc
index d6f48b9390d5e1bb7644fdf33cfcbb4c37b0aa4c..df3c01d8a215599f65f3f396a1c793271f03bb24 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-doh-common.cc
+++ b/pdns/dnsdistdist/dnsdist-doh-common.cc
@@ -69,7 +69,7 @@ bool HTTPPathRule::matches(const DNSQuestion* dq) const
     const auto path = dq->ids.du->getHTTPPath();
     return d_path == path;
   }
-  else if (dq->ids.doh3u) {
+  if (dq->ids.doh3u) {
     return dq->ids.doh3u->getHTTPPath() == d_path;
   }
   return false;
@@ -91,7 +91,7 @@ bool HTTPPathRegexRule::matches(const DNSQuestion* dq) const
     const auto path = dq->ids.du->getHTTPPath();
     return d_regex.match(path);
   }
-  else if (dq->ids.doh3u) {
+  if (dq->ids.doh3u) {
     return d_regex.match(dq->ids.doh3u->getHTTPPath());
   }
   return false;

diff --git a/pdns/dnsdistdist/dnsdist-lua-ffi.cc b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
index d531cb95dd5610b3e96ec0d233923156954f6ee5..79ee928428a0b849cedbce551d75ef958193bacc 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-lua-ffi.cc
+++ b/pdns/dnsdistdist/dnsdist-lua-ffi.cc
@@ -422,10 +422,10 @@ size_t dnsdist_ffi_dnsquestion_get_http_headers(dnsdist_ffi_dnsquestion_t* dq, c
 {
 #if defined(HAVE_DNS_OVER_HTTPS) || defined(HAVE_DNS_OVER_HTTP3)
   const auto processHeaders = [&dq](const std::unordered_map<std::string, std::string>& headers) {
-    if (headers.size() == 0) {
+    if (headers.empty()) {
       return;
     }
-    dq->httpHeaders = std::make_unique<std::unordered_map<std::string, std::string>>(std::move(headers));
+    dq->httpHeaders = std::make_unique<std::unordered_map<std::string, std::string>>(headers);
     if (!dq->httpHeadersVect) {
       dq->httpHeadersVect = std::make_unique<std::vector<dnsdist_ffi_http_header_t>>();
     }

diff --git a/pdns/dnsdistdist/doh3.cc b/pdns/dnsdistdist/doh3.cc
index e98880507398e239d391b4b822171789a4196a52..7f4586fe88f3eb22856b781a414a25209ab4c1f0 100644 (file)
--- a/pdns/dnsdistdist/doh3.cc
+++ b/pdns/dnsdistdist/doh3.cc
@@ -1049,7 +1049,7 @@ std::string DOH3Unit::getHTTPQueryString() const
   const auto& path = headers.at(":path");
   auto pos = path.find('?');
   if (pos == string::npos) {
-    return std::string();
+    return {};
   }
 
   return path.substr(pos);
@@ -1082,22 +1082,22 @@ const dnsdist::doh3::h3_headers_t& DOH3Unit::getHTTPHeaders() const
 
 std::string DOH3Unit::getHTTPPath() const
 {
-  return std::string();
+  return {};
 }
 
 std::string DOH3Unit::getHTTPQueryString() const
 {
-  return std::string();
+  return {};
 }
 
 std::string DOH3Unit::getHTTPHost() const
 {
-  return std::string();
+  return {};
 }
 
 std::string DOH3Unit::getHTTPScheme() const
 {
-  return std::string();
+  return {};
 }
 
 const dnsdist::doh3::h3_headers_t& DOH3Unit::getHTTPHeaders() const