Process review comments

Prometheus does not like counter names ending in -count

(Partial) Tidy

Extend SNMP test to include new OIDs

Add Prometheus and SNMP version of NOD/UDR metrics

Keep track of metrics for NOD and UDR events.

While there, change level of ndr logs to Notice, it was Debug before

Merge pull request #12793 from omoerbeek/assorted-delint

Another set of delinting

Merge pull request #12867 from omoerbeek/rec-prep-4.9.0-beta1

rec: Prep for rec-4.9.0-beta1

Prep for rec-4.9.0-beta1

Merge pull request #12880 from Habbie/swagger-no-docker

swagger-syntax-check: run outside of Docker

swagger-syntax-check: run outside of Docker

Merge pull request #12875 from Habbie/upgrade-notes-46-47-master

auth upgrading: fix 4.6/4.7 header; update LMDB text

Merge pull request #12856 from romeroalx/enable-ipv6-actions-container

Enable ipv6 for actions running on docker containers

auth upgrading: fix 4.6/4.7 header; update LMDB text

Merge pull request #12874 from Habbie/auth-4.8.0-docs-secpoll

auth-4.8.0: docs and secpoll

auth-4.8.0: docs and secpoll

Merge pull request #12873 from aerique/feature/add-bookwork-to-repo-test-script

Add Bookworm to repo test script.

Add Bookworm to repo test script.

Add Debian Bookworm for `auth-master` & `auth-48` to repo test script.

Merge pull request #12866 from Habbie/auth-lmdb-dup-delete

auth lmdb: delete duplicate domain entries in deleteDomain

Merge pull request #12860 from Habbie/auth-ent-wildcard

auth: do not answer with broken TYPE0 data when expanding an ENT wildcard

Merge pull request #12859 from Habbie/pdnsutil-edit-zone-wrong-key

pdnsutil: if user pushes unknown key in response to "problem with zone" prompt, do not throw away their changes

auth: do not answer with broken TYPE0 data when expanding an ENT wildcard

Process review comments

Reformat

More delinting

A set of random files made clean. During this process .clang-tidy.full
was also amended a bit.

document shards=1 enforcement

format

simplify

Merge pull request #12862 from omoerbeek/rec-recursion-bound

rec: bound maximum recursion depth to 16.

Merge pull request #12861 from omoerbeek/rec-no-refresh

rec: Introduce a way to completely disable root-refresh

use d_transactiondomain[id]

Update modules/lmdbbackend/lmdbbackend.cc

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

only throw when idvec is empty

Merge pull request #12673 from omoerbeek/rec-origttl

rec: Sanitize d_orig_ttl stored in record cache

auth lmdb: enforce shards=1 when running in lightning-stream mode

auth lmdb: delete duplicate domain entries in deleteDomain

Add the d_orig_ttl sanitization back, there still is a case where
it can wrap (which I'm unable to spot right now).

Merge pull request #12863 from omoerbeek/rec-val-log-typo

rec: fix typo in trace message

Merge pull request #12838 from omoerbeek/delint-rec-tcp

Delint rec-tcp.cc

Merge pull request #12837 from omoerbeek/nolint-validate

Delint validate.cc and related files

Merge pull request #12836 from omoerbeek/rec-main-delint-followup

rec: Full delint rec-main.cc

Fix root-priming test, which now depends on a arg being set.

rec: fix typo in trace message

rec: bound maximum recursion depth to 16.

Before #12779, the fixed limit on CNAME chain length (16) effectively
worked as recursion depth limit.

rec: Introduce a way to completely disable root-refresh

pdnsutil: if user pushes unknown key in response to "problem with zone" prompt, do not throw away their changes

gh actions: enable ipv6 in docker containers

Merge pull request #12726 from rgacogne/ddist-wait-reconnect

dnsdist: Properly handle reconnection failure for backend UDP sockets

Merge pull request #12819 from rgacogne/ddist-reuseaddr-udp

dnsdist: Stop setting SO_REUSEADDR on outgoing UDP client sockets

Tweaks

Reformat

Delint rec-tcp.cc

Delint validate.cc and related files

Merge pull request #12835 from romeroalx/fix-clang-misc

GH actions - misc-dailies.yaml: set clang version for Ubuntu-20

Full delint rec-main.cc

Merge pull request #12834 from jpmens/patch-14

Be more specific in describing config-dir

GH actions - misc-dailies: Set clang version for Ubuntu-20

Merge pull request #12833 from romeroalx/fix-cifuzz-apt-repo

Changing apt mirror for cifuzz base-builder image

Merge pull request #12810 from romeroalx/actions-debian-container

Run build-and-test-all.yml on a custom Debian 11 container instead of an Ubuntu Runner VM

Be more specific in describing config-dir

I think this more clearly describes the option

set apt mirror for cifuzz base builder image

Run build-and-test-all.yml on a custom debian container

delint updateCacheFromRecords

Record time the ttd was computed instead of compensating

Alternate approach: remmeber the actual orig_ttl. A bit harded than you think, as
in somce cases TTDs are modified later.

Sanitize d_orig_ttl

The computed orig_ttl can be wrong if d_now.tv_sec is more recent
than the d_now.tv_sec the ttd was computed from. This can happen
if we went out to get e.g. keys and that took time. d_orig_ttl wraps
around in that case if the TTL value was smaller than that delay.
Work around that and make sure d_orig_ttl is within the legal range.

Merge pull request #12829 from omoerbeek/fix-log-fail

rec: fix clang-tidy botch wrt spelling of "log-fail"

Merge pull request #12830 from omoerbeek/secpoll-ubuntu-mirror

secpoll check: use preferred ubuntu mirror

secpoll check: use preferred ubuntu mirror

Merge pull request #12831 from Habbie/apt-no-qq

Fix error retrieving firefox package from ubuntu repo by removing it.

get rid of firefox

more apt no -qq

apt no qq

Merge pull request #12825 from omoerbeek/rec-fix-simpledottest

rec: Fix DoT test to no use www.powerdns.com, as it changed

rec: fix clang-tidy botch wrt spelling of "log-fail"

Followup to #12790

Fix DoT test to no use www.powerdns.com, as it changed

dnsdist: Really disable REUSEADDR on outgoing UDP client sockets

It turns out that our `Socket::bind()` will _helpfully_ enable this
by default.

Merge pull request #12779 from omoerbeek/rec-cname-length-vs-depth

rec: Distinguish between recursion depth and cname length

Merge pull request #12818 from rgacogne/cang-tidy-inc-cognitive-complexity

clang-tidy: Increase the cognitive complexity threshold to 75

dnsdist: Stop setting SO_REUSEADDR on outgoing UDP client sockets

`SO_REUSEADDR` is useful on TCP server sockets to allow binding quickly
after restarting the process without waiting `TIME_WAIT` seconds, or
to allow some port reuse on BSD. It also allows reusing a port more
quickly for TCP client sockets.
For UDP sockets, however, Linux allows two sockets to be bound to the
same address and port, and will distribute all packets to the most
recent socket, which is very unexpected, to say the least.

-cppcoreguidelines-avoid-non-const-global-variables

NOLINT(readability-function-cognitive-complexity) markers

As an experiment, just fix the two reported cases of performance-unnecessary-value-param

rec: Distinguish between recursion depth and cname length, they are not the same

Merge pull request #12790 from omoerbeek/rec-lint

rec: delint rec-main.cc wrt high complexity

Merge pull request #12750 from omoerbeek/rec-log-variable

rec: Log if the answer was marked variable by SyncRes and if it was stored into  the packet cache (if !quiet)

Also log in the non-structured logging case

Process review comments

clang-tidy: Increase the cognitive complexity threshold to 75

The current level of 50 is too low for a function like
'handleQueuedHealthChecks' which is quite simple.

Merge pull request #12780 from zeha/workaround11804

auth: add setting workaround-11804-max-chunk-records

Merge pull request #12813 from rgacogne/ci-disable-clang-tidy-non-const-global-vars

clang-tidy: Disable cppcoreguidelines-avoid-non-const-global-variables

Apply changes for doc nits

clang-tidy: Disable cppcoreguidelines-avoid-non-const-global-variables

While I agree that we should try to avoid non-const global vars
as much as possible, this rule kills the boost testing framework
so we cannot keep it.

dnsdist: Add a TSAN suppression for the backend re-connection code

dnsdist: check that the UDP responder thread is running after reconnecting

dnsdist: Only log failed backend connection attempt at info on first try

dnsdist: Properly handle reconnection failure for backend UDP sockets

We try to reconnect our UDP sockets toward backends on some kind of
network errors that indicate a topology change, but we need to be
careful to handle the case where we actually fail to reconnect, as
we end up with no remaining sockets to use.
This commit properly deals with this case by pausing the thread handling
UDP responses from the backend, instead of having it enter a busy loop,
and by attempting to reconnect if we get a `bad file number` error when
trying to send a UDP datagram to the backend.

Merge pull request #12812 from rgacogne/ci-clang-tidy-disable-union-warning

clang-tidy: Disable cppcoreguidelines-special-member-functions

Merge pull request #12712 from Habbie/move-keyblock

docs: update GPG keyblock link

clang-tidy: Disable cppcoreguidelines-special-member-functions

We will need to re-enable it once https://github.com/PowerDNS/pdns/issues/12809
has been fixed.