rec: explictly log port of listening addresses

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

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

rec: start building rec also using meson in ci

Typo in comment

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

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

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

Merge pull request #15010 from g0tar/taggedOnly

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

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

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

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

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>

dnsdist: fix typos in custom metrics docs

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

dnsdist: set version changed for new custom metrics options in docs

Merge pull request #14935 from omoerbeek/rec-regr-test-markers

rec: regression test markers

Merge pull request #14967 from rgacogne/ddist-fix-ecs-zero-scope-incoming-doh

dnsdist: Fix ECS zero-scope with incoming DoH queries

dnsdist: fix empty labels check in json stats endpoint

dnsdist: use `empty()` instead of "" check

dnsdist: add docs for new arguments for custom metric functions

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

Alternative approach to #14918

dnsdist: update metrics related regression tests

Merge pull request #14931 from miodvallat/bugfix/12653

Fix ipv*hint=auto on lmdb backend

Needs C++17 nowadays.

Update backend writer's guide.

Tweaks to get manpages to render better.

Make the Prometheus scrape example visible.

Apeaze spayl tcheker

Update external urls

Update last TAI change date.

DNSSEC: Change to primary/secondary wording and mention RFC9077.

Use current settings names rather than the ones deprecated in 4.5.

Improve layout/rendering, appease the grammar gods, various tweaks.

dnsdist: ignore clangd-tidy warnings

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

dnsdist: Fix ECS zero-scope with incoming DoH queries

The zero-scope feature involves a first cache lookup before the ECS
information has been added to the query, then on a miss a second,
regular lookup is done. When we get a response from the backend that
contains an ECS scope set to 0, we can insert it into the cache in a
way that allows using it for all clients, but we must be careful to
use the key that was computed during the first lookup, and not the
second one.
Incoming DoH queries make that even more interesting because while
they are received over TCP, they are initially forwarded to the
backend over UDP but can be retried over TCP if a TC=1 answer is
received. In that case we must be very careful not to insert the
answer into the cache using the wrong protocol, as we don't want to
serve a TC=1 answer to a client contacting us over TCP, for example.
The computation of the cache key and protocol was unfortunately broken
for the incoming query received over DoH, forwarded over UDP and
response has a zero scope case. This commit fixes it.

Merge pull request #14964 from omoerbeek/rec-prep-5.2.0-rc1

rec: prep for rec-5.2.0-rc1

dnsdist: remove todo comments from `dnsdist-lua-ffi`

rec: prep for rec-5.2.0-rc1

Merge pull request #14957 from zjs604381586/avoid-local-port

recursor udp-source-port-avoid default setting add 4791 port

Update pdns/recursordist/settings/table.py

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

Add versionchanged clause

sort udp_source_port_avoid  list

recursor local udp port avoid 4791

When the recursor sends a DNS request and the local port is specified as 4791, if there is RDMA hardware on the machine, when the response with dst as port 4791 reaches the network card driver, it will be directly processed by the hardware as Roce v2 protocol, and the recursor will not get a response to this request.

Merge pull request #14944 from omoerbeek/rec-docs-assorted

rec: minor doc fixes

dnsdist: skip metrics with labels for carbon export and stats dump

dnsdist: clean up clang-tidy warnings in lua and metrics

dnsdist: use template function to simplify metrics mutation functions

dnsdist: fix formatting in dnsdist-lua.cc

dnsdist: remove unnecessary optional around variant opts

chore: reformat metrics related files

dnsdist: add options to `declareMetric` to skip initialization if labels are used

dnsdist: add support for labels on custom metrics

This adds support for prometheus labels on custom metrics. Changes the
metrics maps to hold a map of labels to metric values for each metric.
Metrics without labels have an empty string label combination value.

Adds an optional options table as the last parameter of `incMetric`,
`decMetric`, `setMetric` and `getMetric`, which can be used to set
labels by using the `label` key or to set the previously used parameter
(either `step` or `value`).

Closes: #13359

Extend the SVCB tests to also test with lmdb backend.

Merge pull request #14950 from omoerbeek/docs-top-searchbox

web docs: Render search box on top.

web docs: Render search box on top.

Also uses the proper style for the name of the beast, except for
dnsdist which uses a different theme that does not have logo-text.html

Merge pull request #14942 from omoerbeek/rec-sockets-less-chatty

rec: log only one line per protocol for listening sockets,

Merge pull request #14941 from omoerbeek/rec-meson-gen-metrics-fixup

rec: generate metrics files (also) where meson expects them

Add comments about the prupose of the added arguments to make{UDP,TCP}ServerSockets

Add FIXME comment

Merge pull request #14947 from rgacogne/ddist-doc-dq-elapsedus

dnsdist: Document that `DNSQuestion::getElapsedUs` will be added in 1.9.8

dnsdist: Document that `DNSQuestion::getElapsedUs` will be added in 1.9.8

Merge pull request #14938 from karelbilek/keylogfile_master

dnsdist: allow setting keyLogFile to DoT/DoH backends

rec: minor doc fixes

Fixes #14860

Merge pull request #14930 from rgacogne/ddist-fix-WeightedBalancingFactor

dnsdist: Fix issues with the balancing factors

Simple regression test for existence of SSL keys

There is no test for actual validity of the files... just if there is something there.

Allow setting keyLogFile to DoT/DoH backends

Merge pull request #14943 from omoerbeek/rec-ds-zonecut-invalid

rec: Skip the current zone when looking for a cut after an invalid DS denial proof

rec: Skip the current zone when looking for a cut after an invalid DS denial proof

rec: log only one line per protocol for listening sockets,

Instead of addresses * threads * protocol lines

Update pdns/recursordist/metrics.py

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

Merge pull request #14913 from miodvallat/bugfix/12650

[bugfix] Reject hexadecimal blobs with odd number of characters

rec: generate metrics files (also) where meson expects them

This avoids unneccesary regeneration of files

Change back to default concurrency

fix meson setup

rec: also build using meson in CI

dnsdist: Add a regression tests for whashed and chashed