dnsdist: Apply Charles-Henri's suggestions from code review (thanks!)

diff --git a/pdns/dnsdist-lua.cc b/pdns/dnsdist-lua.cc
index 5633307d14d52f53051b379c8749399f88306da8..ebd48fb6fc473eb7134a67f8b5eadc1c04eb91d1 100644 (file)
--- a/pdns/dnsdist-lua.cc
+++ b/pdns/dnsdist-lua.cc
@@ -1053,7 +1053,7 @@ static void setupLuaConfig(LuaContext& luaCtx, bool client, bool configCheck)
 
   luaCtx.writeFunction("showWebserverConfig", []() {
     setLuaNoSideEffect();
-    return getWebServerConfig();
+    return getWebserverConfig();
   });
 
   luaCtx.writeFunction("hashPassword", [](const std::string& password, boost::optional<uint64_t> workFactor) {

diff --git a/pdns/dnsdist-web.cc b/pdns/dnsdist-web.cc
index 76f772b7047de148117cb656f8652a1c4a0b043f..81c81d88bc3d5e8f69a63bce25f7097966d6399c 100644 (file)
--- a/pdns/dnsdist-web.cc
+++ b/pdns/dnsdist-web.cc
@@ -64,7 +64,7 @@ static const MetricDefinitionStorage s_metricDefinitions;
 
 static ConcurrentConnectionManager s_connManager(100);
 
-std::string getWebServerConfig()
+std::string getWebserverConfig()
 {
   ostringstream out;
 
@@ -74,14 +74,14 @@ std::string getWebServerConfig()
     out << "ACL: " << config->acl.toString() << endl;
     out << "Custom headers: ";
     if (config->customHeaders) {
+      out << endl;
       for (const auto& header : *config->customHeaders) {
         out << " - " << header.first << ": " << header.second << endl;
       }
     }
     else {
-      out << "None";
+      out << "None" << endl;
     }
-    out << endl;
     out << "Statistics require authentication: " << (config->statsRequireAuthentication ? "yes" : "no") << endl;
     out << "Password: " << (config->password ? "set" : "unset") << endl;
     out << "API key: " << (config->apiKey ? "set" : "unset") << endl;

diff --git a/pdns/dnsdistdist/dnsdist-web.hh b/pdns/dnsdistdist/dnsdist-web.hh
index a4f30fc80d4079bc1c924c35ea2415f26b4e4eb3..c119eee11d9e926f7ae86728f5fbfd81ad12742d 100644 (file)
--- a/pdns/dnsdistdist/dnsdist-web.hh
+++ b/pdns/dnsdistdist/dnsdist-web.hh
@@ -14,4 +14,4 @@ void dnsdistWebserverThread(int sock, const ComboAddress& local);
 void registerBuiltInWebHandlers();
 void clearWebHandlers();
 
-std::string getWebServerConfig();
+std::string getWebserverConfig();