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>

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

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

dnsdist: Add a regression tests for whashed and chashed

Merge pull request #14936 from rgacogne/gate-daily-packages-build

CI: Run the daily package building workflow only if requested

Address review and clang-tidy comments.

CI: Run the daily package building workflow only if requested

After this commit, the `build-packages-daily-master` workflow will
only be executed on repositories that have a `SCHEDULED_PACKAGES_DAILY`
variable set to a non-negative value.

Handle empty ipv*hint values as "auto", when coming from lmdb.

Fixes #12653

Merge pull request #14926 from miodvallat/feature/12953

[feature] remove support for libdecaf

Add unit test for invalid NSEC3 hex blob.

Remove support for libdecaf.

Fixes #12953

rec: mark test that depend on external servers and or are unreliable

While there get rid of the special status of test_WellKnown.py and
correct the way args are passed to pytest

Merge pull request #14934 from omoerbeek/gh-action-build-name

GH workflow for building packages: change name so target isn't chopped off

dnsdist: Fix the consistency check for the balancing factors

Zero is the initial value, but until now it was only possible to pass a value greater than or equal to 1.0 to `setWeightedBalancingFactor()` so it was not possible to reset it to the default value.

dnsdist: Fix `whashed` and `wrandom` using the wrong parameter for consistent hashing

Merge pull request #14923 from belminf/master

dnsdist: Fix addMaintenanceCallback example

Merge pull request #14925 from milzi234/bugfix/recursor-docs-build

adjusted paths

adjusted paths

dnsdist: Fix addMaintenanceCallback example

Merge pull request #14922 from milzi234/bugfix/pdfs-in-wrong-location

put pdfs in latex subdirectory before uploading to web1

or remove the extra upload step :)

put pdfs in latex subdirectory before uploading to web1

GH workflow for building packages: change name so target isn't chopped off

Merge pull request #14921 from milzi234/bugfix/missing-pdfs

also add the PDFs to the websites

also add the PDFs to the websites

Merge pull request #14919 from milzi234/bugfix/untar-docs-before-pushing-to-web2

Fix for upload to web1

untar the html docs before syncing them to web1. Remove tar.bz2 of documentation sites

Merge pull request #14912 from PowerDNS/dependabot/github_actions/actions/setup-node-4

build(deps): bump actions/setup-node from 3 to 4

Merge pull request #14917 from omoerbeek/rec-dockerfile

rec: sync Dockerfile build options with packages

Merge pull request #14916 from omoerbeek/docs-modernize-urls

Fix a few URLs to point directly to the powerdns website

rec: sync Dockerfile build options with packages

Fixes #14915

Before:
Features: libcrypto-ecdsa libcrypto-ed25519 libcrypto-ed448 libcrypto-eddsa lua nod protobuf dnstap-framestream sodium curl scrypt

After:
Features: libcrypto-ecdsa libcrypto-ed25519 libcrypto-ed448 libcrypto-eddsa lua nod protobuf dnstap-framestream snmp sodium curl DoT scrypt

Fix a few URLs to point directly to the powerdns website

Merge pull request #14910 from milzi234/bugfix/push-docs-to-web1

Install inv before using it

build(deps): bump actions/setup-node from 3 to 4

Bumps [actions/setup-node](https://github.com/actions/setup-node) from 3 to 4.
- [Release notes](https://github.com/actions/setup-node/releases)
- [Commits](https://github.com/actions/setup-node/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/setup-node
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

Reject hexadecimal blobs with odd number of characters, fixes #12650.

Changes in 4e75f84ac9a0ad451d096d8c9cad58fd3b5574dd to allow ignoring
non-hexa characters (such as whitespace) have also allowed dangling
hex digits, which were then padded with an implicit 0. This restores
the original behaviour of throwing an exception for these inputs.

removes cargo culted lines

Install inv before using it

Merge pull request #14767 from milzi234/master

Publish Docs to Netlify and AWS

Merge pull request #14909 from omoerbeek/rec-startstop-fix

re: followup to 14796: also call stop hook in 1 thread case

Merge pull request #14892 from omoerbeek/rec-docs-runtime-quoting

rec: fix quoting issue in #14886

re: followup to 14796: also call stop hook in 1 thread case

Merge pull request #14904 from Habbie/repo-test-git-versions

generate-repo-files: pick up git versions from output too