Apply Otto's suggestion

(cherry picked from commit 7f73a566805979f94bc1a23c9088372e00177bec)

misc: Switch to a std::array in makeHexDump()

(cherry picked from commit 8c7a1b8a671291e6ee2e7e4abdbdd41e9c714b31)

auth: Detect invalid bytes in makeBytesFromHex()

Also only allocate the required number of bytes, not twice that.

(cherry picked from commit 50953de897023742e43d3feab976b891be1c6e63)

Merge pull request #12055 from omoerbeek/backport-12054-to-rec-4.8.x

rec: Backport 12054 to rec 4.8.x: fix locked regr test

Rec: fix locked regr test

(cherry picked from commit 23064464fe298ac748470942de6e04a429be9a31)

Merge pull request #12051 from omoerbeek/rec-specialize-to-rec48

rec 4.8.x branch: Remove unneeded targets according to checklist

Remove unneeded targets according to checklist

Merge pull request #12041 from omoerbeek/rec-regr-test-locking

rec: Regression test for locked cache

Merge pull request #12047 from chbruyand/rec-dnstap-nod

rec: add support for NOD/UDR notifications using dnstap

Apply suggestions from code review

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

Fix unused function warnings if !ENABLE_NOD

Add dnstapNODFrameStream stats
Fix docs refs
Fix #ifdef

Doc tweaks

Tweaks: fix build with dnstabp disabled, process config changes

rec: add support for streaming NOD and UDR using dnstap

rec: make sure data is at least the size of a dnsheader when constructing a DnstapMessage

Merge pull request #12049 from omoerbeek/rec-followup-11903

rec: fix build with --disable-dnstap, followup to #11903

Merge pull request #12048 from omoerbeek/rec-config-default

rec: Fix --config (should be equal to --config=default),  followup to #11907

rec: fix build with --disable-dnstap, followup to #11903

Merge pull request #12046 from rgacogne/fix-portsmplexer

Fix compilation of the event ports multiplexer

rec: Fix --config (should be equal to --config=default),  followup to #11907

Fix compilation of the event ports multiplexer

Thanks to Jonathan Perkin for the patch!

Merge pull request #11960 from Habbie/builder-dispatch-release

expand builder-dispatch workflow for release building

Merge pull request #11954 from Habbie/axfr-primary-tc-servfail

auth AXFR server: abort on chunk with TC set

Merge pull request #12028 from mind04/pdns-edit-zone

auth: pdnsutil edit-zone, detect capitalization changes in LUA, TXT a…

Merge pull request #11983 from chbruyand/keyroller

auth: Keyroller first import

Merge pull request #12001 from rgacogne/ddist-mac-cache

dnsdist: Implement a small MAC address cache

Merge pull request #12039 from omoerbeek/dnsdist-coverity-20220928

Dnsdist: coverity 20220928

Merge pull request #11840 from rgacogne/ddist-doc-missing-json-objects-api

dnsdist: Document all the API JSON objects

Merge pull request #11732 from rgacogne/ddist-reg-tests-fd

dnsdist: Stop background threads and close file descriptors in regression tests

Merge pull request #12007 from rgacogne/ddist-cache-inspection

dnsdist: Add Lua bindings to look up domain and IP addresses from the cache

Merge pull request #11995 from rgacogne/ddist-more-discovery-tests

dnsdist: More backend discovery regression tests

Merge pull request #12000 from rgacogne/ddist-doh-skip-headers-alloc

dnsdist: Speed up DoH handling by preventing allocations and copies

Merge pull request #11903 from omoerbeek/pb-dnstap-stats

rec: protobuf and dnstap metrics

Process review commment from @fredmorcos, thanks!

rec: Regression test for locked cache

Fixes #11984

Mark a few methods non-const, to allow the guard to not be mutable, as suggested by @rgacogne

Rename prometheus remote logging metric, as suggested by @wojas

Fix auto_causes_copy coverity reports.

In once case we actually want a copy so annotate it.

Coverity 380098, 380102, 380103, 380106, 380107, 280109, 380111, 380112, 380113, 380117

Fix store_truncates_time_t coverity reports.

A few instances of store_truncates_time_t, almost all due to DNS specific protocol properties

Coverity 380097, 380099, 380100, 380101, 380105, 380108, 380114, 380115, 380116, 380118

Fix merge botch

fix botched help line

Reformat

Skip remote logger counts for normal stats retrieval

Prometheus stats plus tweaks in rec_control

Make sure things compile with !HAVE_FSTRM

Add statistics command for remote logging stats

Implement a common interface to get stats for both RemoteLogger and FrameStreamLogger

These two classes do not use the same mechanism to protect the stats from mult-threaded
access. Should maybe be revisited. Recursor has an instance per pthread, don't know
about dnsdist.

Merge pull request #12038 from omoerbeek/rec-aggr-nsec3-expired

rec: when an expired nsec3 entry is seen move it to the front of the expiry queue

Merge pull request #12026 from rgacogne/dist-faster-protocol-ctor

dnsdist: Speed up the dnsdist::Protocol() constructor

Merge pull request #12037 from rgacogne/ddist-only-rewrite-to-remove-if-needed

dnsdist: Rewrite packets to remove records only if actually needed

Update pdns/keyroller/pdnsapi/api.py

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

rec: when an expired nsec3 entry is seen, move it to the front of the expiry queue

Merge pull request #11949 from omoerbeek/rec-auth-rcode-counts

Rec: provide rcode received from auths metrics

Merge pull request #12027 from omoerbeek/rec-locked-auth-upgrade

rec: If new data is auth and existing data is not, replace even if cache locking is active

copy-paste error in comment

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

use at instead of []

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

Docs and account for forwarding

Add SNMP metrics for new counts

Provide metrics for rcodes returned bu auths.

Only basic rcodes (found in dns header) counted for now, as we
do not process extended error codes in OPT records.

Maybe this should be a histogram in the Prometheus case?

Merge pull request #11866 from omoerbeek/rec-proxymapping-stats

Rec: proxymapping stats

Merge pull request #11909 from omoerbeek/rec-dq-querytime

rec: add querytime attribute to dq

dnsdist: Rewrite packets to remove records only if actually needed

dnsdist: Add a dnsdist::MacAddress type

Merge pull request #12024 from omoerbeek/rec-getting-started-docs

rec: Update docs for getting started and compiling.

Better description of queryTime.tv_usec

Process comments from review

Approach: each thread gets its own copy of the PorxyMapping config, which
includes the counters. When we have a proxymapping match, we immediately
have the counters available without the need for a new lookup.

When reloading, make sure the old counters of lines present in the
new config are preserved.

Don't forget to count on PC cache hits.

Test for unauth to auth upgrade with locked cache

auth: pdnsutil edit-zone, detect capitalization changes in LUA, TXT and SPF records

auth AXFR server: abort on chunk with TC set

rec: If new data is auth and old dat is not, upgrade even if cache locking is active

Followup to #11958

Merge pull request #11987 from rgacogne/ddist-api-less-allocs

dnsdist: Slightly reduce the number of allocations in API calls

Merge pull request #11996 from rgacogne/ddist-coverage-dump

dnsdist: Better code coverage stats when enabled

Merge pull request #11993 from rgacogne/ddist-more-options-to-disable-features

dnsdist: Add build-time options to disable the dynamic blocks and UDP response delay

Merge pull request #11992 from rgacogne/ddist-thread-names

dnsdist: Add missing thread names

Merge pull request #11988 from rgacogne/ddist-openssl-no-err-strings

dnsdist: Add a build option (define) to prevent loading OpenSSL's errors

rec: Update docs for getting started and compiling.

I'm also making the mention of PowerDNS Recursor more consistent:
no ``the`` and either :program:`PowerDNS Recursor` or :program:`Recursor`.
Use short form if the long form was used already in a section or
it would become tedious to have the long form for another reason.

Fixes #11828, though the recursor.conf man page is not going to
happen anytime soon, as we lack the infrastructure for that and
maintaining both a .rst file and a man page would be a pain.

With released versions, pdns_recursor --config=default helps a bit.
In master pdns_recursor --config=[default|check|diff] was added.

Merge pull request #12020 from omoerbeek/rec-sl-command-line

rec: Make clear structured-logging-backend must be set on the command line

dnsdist: Use static arrays for the protocols, as suggested by Otto

dnsdist: Speed up dnsdist::Protocol() ctor

Merge pull request #11768 from franklouwers/feature/rpmbuilds-include

Enable include-dir by default in RPM builds, to be in line with DEB builds

Merge pull request #12021 from omoerbeek/rec-defer-accept-warning

rec: Properly guard  TCP_DEFER_ACCEPT

Merge pull request #12023 from phonedph1/patch-35

dnsdist: Update rules-actions.rst

Update rules-actions.rst

Properly guard  TCP_DEFER_ACCEPT

Merge pull request #11856 from omoerbeek/rec-no-more-xpf

rec: Remove XPF support

Merge pull request #11989 from fredmorcos/fred/11927-rec-stl-exception

Improve error message when invalid values for `local-address` are provided in recursor config file

Make clear structured-logging-backend must be set on the command line

Merge pull request #12013 from aerique/feature/add-rec-48-to-repo-script

Add `rec-48` to repo test script.

Merge pull request #12017 from omoerbeek/throw-long-label

Include (with precautions) offending data in messages about too long a label

Merge pull request #12011 from omoerbeek/rec-snmp-debian-ubuntu

rec: Enable SNMP for debian and ubuntu builds.

Merge pull request #12009 from omoerbeek/rec-snmp-used

Warn if snmp-agent is set but SNMP is not available.

Merge pull request #11959 from omoerbeek/rec-sl-tweaks

rec: A few tweaks to structured logging calls

Use naming convention: s/maxDNSNameLength/s_maxDNSNameLength

Move const maxDNSNameLength to class, and use it where appropriate

auth: Remove usage of parseService[4|6] and ServiceTuple

Cleanup

Formatting

rec: Remove the now unused parseService[4|6] and ServiceTuple