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>

use min_element

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

clang-tidy: use std::min/max

Found with readability-use-std-min-max

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

Dedup only in specific places

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

dnsdist: add docs for new arguments for custom metric functions

Refactor serialize/wireFormatContent as suggested by @rgacogne

rename pdns::shuffle to pdns::shufleRecords, as suggested by @rgacogne

Add speedtest for shuffle, plus a speedup in shuffle itself

Faster dedup, not using zoneRepresentation but wire format, which allows for
an unordered_set as well.

Rework dedup code and add a test for pdsn::dedup

Adapt test to not use repeating records

No need to dedup the dns64 case seperately anymore

rec: dedup results from auths and results constructed ourselves

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.

statnode: boost > std optional

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

geoipbackend: boost > std optional

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

sdig: boost > std optional

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

ws-auth: boost > std optional

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

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

clang-tidy: simplify some algorithms

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

remove lexical_cast headers

These are unused

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

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