Merge pull request #14976 from omoerbeek/rec-quit-nicely-sync

rec: make quit-nicely wait on actual quit and start using it for stopping by systemd

Typo in comment

Co-authored-by: Miod Vallat <miod.vallat@open-xchange.com>

Merge pull request #15149 from rgacogne/ddist-meson-min-version

dnsdist: Bump the required version of `meson` to 1.3

Merge pull request #15148 from rgacogne/ddist-unbreak-console

dnsdist: Fix console ("client mode") on non-default address or port

dnsdist: Bump the required version of `meson` to 1.3

Since we now use the `follow_symlinks` option of `install_data()`.
We could probably make it work with older versions but a quick look
at what distributions provide suggests that 1.2.1 or 1.3.0 is
practically the same amount of pain.

dnsdist: Fix console ("client mode") on non-default address or port

The console address and port were not properly set in client mode,
so `dnsdist` tried to connect to the default console address and port.

Merge pull request #15138 from rgacogne/ddist-meson-install

dnsdist: Install binary, man page and systemd unit files with meson

Merge pull request #15147 from miodvallat/rrset_me_on_fire

Low-hanging LMDB fruits

Merge pull request #15144 from PowerDNS/dependabot/pip/regression-tests.recursor-dnssec/cryptography-44.0.1

build(deps): bump cryptography from 43.0.1 to 44.0.1 in /regression-tests.recursor-dnssec

Merge pull request #15143 from Habbie/no-ubuntu-20.04

switch ubuntu-20.04 workflows to 24.04

dnsdist: Tell meson to follow symbolic links for `dnsdist.conf-dist`

Merge pull request #15084 from miodvallat/fire_of_unknown_origin

Boring minor cleanups

Merge pull request #15080 from miodvallat/import_with_care

pdnsutil: check for key presence after import

Merge pull request #15146 from omoerbeek/docs-requirements

docs: update requirements for auth and dnsdist

Sprinkle reserve() calls on vectors when possible and useful.

Disable read ahead in LMDB.

This contributes to not degrading performance too much when the system
memory is almost completely used, by preventing page thrashing due to
the read ahead behaviour of the system.

docs: update requirements for auth and dnsdist

rec was already done earlier in #15076

Merge pull request #15145 from miodvallat/gimme_more_salt

[tools] fix sdig link

Always link sdig against libsodium if used.

Fallout from #15140

Clearly separate signal handling from rec_control quit(-nicely) path

Also: as the bulk and regression tests now use quit-nicely, we don't need to do special processing
for the SAN case anymore on signals.

rec_control quit still does some SAN work and exits 1 (legacy beheviour).

build(deps): bump cryptography in /regression-tests.recursor-dnssec

Bumps [cryptography](https://github.com/pyca/cryptography) from 43.0.1 to 44.0.1.
- [Changelog](https://github.com/pyca/cryptography/blob/main/CHANGELOG.rst)
- [Commits](https://github.com/pyca/cryptography/compare/43.0.1...44.0.1)

---
updated-dependencies:
- dependency-name: cryptography
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #15102 from romeroalx/enable-meson-auth-deb11

GH Actions - build-and-test-all.yml: use virtual environment for python packages

Reformat with clang-format-19

Remove redundant CircleCI condition

Switch to clang-format-19 by default and allow //comment with no leading space

Still some formatting differences, but they look legit. It seems
older versions of clang-format did not process preprocessor
conditionals correctly in all cases.

Merge pull request #15142 from omoerbeek/docs-keyblock-link-text

docs: correct text of link for key block

bump clang

use pdns from noble

switch ubuntu-20.04 workflows to 24.04

correct text and link for key block

Merge pull request #15140 from omoerbeek/cookies-phase1

sdig: implement cookie support

dnsdist: Install the sample configuration file in sysconfdir with meson

Hopefully fix man pages with meson

Update docs/manpages/sdig.1.rst

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Merge pull request #15139 from omoerbeek/rec-bound-check-for-future

rec: check bounds of rcode stats counter index (safe right now)

Apply suggestion

Co-authored-by: Remi Gacogne <github@coredump.fr>

Apply suggestions from code review

Co-authored-by: Remi Gacogne <github@coredump.fr>

clang-tidy

Add cookies support for sdig

dnsdist: Add an option to meson to disable man page generation

dnsdist: Install the example configuration file with meson

rec: Fix handling of man pages with meson

This commit moves to `custom_target` to build the man pages since
we don't want to have to explicitely run a different meson command
to build them. As opposed to `run_target`, `custom_target` does not
have access to the build and source roots via env variables, so the
man pages generation script now takes these as required parameters.

auth: Fix handling of man pages with meson

This commit moves to `custom_target` to build the man pages since
we don't want to have to explicitely run a different meson command
to build them. As opposed to `run_target`, `custom_target` does not
have access to the build and source roots via env variables, so the
man pages generation script now takes these as required parameters.

dnsdist: Fix handling of man pages with meson

This commit moves to `custom_target` to build the man pages since
we don't want to have to explicitely run a different meson command
to build them. As opposed to `run_target`, `custom_target` does not
have access to the build and source roots via env variables, so the
man pages generation script now takes these as required parameters.

dnsdist: Install binary, man page and systemd unit files with meson

Tidy

Merge pull request #15123 from esensar/feature/dnstap-multi-stream

dnsdist: add pooling support for `RemoteLoggerInterface`

Merge pull request #15112 from rgacogne/ddist-build-xsk-noble-oracular

dnsdist: Enable XSK in our Noble and Oracular Ubuntu packages

rec: check bounds of rcode stats counter index (safe right now)

Safe right now as LWResult::d_rcode gets assigned from the 4 bit
rcode in the header.  But that might change one day. I'd rather
make LWResult::d_rcode an uint8_t, but that causes a conflict with
the OOB resolving code that does not make a difference between res
and d_rcode.

Fix underline length warning.

Make dnstap test handle EOF and wait on process to exit.

The wait is essential if you have multiple tests classes in a test file.

Don't use logging in signal handler

Start using quit-nicely in rec bulk test

Fix autotools case for @ and adapt meson.build to handle ExecStop as well

mangle unit file to take into account bindir and multiple instances

Start using rec_control quit-nicely to stop rec runing under systemd's control

Use structured logging

Hack to get the TSAN case working (it just calls _exit())

Don't explictly close control channel

Update pdns/recursordist/rec-main.cc

Co-authored-by: Miod Vallat <miod.vallat@open-xchange.com>

rec: make quit-nicely wait on actual quit

Merge pull request #15131 from miodvallat/seventh_zone_of_a_seventh_zone

[pdnsutil] Let add-zone-key use defaults from pdns.conf

Make add-zone-key default to KSK rather than ZSK.

Merge pull request #15137 from rgacogne/ddist-fix-xsk-boost-lockfree-1.87

dnsdist: Fix compatibility with `boost::lockfree` >= 1.87.0

dnsdist: Fix compatibility with boost::lockfree >= 1.87.0

In https://github.com/boostorg/lockfree/pull/90 `boost::lockfree::spsc_queue`
introduced moved semantics, which is great, but added restrictions
to the callback functor that did not exist before, breaking the API.
This PR fixes that by updating our callbacks to expect an object
instead of a reference.

Merge pull request #15134 from rgacogne/ddist-quiche-version-fallback

dnsdist: Gracefully fall back to older versions of Quiche

Add `RemotePoolLogger` tests for remote protobuf logger

Add `RemotePoolLogger` tests for unix socket based logger

Merge pull request #15132 from rgacogne/ddist-support-several-xdp-itf

dnsdist: Add support for multiple network interfaces in the XDP helper

dnsdist: Gracefully fall back to older versions of Quiche

Before this commit we only tried older versions of Quiche if the
`quiche` feature was not explicitely enabled, and failed otherwise.

Merge pull request #15128 from omoerbeek/rec-fw-recurse

rec: better explain how the recurse field works in various contexts

dnsdist: Only defaults to `eth0` when no interface has been set

Fix remaining naming in listener for TcpLogger test

Fix remaining listener naming in `RemotePoolLogger` TCP tests

dnsdist: Add support for multiple network interfaces in the XDP helper

Fix naming in listener for TcpLogger test

Fix listener naming in `RemotePoolLogger` TCP tests

Use integer placeholder in config template in `test_Dnstap.py`

Co-authored-by: Remi Gacogne <github@coredump.fr>

Apply suggestions from code review

Co-authored-by: Remi Gacogne <github@coredump.fr>

Add `RemotePoolLogger` tests for TCP based logger

Let pdns add-zone-key use defaults from pdns.conf whenever possible.

Fixes #3878

build-and-test-all.yml: do not test test-recursor-ubicloud-bulk for deb11 (libssl 1.1 vs 3)

Update docs for `connectionCount` parameter

Implement `RemoteLoggerPool::name`

Change `RemoteLoggerPool::toString` to describe just general stats

Replace locks in `remote_logger_pool` with an atomic counter

build-and-test-all.yml: skip dnsdist eBPF regression tests if skipXDP=True

build-and-test-all.yml: use venv for executing tasks

Add pyinvoke as pip requirement for builds

build-and-test-all.yml: use container runner for swagger-syntax-check

build-and-test-all.yml: enable build auth with meson on debian11

Merge pull request #15124 from miodvallat/rotd

auth-{4.7.5,4.8.5,4.9.4}: changelog entries + secpoll update

Do a consistency check of the recurse flag for the forward_zones case

Merge pull request #15126 from rgacogne/ddist-ignore-re2-broken-cflags

dnsdist: Ignore `re2`'s broken compilation flags

rec: better explain how the `recurse` field works in various contexts

Merge pull request #15125 from rgacogne/ddist-no-doh3-meson

dnsdist: Fix meson build failure when DoH3 support is disabled

dnsdist: Ignore `re2`'s broken compilation flags

Apparently some version(s) of `re2` pass `-std=c++11` in the result
of `pkg-config --cflags`, which helpfully downgrades us from C++17
to C++11. Let's just ignore what it says.

dnsdist: Fix meson build failure when DoH3 support is disabled

auth-{4.7.5,4.8.5,4.9.4}: changelog entries + secpoll update

Merge pull request #15083 from miodvallat/napster

Provide additional answers in NAPTR queries