Process review comments from @rgacogne, thanks!

Some cases of d_prefix vs prefix were missed previously

Fix VLOG in !rec case, gcc does not like #define X #error ...

Reformat

const correctness and better macro defined in logger.hh instead of two spots

Remove remains of g_dnssecLOG

It was never set from !recursor and has the same value as trace in
rec.  Now validationo logging goes the same  as other trace handling,
so trace-regex also shows validation.

Avoid recomputing prefix all the time, pass it as a argument if possible

Consistently use prefix and mark trace regex lines

Prefix helper method

Plumbing to let validate.cc and aggressive-nsec.cc use the recursor trace logging

Until now the logging of the DNSSEC code used it's own logging, which has
the consequence that prefixed, tracing and indenditng all worked differently
compared to the regular (trace) logging in rec.

So provide a construct to unify them without exposing syncres
internals to the validation code.

Merge pull request #12425 from omoerbeek/rec-forward-nord

rec: Change the way RD=0 forwarded queries are handled.

Merge pull request #12428 from rgacogne/moved-credentials

credentials: Explicitly clear credentials once they have been moved

credentials: Explicitly clear credentials once they have been moved

Merge pull request #12415 from rgacogne/ddist-improv

dnsdist: Small improvements

Change the way RD=0 forwarded queries are handled.

Since forever, there has been special case code for forwarded queries
in the RD=0 case.  This special case code does a hardcoded RD=0
query to the specified forwarder.  This code has two consequences:

1. Even if the forwarder is marked recursive it gets a RD=0 query
2. The cache is not consulted at all

The corresponding unit tests actually test this behaviour, but after
historic digging with help from @rgacogne it turns out the the unit
test do not reflect the desired functionality, but the current state
of affairs to help with a refactoring PR.  That is good, since
refactoring should not change functionality.

But now the time has come to change the code to do the desired thing:

1. If an RD=0 query is received, do a cache only-lookup in all cases.
2. Never send a RD=0 query to a recursive forwarder

I already did a similar thing when I wrote the QName Minimization
code, introducing a conditional that only gets set for that case,
to avoid changing unrelated (to QM) functionality.

Merge pull request #12381 from rgacogne/fortify-3

Enable FORTIFY_SOURCE=3 when supported by the compiler

Merge pull request #12420 from omoerbeek/rec-README

rec: README tweaks, .md does not know :program:`...`

Merge pull request #12407 from omoerbeek/rec-ecs-chaining

rec: do not chain ecs enabled queries

Merge pull request #12419 from omoerbeek/rec-close-fd-on-fdopen-fail

rec: negcache dump code: close fd on fdopen fail

rec: README tweaks, .md does not know :program:`...`

Merge pull request #12364 from rgacogne/rec-doc-compiling

rec: In compiling docs, warn about the GH tarballs, add deps from README

Merge pull request #12396 from omoerbeek/stringerror-thread-safe

Introduce a thread-safe version of stringerror()

Close fd on fdopen fail.

Originally suggested by @rgacogne, but not included in the merge of #12374
somehow.

Merge pull request #12412 from fredmorcos/openssl3-tls-mac

OpenSSL 3.0: TLS MAC handling

Merge pull request #12416 from Habbie/variable-mirror

build-and-test-all: add functionality to quickly switch ubuntu mirrors

build-and-test-all: add functionality to quickly switch ubuntu mirrors

Merge pull request #12405 from rgacogne/ddist-frontend-responses

dnsdist: Proper accounting of response and cache hits

Merge pull request #12386 from rgacogne/single-doh-acceptor

dnsdist: Merge the 'main' and 'client' DoH threads in single acceptor mode

Merge pull request #12384 from rgacogne/ddist-cap-ttl

dnsdist: Add the ability to cap the TTL of records after insertion into the cache

dnsdist: Add more unit tests for the packet overlay

m4: Fix interleaved output during `FORTIFY_SOURCE` detection

Merge pull request #12411 from fredmorcos/openssl3-ipcipher-ca6-crypto

Support OpenSSL 3.0 for ipcipher CA6 encryption/decryption

OpenSSL 3.0: TLS MAC handling

libssl: Cleanup TLS-related code

libssl: Formatting TLS-related code

dnsdist: Appease the formatting gods

Merge pull request #12403 from rgacogne/spell-allow-eaca

spell-checker: Allow EACA (from the PGP public keys in the docs)

dnsdist: Use smaller event buffers in our socket multiplexers

dnsdist: Silence a warning about an unused variable in discovery.cc

dnsdist: Remove a duplicated ';' in dnsdist-lua-actions.cc

credentials: Add more unit tests

OpenSSL 3.0: decryptCA6

OpenSSL 3.0: encryptCA6

spell-checker: Allow prometheus values

dnsdist: Fix a typo in the prometheus type

dnsdist: Update the prometheus sample in the documentation

It was very old and not up-to-date.

dnsdist: Only record one hit or miss per query in the cache metrics

The scope-zero feature and the DoH paths can actually do more than
one lookup per query, and until now this led to an increase of the
per-cache metric for every lookup, while the global `cache-hits`
and `cache-misses` metrics were only updated once per query.
This has led to several questions and misunderstandings, so we now
only update the per-cache metrics once per query as well.

dnsdist: Properly record cache-hits as responses

For a very long time we have not been adding cache-hits to the
responses counter, which is wrong. Let's fix it now.

Cleanup ipcipher.hh

dnsdist: Merge the 'main' and 'client' DoH threads

When we are in "single acceptor thread" mode, merge the 'main' and
'client' DoH threads into a single one. We use separate threads to
reduce the separate the handling of the HTTP/2 traffic from the DNS
handling, to reduce latency, but that does not really make sense on
small devices with a single, limited CPU core. On these we prefer
using as few threads as possible to reduce the context switches and
the memory usage.

dnsdist: Remove ttlCapTypes from the internal state

dnsdist: Use unordered sets to speed things up a bit

dnsdist: Implement the ability to cap TTLs for some record types only

dnsdist: Add MaxReturnedTTLResponseAction to cap the TTL after packet cache

Merge pull request #12383 from rgacogne/ddist-stronger-udp-path

dnsdist: Stronger guarantees against data race in the UDP path

Merge pull request #12401 from fredmorcos/openssl-3-prep

Openssl 3 prep work

dnsdist: Really fix the formatting this time

dnsdist: Fix formatting

dnsdist: Apply suggestions from Otto's code review (thanks!)

dnsdist: Set a proper HTTP error code on Proxy Protocol failure

dnsdist: Stronger guarantees against data race in the UDP path

Merge pull request #12410 from rgacogne/unbreak-ci

CI: Work-around the "503  Service Unavailable" erros on azure.archive.ubuntu.com

Merge pull request #12402 from rgacogne/ddist-dq-time

dnsdist: Add bindings for the current and query times in DQ/DR

CI: Work-around the "503  Service Unavailable" erros on azure.archive.ubuntu.com

Merge pull request #12400 from rgacogne/ddist-reduce-ttl

dnsdist: Add SetReducedTTLResponseAction

dnsdist: Add Lua bindings for the current time and query time

Print compiler version at the end of configure

dnsdist: Better description of SetReducedTTLResponseAction

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>

Include failure message when creating DSRCs

Lint cleanups of test-signers.cc

OpenSSL signers test for getPubKeyHash()

Cleanup DNSCryptoKeyEngine

Print checkKey() error messages in signer tests

Cleanup test_generic_signers in test-signers.cc

Cleanup of ipcipher.cc

Cleanup includes in misc.cc

Add OpenSSL error handler

Merge pull request #12385 from rgacogne/ddist-ffi-metrics

dnsdist: Add a Lua FFI interface for metrics

Reformat

Better wording of reason to not chain ECS enabled queries

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

Typo in comment

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

rec: do not chain ecs enabled queries.

asyncresolve() assumes the recieved ecs info corresponds to the one sent out.

Merge pull request #12387 from rgacogne/ddist-oom-udp

dnsdist: Handle out-of-memory exceptions in the UDP receiver thread

Merge pull request #12399 from omoerbeek/rec-threadnames

rec: Name recursor threads consistently with a "rec/" prefix.

Merge pull request #12392 from omoerbeek/bind-errno-cleanup

Be more careful saving errno in makeClientSocket() and closesocket()

Merge pull request #12367 from jsoref/pdns_control-ccounts-comma

Add missing comma to pdns_control ccounts

spell-checker: Allow EACA (from the PGP public keys in the docs)

Formatting of various files like ipcipher and opensslsigners

Relax clang-tidy regarding magic numbers

Add _build dir to docs/.gitignore

Name recursor threads consistently with a "rec/" prefix.

Use thread names without capitals, as they look a bit ugly otherwise.

Threads started by libfstrm are not named, as they are created internally by the lib.

Partly supsersedes #11138

Merge pull request #12397 from rgacogne/fix-cachecleaner

cachecleaner: Add cmath, needed for std::ceil and std::round

Introduce a thread-safe version of stringerror()

It turns out we already have the code, so call it.

Merge pull request #12354 from rgacogne/ddist-unique-ednsoptions

dnsdist: Move DNSQuestion's EDNS options to a unique pointer

cachecleaner: Add cmath, needed for std::ceil and std::round

dnsdist: Move the exception handling out of handleOnePacket()

dnsdist: Use EDNSOptionViewMap in more places

As suggested by Otto.

Merge pull request #12365 from rgacogne/ddist-fix-tcp-queued-underflow-12357

dnsdist: Prevent an underflow of the TCP d_queued counter

Add an --enable-fortify-source configure option

The option defaults to 2 to keep the existing behaviour, but 3 and
auto are supported, with auto trying to select the highest version
supported by the compiler.

Merge pull request #12391 from omoerbeek/dnsdist-pgpkeys

dnsdist: update pgp keys listed in docs: add Otto, move Pieter and Bert to old keys