From 7931038fa875501a57c18b70daeb6f74ae013dc9 Mon Sep 17 00:00:00 2001 From: Mark Andrews Date: Fri, 6 Feb 2026 12:52:55 +1100 Subject: [PATCH] Record query time for all dnstap responses The description in the protobuf specification is not a list of request types to process but rather a list of examples to qualify the description of whether the time indicates when the message is received or sent. (cherry picked from commit 479c7375173afd6f9e9cdd894752668682fba1e1) --- lib/dns/dnstap.c | 9 --------- 1 file changed, 9 deletions(-) diff --git a/lib/dns/dnstap.c b/lib/dns/dnstap.c index 18b365c303..4705f23dbf 100644 --- a/lib/dns/dnstap.c +++ b/lib/dns/dnstap.c @@ -781,15 +781,6 @@ dns_dt_send(dns_view_t *view, dns_dtmsgtype_t msgtype, isc_sockaddr_t *qaddr, dm.m.has_response_time_sec = 1; dm.m.response_time_nsec = isc_time_nanoseconds(t); dm.m.has_response_time_nsec = 1; - - /* - * Types RR and FR can fall through and get the query - * time set as well. Any other response type, break. - */ - if (msgtype != DNS_DTTYPE_RR && msgtype != DNS_DTTYPE_FR) { - break; - } - FALLTHROUGH; case DNS_DTTYPE_AQ: case DNS_DTTYPE_CQ: