Fix typo

(cherry picked from commit 68dbf30cebc92d29853314b1160257a9dbc5f4f1)

Merge pull request #12163 from omoerbeek/backport-12155-to-rec-4.8.x

rec: Backport 12155 to rec 4.8.x: Fix snmp OID numbers for rcode stats

Merge pull request #12162 from omoerbeek/backport-12122-to-rec-4.8.x

rec: Backport 12122 to rec 4.8.x: Implement output operator for QTypes

Merge pull request #12161 from omoerbeek/backport-12125-to-rec-4.8.x

rec: Backport 12125 to rec 4.8.x: handle ixfr connect and transfer timeouts

Typo

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>
(cherry picked from commit 38b69143124de1d2f7341148c6250e821c02e4be)

Clarify Prometheus names

(cherry picked from commit fe6797befcdb8b9287ac87dacc796282920e7001)

Fix snmp OID numbers for rcode stats

(cherry picked from commit bdc4ee0a00ad913a4e0d909f4bfc4e1a404ead68)

Pass an object as const ref

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>
(cherry picked from commit fa61cf82d85f5e26178853e9b0f4be14f55565db)

Implement output operator for QTypes.

This fixes #12089

The theory says there should be no unwanted side effects, but I'd like
that to be validated independently.

(cherry picked from commit da19e4feb842464659536076b47047b2910c73fb)

Better wording in comment

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>
(cherry picked from commit 240460d77be35a6a1c1e6fa22364efe19dc3ee84)

Timout handling for ixfrs as a client.

One complicating factor is that this is shared code, but auth and
rec do not agree on the definiton of the timeout value: auth states
it is a maximum idle time, while rec state it is the total xfr time.
While both apporaches make sense and in the end we would like to
enforce both, we now go for a more simple solution that respects
auth or rec behaviour based on a flag.

(cherry picked from commit fee334ae0f5083d47f9adc207d5a1a6d36ebc2ac)

Merge pull request #12146 from omoerbeek/backport-12063-to-rec-4.8.x

rec: backport 12063 to 4.8.x: Also do the compare for protobuf logger config objects.

Merge pull request #12150 from omoerbeek/backport-12140-to-rec-4.8.x

rec: Backport 12140 to rec 4.8.x: be more lenient replacing auth by non-auth records in cache

Merge pull request #12145 from omoerbeek/backport-12081-to-rec-4.8.x

rec: backport 12081 to rec-4.8.x: Log invalid RPZ content when obtained via IXFR

Merge pull request #12144 from omoerbeek/backport-12133-to-rec-4.8.x

rec: backport 12054 to rec-4.8.x: Fix assertAlmostEqual call to compare absolute difference

Merge pull request #12147 from omoerbeek/backport-12066-to-rec-4.8.x

rec: backport 12066 to rec 4.8.x: Detect invalid bytes in makeBytesFromHex()

typo in comment

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>
(cherry picked from commit 681a49f10b00ba7ee38a55402953d076c7221160)

Add test

(cherry picked from commit f4621cce40bdc1648edf82717239ef040eedb9b6)

QM can get confused by expiring infra records, so be a bit more lenient
with replacing auth records by unauth, as unauth (typically infra
records) might come in while we are resolving.

Should fix #12078

(cherry picked from commit 1ce6006d7ddc925a13b868488b6e987bca41cc90)

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)

Also do the compare for protobuf logger config objects.

I'm not doing the async part now, as tsan reports there would be a race.
The case occuring the most: no changes is now handled correctly and quickly and that is the main thing.

(cherry picked from commit babe943035818b7a97e59cfd70921ba06bbf31cf)

rec: Log invalid RPZ content when obtained via IXFR

That kind of content was properly logged and handled when received
during the initial loading (AXFR) but not when received via an
incremental update.

(cherry picked from commit 55a99233728fc01e3946a97fb8dbb073a3003622)

Fix assertAlmostEqual call to compare absolute difference

(cherry picked from commit 67dd4251f758a695dab05e206a32199ccee6c214)
(cherry picked from commit 23064464fe298ac748470942de6e04a429be9a31)

Backport of #12054 and #12133

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.