From f019f773cbbc0aacf02f221d8d0b5d46c38a1667 Mon Sep 17 00:00:00 2001 From: Otto Moerbeek Date: Tue, 17 Jan 2023 10:59:04 +0100 Subject: [PATCH] Remove remains of g_dnssecLOG It was never set from !recursor and has the same value as trace in rec. Now validationo logging goes the same as other trace handling, so trace-regex also shows validation. --- pdns/recursordist/rec-main.cc | 1 - pdns/recursordist/test-syncres_cc.cc | 1 - pdns/validate.cc | 5 +++-- pdns/validate.hh | 1 - 4 files changed, 3 insertions(+), 5 deletions(-) diff --git a/pdns/recursordist/rec-main.cc b/pdns/recursordist/rec-main.cc index ad0cb39fd5..2c5c8260d9 100644 --- a/pdns/recursordist/rec-main.cc +++ b/pdns/recursordist/rec-main.cc @@ -1498,7 +1498,6 @@ static int serviceMain(int argc, char* argv[], Logr::log_t log) SyncRes::setDefaultLogMode(SyncRes::Log); ::arg().set("quiet") = "no"; g_quiet = false; - g_dnssecLOG = true; } auto myHostname = getHostname(); if (!myHostname.has_value()) { diff --git a/pdns/recursordist/test-syncres_cc.cc b/pdns/recursordist/test-syncres_cc.cc index f470371095..c799b60b5f 100644 --- a/pdns/recursordist/test-syncres_cc.cc +++ b/pdns/recursordist/test-syncres_cc.cc @@ -208,7 +208,6 @@ void initSR(bool debug) g_luaconfs.setState(luaconfsCopy); g_dnssecmode = DNSSECMode::Off; - g_dnssecLOG = debug; g_maxNSEC3Iterations = 2500; g_aggressiveNSECCache.reset(); diff --git a/pdns/validate.cc b/pdns/validate.cc index 49d1001313..2f68a833f0 100644 --- a/pdns/validate.cc +++ b/pdns/validate.cc @@ -5,12 +5,13 @@ #include "rec-lua-conf.hh" #include "base32.hh" #include "logger.hh" -bool g_dnssecLOG{false}; + time_t g_signatureInceptionSkew{0}; uint16_t g_maxNSEC3Iterations{0}; #ifndef RECURSOR -#define LOG(x) if(g_dnssecLOG) { g_log <