[Dnsmasq-discuss] [PATCH 3] Some upstream replies not being logged

[Dominik DL6ER]

Hey Simon,

On Tue, 2021-08-31 at 21:09 +0100, Simon Kelley wrote:
> I've tagged 2.86rc2. I've systematically tested the affected code,
> and we're dog-fooding it now. It would be good to get as much other
> testing in as possible before the 2.86.

We have automated CI testing putting a lot of queries (and different
query types) through it and nothing unexpected happened.
We've added the patch into our beta testing so we have at least a few
dozen additional testers of the new code on our side, too.

I've seen two minor things but this is really nit-picky:

> dig TYPE65 https.dns.netmeister.org

leads to

> reply https.dns.netmeister.org is type=65

whereas I'd consider

> reply https.dns.netmeister.org is [type=65]

slightly more elegant because it is the type of the reply, not the
content. The necessary change would be querystr(NULL, aqtype) ->
querystr("", aqtype) in the "else" branch of "if (aqtype == T_TXT)" but
that's entirely your call.

Also, DNSSEC signed TXT records were not logged as such with log-
queries=extra. print_txt() needs secflag for this. Patch attached.

I'll report back if anything odd comes up. Silence will be a good
thing.

Best,
Dominik
-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0001-Print-DNSSEC-signed-for-signed-TXT-records-when-log-.patch
Type: text/x-patch
Size: 1895 bytes
Desc: not available
URL: <http://lists.thekelleys.org.uk/pipermail/dnsmasq-discuss/attachments/20210901/ba220a99/attachment.bin>