rec: Bow to formatting gods

(cherry picked from commit 68ffd18c09061cca424cf2bd5ab4305b8271cfd8)

Fix braces formatting in pdns/recursordist/test-syncres_cc.cc

Co-Authored-By: Otto Moerbeek <otto.moerbeek@open-xchange.com>
(cherry picked from commit 6850fa5d095bf2bb4251a6bbf9f763a2116f5c6a)

rec: Add unit tests for the NSEC3 Opt-Out case

An Opt-Out NSEC3 only proves that there is no delegation, so we
should not consider a DS NODATA or a NXDOMAIN proved by that RR
secure but insecure.
This was fixed in 18c8faae6c67f734583c5c881d0d083d3253b49e and this
commit adds a few unit tests to cover the fix.

(cherry picked from commit c179741988b9273b12c15a6b318ff0a43fe50081)

Merge pull request #8704 from omoerbeek/backport-8702-to-rec-4.3.x

rec: Backport 8702 to 4.3.x: Add the source and destination ports to the protobuf msg

Handle source and destination ports in the sample protobuf logger

(cherry picked from commit f39c1ce956d4bf3fa73d08d996b638e3a8221859)

rec: Add the source port to protobuf messages for incoming queries

(cherry picked from commit 5d2e9a839cd59911530bf124badfa239304af5b6)

dnsdist: Add the source and destination ports to the protobuf msg

(cherry picked from commit 36de33306ac5bfa4f55e26975f35fb35a90914da)

Merge pull request #8673 from omoerbeek/backport-8639-to-rec-4.3.x

rec: backport 8639 to 4.3.x: debian postinst / do not fail on user creation if it already exists

Merge pull request #8685 from omoerbeek/backport-8684-to-rec-4.3.x

rec: backport 8684 to 4.3.x: parsing `dont-throttle-names` and `dont-throttle-netmasks` as comma separated lists

Merge pull request #8692 from omoerbeek/backport-8665-to-rec-4.3.x

rec: backport 8665 to 4.3.x: An Opt-Out NSEC3 RR only proves that there is no secure delegation

Merge pull request #8695 from omoerbeek/backport-8694-to-rec-4.3.x

rec: backport 8694 to 4.3.x: Fix ./syncres.hh:228:20: warning: initialized lambda captures are a C++14 extension

Fix ./syncres.hh:228:20: warning: initialized lambda captures are a C++14 extension

(cherry picked from commit 7b07a07cce7b9476ef602b9e9b35d30a239bdc96)

rec: An Opt-Out NSEC3 RR only proves that there is no secure delegation

(cherry picked from commit 18c8faae6c67f734583c5c881d0d083d3253b49e)

Merge pull request #8670 from omoerbeek/backport-8648-to-rec-4.3.x

rec: backport 8648 to rec 4.3.x: fix wrong zoneCuts caused by cache only lookup

parsing `dont-throttle-names` and `dont-throttle-netmasks` as comma separated lists

(cherry picked from commit 5285831402f3f3a209c16cb3bd743eea640cf894)

Merge pull request #8675 from omoerbeek/backport-8647-to-rec-4.3.x

rec: Backport 8647 to rec 4.3.x: increase default maxqperq

correct cast

Co-Authored-By: Remi Gacogne <rgacogne@users.noreply.github.com>
(cherry picked from commit 13c43bdd982b2a6df7096a167b0c32c3cc807a8d)

Slightly different approach as suggested by rgacogne: if
qname-minimization is active, force maxqperq to be >= 100.

(cherry picked from commit edea1bf7e1213f9643bd0ce29cb81d4413774d98)

Set default maxqperq to 60, and allow for extra if qname-minimization is on.

(cherry picked from commit 409b839837f174a1ca1e614dc7cd66f581e4c1be)

Bump max-qperq default to 100; this is enough for rev v6 queries with cold
cache with validations and qname minimization enabled.  Fixes #8646.

(cherry picked from commit 15d63d5a4a17aece479be953651643736a26d3d0)

rec: debian postinst / do not fail on user creation if it already exists

(cherry picked from commit fddad2718fb994d4fd016c03e1e05ef6c67aae14)

Introduce test for PR 8648.

While there, explicitly init qname-minimization in test-syncres to
false.  The current code gave the impression it was turned on by
default for unit tests.  We have a lot of test that count queries,
and that is highly depedent on qname minimization being on or off.

QName Minimization consults the cache first to see work needs to
be done. Via doCacheCheck() this can lead to a call to computeZoneCuts()
which might/does not work properly on only cached records, so always
make sure it can go out if needed. Should fix #8642.

(cherry picked from commit 0eb07f3a5c23cc94c95a2bed022367085f8dfcef)

Merge pull request #8539 from pieterlexis/4.3-docs

Auth 4.3.0 docs

auth 4.3.0-alpha1 secpoll

document local-ipv6 removal in 4.3

auth 4.3 changelog

git-to-changelog-merges: remove trailing spaces

EL pkgs: chown dirs to service user

Add auth 4.3.0 upgrade docs

Merge pull request #8571 from omoerbeek/rec-better-time-based

Rec: better time based data structures

Merge pull request #8599 from aerique/feature/update-docker-repo-script

Update docker repo script for auth-4.2.1.

Merge pull request #8249 from chbruyand/lua-monitoring-thread

auth: refactor health checks monitoring done from LUA records

Merge pull request #8497 from mind04/pdns-api2

Auth: api: cache and rectify fixes in patchZone()

Fix CentOS 8 issue and install `dig`.

Update docker repo script for rec-4.2.1.

Merge pull request #8611 from omoerbeek/rec-prep-4.2.1

Prepare for rec-4.2.1

remove unused variable

add qname/qtype to LUA Lua errors

auth: lua monitoring, lock on status change, better report malformed options content

auth: lua refactor health checks monitoring

Merge pull request #8512 from rgacogne/dedup-rrs-signature

Remove duplicate RRs inside a RRSet when computing the signature

Rebased; test files were reformatted

rec: Check duplicated records inside a RRSet validate correctly

Remove duplicate RRs inside a RRSet when computing the signature

These duplicates should not exist but they do happen in the wild.
Since we need to sort the RRs anyway, let's be nice and remove
duplicates as well.

Prepare for rec-4.2.1

Merge pull request #8435 from mind04/pdns-remove-opendbx

auth: remove opendbx backend

Merge pull request #8603 from omoerbeek/rec-prep-4.1.15

rec: Prep for rec-4.1.15

Merge pull request #8601 from omoerbeek/no-copy-rwlocks

Avoid copying of pthread_rwlock_t

Merge pull request #8607 from rgacogne/ddist-doc-zeroscope-parseecs

dnsdist: Document that the 'zero-scope' feature requires 'parseECS'

Merge pull request #8027 from Habbie/sdig-stdin-exitvalue

sdig: decode DNS packets from stdin; exit non-zero on failure

Merge pull request #8604 from rgacogne/ddist-kvs-errors

dnsdist: Add missing exception message in KVS error

Merge pull request #8605 from rgacogne/ddist-tuning-doc-typo

dnsdist: Add missing newline in setMaxUDPOutstanding() documentation

dnsdist: Document that the 'zero-scope' feature requires 'parseECS'

dnsdist: Add missing newline in setMaxUDPOutstanding() documentation

dnsdist: Add missing exception message in KVS error

Merge pull request #8585 from pieterlexis/auth-api-lenient

Auth API: be a bit more lenient

Prep for rec-4.1.15

sdig: support reading packets from stdin

fix formatting

fix typo

Merge pull request #8430 from Habbie/builder-no-all

builder: demand user chooses a module except when building docs

builder: demand user chooses a module except when building docs

Merge pull request #8334 from Habbie/dnsscope-nits

dnsscope: two nits

auth: remove opendbx backend

Merge pull request #8486 from pieterlexis/auth-rm-local-ipv6

Auth: Remove local-ipv6 setting

Avoid copying of pthread_rwlock_t

Merge pull request #8591 from rgacogne/ddist-wait-longer-for-tls-tickets

dnsdist: Wait longer for the TLS ticket to arrive in our tests

Merge pull request #8574 from Habbie/auth-4.2.1-docs

Secpoll and changelog for auth-4.2.1.

Update docker repo script for auth-4.2.1.

Merge pull request #8504 from rubenk/docs-fix-redhat

docs: s/RedHat/Red Hat/

secpoll and changelog for auth 4.2.1

expand do-not-credit username list

Merge pull request #8598 from renchap/patch-1

Add increase-serial to pdnsutil doc

Add increase-serial to pdnsutil doc

Description is from `pdnsutil --help`

Correct when to replace a throttle entry

Merge pull request #8583 from omoerbeek/format-code-silent-if-not-tty

format-code script: Do not show verbose messages if !tty or CIRCLECI

Merge branch 'rec-better-time-based' of github.com:omoerbeek/pdns into rec-better-time-based

NULL -> nullptr

Co-Authored-By: Remi Gacogne <rgacogne@users.noreply.github.com>

Use [ -t 1 ] as suggested

dnsdist: Wait longer for the TLS ticket to arrive in our tests

In TLS 1.3 the server sends the TLS ticket after the handshake has
been completed, but not necessarily right after that.
Ideally we would like to wait for up to several seconds, but stop
waiting as soon as we receive a ticket. Unfortunately we can't ask
that from `openssl s_client`, and we are currently not always waiting
long enough to get a ticket, leading to spurious failures.
Let's try waiting a bit longer to see if that helps.

Merge pull request #8575 from Habbie/lua-view-empty

LUA view: do not crash on empty IP list, fixes #8572

Merge pull request #8441 from cmouse/ipv6

global: Normalize isIpv4/6 to isIPv4/6

API: Accept null as empty list in createZone

Some clients refuse to send empty arrays.

Merge pull request #8580 from omoerbeek/thread_rwlock_destroy

pthread_rwlock_init() should be matched by pthread_rwlock_destroy()

Merge pull request #8579 from omoerbeek/cachecleaner-no-nth

Avoid having to use numbered indexes by given all indexes a tag

Avoid having to use numbered indexes by given all indexes a tag
and specifying them in the call if needed.

Do not show verbose messages if !tty

Merge pull request #8582 from omoerbeek/rec-reformat

rec: Reformat code

Merge pull request #8581 from phonedph1/patch-19

dnsdist: Fix typos in dnsnameset.rst' example code

docs: backends/pipe - Fix performance page location

docs: Update upgrading info

docs: Update documentation about isIPv4 and isIPv6

global: Normalize isIpv4 to isIPv4

global: Normalize isIpv6 to isIPv6

Check if formatting is ok

Reformat

Update dnsnameset.rst

Fix example

Merge pull request #8345 from omoerbeek/format-code-script

Add script to format code,

quotes