Re: [Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2023-01-25 Thread Simon Kelley
Patch applied. As this is a low-complexity change, it has gone into the 2.89 release. I've tagged 2.89test1, as a move towards that. Cheers, Simon. On 18/01/2023 03:24, Dominik Derigs wrote: Hey Simon, totally an oversight on my side. Updated patch attached. Best Dominik On Mon,

Re: [Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2023-01-17 Thread Dominik Derigs via Dnsmasq-discuss
Hey Simon, totally an oversight on my side. Updated patch attached. Best Dominik On Mon, 2023-01-16 at 22:46 +, Simon Kelley wrote: > The patch moves the relevant code to after processing of command-line > arguments, but not reading configuration file(s). I can't think of any > reason why

Re: [Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2023-01-16 Thread Simon Kelley
The patch moves the relevant code to after processing of command-line arguments, but not reading configuration file(s). I can't think of any reason why you wouldn't want to put no-ident into /etc/dnsmasq.conf Have I missed something? Cheers, Simon. On 15/01/2023 08:01, Dominik Derigs

Re: [Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2023-01-15 Thread Dominik Derigs via Dnsmasq-discuss
Hey Simon and Petr, CC list, Thanks for pointing this out. I fixed it and rebased on latest master. New patch attached. The name "--no-ident" has been chosen as the corresponding compile-time option is called NO_ID - my feeling is that this corresponds to "no identification (strings)". I

Re: [Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2023-01-03 Thread Petr Menšík
Hi Dominik, Good idea. But there is copy error on line 572, no help description is changed for new option. What does --no-ident stand for by the way? Wouldn't be --no-chaos more descriptive? Should we support optional argument to disable only some parts of provided records? For example cache

[Dnsmasq-discuss] [PATCH] Add run-time option to disable CHAOS TXT records

2022-12-23 Thread Dominik Derigs via Dnsmasq-discuss
Dear mailing list members, In 2016, dnsmasq got the option to prevent serving the server version, some statistics and even the used upstream DNS servers to its clients. Compiling with -DNO_ID removes the entire *.bind info structure. However, setting -DNO_ID requires a (re-)compilation from