dnsdist: Document `dnsdist_ffi_stat_node_get_children_*` return children+node stats

The `dnsdist_ffi_stat_node_get_children_*` methods return the sum of
the queries or responses received for the children of a node AND the node
itself. It's quite unexpected but breaking the existing behaviour now would be painful.

Merge pull request #15031 from omoerbeek/rec-prep-5.2.0

rec: prep for rec-5.2.0

Merge pull request #15033 from omoerbeek/rec-rpz-tmp-file-deleter

rec: delete temp file on failure to dump RPZ file

Don't forget to delete the pointer itself

rec: delete temp file on failure to dump RPZ file

Merge pull request #15025 from miodvallat/dare-to-fail

pdnsutil add-zone-key: more checks

rec: prep for rec-5.2.0

Merge pull request #14996 from miodvallat/lua_finer_intervals

Lua up checks: finer control

Merge pull request #15004 from miodvallat/old_lmdb_schema_leftovers

[lmdb] Ignore unreachable TSIG keys in getTSIGKeys

Implement optional interval and minimumFailure options in if*up.

Merge pull request #10692 from n0tlu5/sulton/self-weighted

auth: added self weighted lua function

More clang-tidy conformance.

Merge pull request #15021 from omoerbeek/rec-daily-bulk

rec: run daily bulk test on ubicloud

Rename var to UBICLOUD_DAILY_REC_BULKTEST

Merge pull request #15028 from omoerbeek/dnsdist-fix-build-no-doh3

dnsdist: fix doh3-less compile

dnsdist: fix doh3-less compile

Merge pull request #15018 from omoerbeek/rec-bulktest-ubicloud

rec: add bigger (and v6 enabled) bulk test on ubicloud

Merge pull request #14617 from omoerbeek/rec-dedup-recs

rec: dedup records

Merge pull request #15026 from omoerbeek/dnsdist-testrunner-libssl

dnsdist:: add -lssl to testrunner link, needed on OpenBSD at least

We also want to run this new regression test on PRs, but those do not get the vars from the pdns repo

Merge pull request #14948 from neheb/ll

clang-tidy: simplify some algorithms

Merge pull request #14970 from neheb/jj

boost > std optional

Move workflow to misc-dailies and rename

dnsdist:: add -lssl to testrunner link, needed on OpenBSD at least

Separate speedtest setup, process review comments

Make add-zone-key try to retrieve it back to confirm operation.

Silence clang-tidy

Move add-zone-key logic to its own routine. NFC

Merge pull request #14946 from rgacogne/ddist-add-test-for-rewriting-response-code-via-lua

dnsdist: Add a regression test rewriting the response code via Lua

Merge pull request #15002 from rgacogne/ddist-doh3-bindings

dnsdist: Add DoH3 headers, query string, path and scheme bindings

Merge pull request #15023 from miodvallat/autoconf-ate-my-brackets

autohell regression fix

Merge pull request #15003 from rgacogne/ddist-regression-test-python3.13

dnsdist: Fix regression tests with Python 3.13

Remove spurious ] causing shell complaints during configure.

Make bigger and run on ubicloud

Use matrix

Run daily recursor bulk test using download from our repo

Conditionalize on if: ${{ vars.REC_BULKTEST_USE_UBICLOUD == '1' }}

rec: start bulk test on ubicloud (experiment)

Merge pull request #15019 from omoerbeek/rec-log-sockets

rec: explictly log port of listening addresses

rec: explictly log port of listening addresses

Merge pull request #14993 from BozhanL/fix7522

auth: createForward and createForward6 will use the zone_record as base

Merge pull request #15016 from Habbie/no-bool-explicit-cmp

coding guidelines: remove bool advice that conflicts with clang-tidy

Merge pull request #15017 from omoerbeek/rec-shared-ptr-followup

rec: followup to #14985: init shared pointers as get() might be passed a nullptr

rec: followup to #14985: init shared pointers as get() might be passed a nullptr

skip readability-identifier-length lint

Merge pull request #14937 from omoerbeek/rec-meson-ci

rec: start building rec also using meson in ci

better text

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

coding guidelines: remove bool advice that conflicts with clang-tidy

Typo in comment

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

auth: remove zoneid from luaSynth parameters

Merge pull request #14985 from omoerbeek/rec-cache-shared-authrecs

rec: store authority recs and signatures as shared pointers to const data

typo

Merge pull request #15010 from g0tar/taggedOnly

recursor: fixed behaviour of protobufServer(.. {taggedOnly=true})

Ignore unreachable TSIG keys in getTSIGKeys.

Pre-v5 database schema would (incorrectly) allow for multiple TSIG
keys to be created with the same name and algorithm. Once the database
gets converted to v5 schema, those redundant keys can no longer be used,
yet would appear in pdnsutil list-tsig-key output.

Change the list logic to only report reachable keys.

Fixes 14779

Process review comments from @rgacogne

Test the non-tagged packet cache hit case and test both UDP and TCP clients while there

recursor: outgoingProtobufServer() ignores taggedOnly protobuf option,
therefore do not annouce it; outgoing server is used as outgoingLoggers
in LWResult::Result asyncresolve(); after being introduced in
4898a34 filtering of outgoingP..S..() was not implemented since b790ef3

typo

rec: TCP counterpart to protobufServer(.. {taggedOnly=true}) fix

recursor: fix protobufServer(.. {taggedOnly=true}) logic for
cache-returned responses (problem existing since initial ac10822)
fixes #15007

dnsdist: Don't dereference a null pointer when accessing DoH{,3} headers via the FFI interface

dnsdist: Fix clang-tidy warnings

dnsdist: Add DoH3 headers, query string, path and scheme bindings

The DoH ones have been there for a long time, but the DoH3 ones were
missing. Note that we still don't have the ability to set a HTTP
response for DoH3 queries (including response maps) and SNI is still
missing (Quiche does not make that last one easy).

dnsdist: Fix regression tests with Python 3.13

The CA certificates that we are generating as par of our regression tests
were lacking the X.509 `Key Usage` extension, causing TLS validation with
Python 3.13 to fail with:

> certificate verify failed: CA cert does not include key usage extension

It appears that Python 3.13 enables `VERIFY_X509_STRICT` by default, which makes OpenSSL stricter, and thus it chokes on our invalid CA.

move nolint annotation to earliest param appearence

follow readability-implicit-bool-conversion lint

follow readability-braces-around-statements lint

skip readability-identifier-length lint

add pickselfweighted to expected spelling

Merge pull request #14997 from rgacogne/ddist-fix-doc-action-typos

dnsdist: Fix some small issues in the documentation for actions

dnsdist: Fix some small issues in the documentation for actions

Merge pull request #14955 from miodvallat/my_docs_look_better_than_yours

Documentation cleanup

add url param; copy ifurlup options doc

auth: createForward and createForward6 will use the zone_record as base
Closes #7522

adds selfweighted test with a set of A records

adds required checkurl url params

resets weight when service is declared as down

removes redundant first check flag; removes trailing whitespace

Merge pull request #14991 from omoerbeek/rec-janitor-at-work

rec: zap dangling symlink, leftover of #14926

rec: zap dangling symlink, leftover of #14926

Merge pull request #14987 from neheb/meson

use some structured binding

rename selfweighted to pickselfweighted; add pickselfweighted docs

use some structured binding

clang-tidy complains here about pointless std::move of a trivially
copyable type.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

also test sigs

Also handle sigs as a shared point to a vector

Add a speedtest like test, not enabled by default

This shows a modest increase in speed:

'null test' 0.11 seconds: 3222473473.3 runs/s, 0.00 us/run
'AuthRecordsTest' 0.10 seconds: 66.8 runs/s, 14977.88 us/run

to

'null test' 0.10 seconds: 3223627246.4 runs/s, 0.00 us/run
'AuthRecordsTest' 0.11 seconds: 72.1 runs/s, 13868.26 us/run

The test is: generate 100 records into the cache (each with an authRecord),
retrieve each record 100 times.

clang-tidy

Improvement: use const vector plus names for the various types used

rec: experiment: authRecs a shared pointer of DNSRecs instead of a vec of shared pointers to DNSRecs

Merge pull request #14980 from rgacogne/ddist198-changelog

dnsdist: Add ChangeLog and secpoll entry for DNSdist 1.9.8

Apply suggestions from Peter (update secpoll, proper quotes)

dnsdist: Add a missing entry into the 1.9.8 ChangeLog

secpoll: Update zone for DNSdist 1.9.8

dnsdist: Add ChangeLog for DNSdist 1.9.8

Merge pull request #14974 from Habbie/auth-4.9.3-secpoll-docs

auth-4.9.3: docs&secpoll

auth-4.9.3: docs&secpoll

Merge pull request #14973 from omoerbeek/rec-full-cname-chain-cached

rec: if the full CNAME chain leading to the answer is cached, indicate that

Add test

The last step of the test shows that there is likely room for more improvement.

Merge pull request #14956 from esensar/feat/metric-labels

dnsdist: add support for labels on custom metrics

if *fromCache is already true, no need to check CNAME chain

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