dnsdist: Plug the new 'redirect-responses' metric in

diff --git a/pdns/dnsdist-carbon.cc b/pdns/dnsdist-carbon.cc
index d6860aee377faa798d5361c59a4fe8ac4b6e9f31..a40e1dff66e128f15297a93020d991c9de6bac8b 100644 (file)
--- a/pdns/dnsdist-carbon.cc
+++ b/pdns/dnsdist-carbon.cc
@@ -197,6 +197,7 @@ try
               {"post-queries", doh->d_postqueries},
               {"bad-requests", doh->d_badrequests},
               {"error-responses", doh->d_errorresponses},
+              {"redirect-responses", doh->d_redirectresponses},
               {"valid-responses", doh->d_validresponses}
             };
 

diff --git a/pdns/dnsdist-lua.cc b/pdns/dnsdist-lua.cc
index 6a0e5211dea19baa2272cbb7a39be014d1b5d950..3e5cf38b7af805ecb282af9d2a5594d49098a28f 100644 (file)
--- a/pdns/dnsdist-lua.cc
+++ b/pdns/dnsdist-lua.cc
@@ -1747,11 +1747,11 @@ void setupLuaConfig(bool client)
         setLuaNoSideEffect();
         try {
           ostringstream ret;
-          boost::format fmt("%-3d %-20.20s %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d");
-          ret << (fmt % "#" % "Address" % "HTTP" % "HTTP/1" % "HTTP/2" % "TLS 1.0" % "TLS 1.1" % "TLS 1.2" % "TLS 1.3" % "TLS other" % "GET" % "POST" % "Bad" % "Errors" % "Valid") << endl;
+          boost::format fmt("%-3d %-20.20s %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d %-15d");
+          ret << (fmt % "#" % "Address" % "HTTP" % "HTTP/1" % "HTTP/2" % "TLS 1.0" % "TLS 1.1" % "TLS 1.2" % "TLS 1.3" % "TLS other" % "GET" % "POST" % "Bad" % "Errors" % "Redirects" % "Valid") << endl;
           size_t counter = 0;
           for (const auto& ctx : g_dohlocals) {
-            ret << (fmt % counter % ctx->d_local.toStringWithPort() % ctx->d_httpconnects % ctx->d_http1Stats.d_nbQueries % ctx->d_http1Stats.d_nbQueries % ctx->d_tls10queries % ctx->d_tls11queries % ctx->d_tls12queries % ctx->d_tls13queries % ctx->d_tlsUnknownqueries % ctx->d_getqueries % ctx->d_postqueries % ctx->d_badrequests % ctx->d_errorresponses % ctx->d_validresponses) << endl;
+            ret << (fmt % counter % ctx->d_local.toStringWithPort() % ctx->d_httpconnects % ctx->d_http1Stats.d_nbQueries % ctx->d_http1Stats.d_nbQueries % ctx->d_tls10queries % ctx->d_tls11queries % ctx->d_tls12queries % ctx->d_tls13queries % ctx->d_tlsUnknownqueries % ctx->d_getqueries % ctx->d_postqueries % ctx->d_badrequests % ctx->d_errorresponses % ctx->d_redirectresponses %ctx->d_validresponses) << endl;
             counter++;
           }
           g_outputBuffer = ret.str();

diff --git a/pdns/dnsdist-web.cc b/pdns/dnsdist-web.cc
index 00c6e36b1d610f35c48cbe160977c24a83c522f9..11df6a9ccc826509820bea3f80684bc55717df49 100644 (file)
--- a/pdns/dnsdist-web.cc
+++ b/pdns/dnsdist-web.cc
@@ -591,6 +591,8 @@ static void connectionThread(int sock, ComboAddress remote)
         output << "# TYPE " << dohfrontsbase << "bad_requests " << "counter" << "\n";
         output << "# HELP " << dohfrontsbase << "error_responses " << "Number of responses sent by dnsdist indicating an error" << "\n";
         output << "# TYPE " << dohfrontsbase << "error_responses " << "counter" << "\n";
+        output << "# HELP " << dohfrontsbase << "redirect_responses " << "Number of responses sent by dnsdist indicating a redirect" << "\n";
+        output << "# TYPE " << dohfrontsbase << "redirect_responses " << "counter" << "\n";
         output << "# HELP " << dohfrontsbase << "valid_responses " << "Number of valid responses sent by dnsdist" << "\n";
         output << "# TYPE " << dohfrontsbase << "valid_responses " << "counter" << "\n";
         output << "# HELP " << dohfrontsbase << "http1_queries " << "Number of queries received over HTTP/1.x" << "\n";
@@ -636,6 +638,7 @@ static void connectionThread(int sock, ComboAddress remote)
           output << dohfrontsbase << "post_queries" << label << doh->d_postqueries << "\n";
           output << dohfrontsbase << "bad_requests" << label << doh->d_badrequests << "\n";
           output << dohfrontsbase << "error_responses" << label << doh->d_errorresponses << "\n";
+          output << dohfrontsbase << "redirect_responses" << label << doh->d_redirectresponses << "\n";
           output << dohfrontsbase << "valid_responses" << label << doh->d_validresponses << "\n";
 
           output << dohfrontsbase << "http1_queries" << label << doh->d_http1Stats.d_nbQueries << "\n";
@@ -798,6 +801,7 @@ static void connectionThread(int sock, ComboAddress remote)
             { "post-queries", (double) doh->d_postqueries },
             { "bad-requests", (double) doh->d_badrequests },
             { "error-responses", (double) doh->d_errorresponses },
+            { "redirect-responses", (double) doh->d_redirectresponses },
             { "valid-responses", (double) doh->d_validresponses }
           };
           dohs.push_back(obj);