From e053d2cece486c06a0fda877094a04ce32abbf68 Mon Sep 17 00:00:00 2001 From: =?utf8?q?H=C3=A5kan=20Lindqvist?= Date: Thu, 11 May 2023 14:25:35 +0000 Subject: [PATCH] Clarify log message for NODATA/NXDOMAIN without AA The old log message did not indicate the specific reason why the record was considered irrelevant, logging only information about a SOA record that quite likely is both normal and relevant. This somewhat helps explaining what the actual problem is. --- pdns/recursordist/syncres.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/pdns/recursordist/syncres.cc b/pdns/recursordist/syncres.cc index f7270474f1..3195b4095b 100644 --- a/pdns/recursordist/syncres.cc +++ b/pdns/recursordist/syncres.cc @@ -4163,7 +4163,7 @@ void SyncRes::sanitizeRecords(const std::string& prefix, LWResult& lwr, const DN } if (!(lwr.d_aabit || wasForwardRecurse)) { - LOG(prefix << qname << ": Removing irrelevant record '" << rec->d_name << "|" << DNSRecordContent::NumberToType(rec->d_type) << "|" << rec->getContent()->getZoneRepresentation() << "' in the AUTHORITY section received from " << auth << endl); + LOG(prefix << qname << ": Removing irrelevant record (AA not set) '" << rec->d_name << "|" << DNSRecordContent::NumberToType(rec->d_type) << "|" << rec->getContent()->getZoneRepresentation() << "' in the AUTHORITY section received from " << auth << endl); rec = lwr.d_records.erase(rec); continue; } -- 2.47.2