fix: added missing closed bracket

Merge pull request #13365 from rgacogne/ddist-clang-tidy-nolintnextline

dnsdist: Better clang-tidy suppressions

dnsdist: Better clang-tidy suppressions

Merge pull request #13321 from fredmorcos/geoip-lmdb-cleanups

Minor cleanups to geoip and lmdb

Merge pull request #13363 from omoerbeek/rec-coverity-1522436

rec: Fix Coverity 1522436 Dereference null return value

rec: Fix Coverity 1522436 Dereference null return value

Merge pull request #13361 from phonedph1/patch-2

dnsdist: Update dq.rst

Update dq.rst

Small refactoring of GeoIPBackend::loadDomain

Set the domain ID

Minor cleanups to geoip and lmdb

Merge pull request #13355 from Habbie/docker-h2o

dnsdist Docker: enable h2o again, using our fork

dnsdist Docker: enable h2o again, using our fork

Merge pull request #13346 from omoerbeek/rec-unittests-cache-init

rec:  Fix a spuriously failing recursorcache_cc test: reset globals (indirectly) used

Merge pull request #13296 from omoerbeek/rec-qm-iters

rec: Make QName Minimization parameters from RFC 9156 settable

Merge pull request #13350 from rgacogne/ddist-182-secpoll-changelog

dnsdist: Prepare ChangeLog and secpoll for DNSdist 1.8.2 and 1.7.5

Apply suggestion from review by @rgacogne, thanks!

Init default values for new settings

Typo

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

Make QName Minimization parameters from RFC 9156 settable

Also fix a counting ommission

spell-check: Allow 'DDoS'

secpoll: Make it lear that a vulnerability exists for all pre-releases

dnsdist: Add a ChangeLog entry for 1.7.5 as well

secpoll: All DNSdist packages >= 1.4.0-alpha2 were built against a bad h2o version

Merge pull request #13348 from rgacogne/ddist-powerdns-h2o-h2-rapid-reset

dnsdist: Switch to our fork of h2o to mitigate http2 rapid reset

dnsdist: Prepare ChangeLog and secpoll for DNSdist 1.8.2

builder-support: Use curl's "fail fast with no output at all on server errors" option

dnsdist: Switch to our fork of h2o to mitigate http2 rapid reset

Merge pull request #13312 from omoerbeek/rec-rfc2181-10.3

rec: conform to rfc2181 10.3

Fix a spuriously failing recursorcache_cc test: reset globals (indirectly) used

For SyncRes tests we have a general mechanism setting the globals
before the test are run, but the non-syncres tests do not have that, while
they still use some globals.  In this particular case, the test would
fail if the last SyncRes test run before was setting
SyncRes::s_locked_ttlperc.

While there, pass the time to the prune functions. This avoids
potential timing issues for some tests.

Merge pull request #13330 from rgacogne/fix-clang-tidy

build-and-test-all: Fix clang tidy checks

Merge pull request #13260 from rgacogne/pdns-code-coverage-with-coveralls

Add code coverage reports via Coveralls

coverage: Do not call __gcov_dump() when using source-based coverage

It's useless and will do nothing.

Code coverage: Fix two typos

Code coverage: Fix a link

Code coverage: Apply Fred's suggestions (thanks!)

Add Code Coverage documentation

build-and-test-all: Generate code coverage data from CI

rec: Properly record coverage data when exiting

dnsdist: Properly record clang coverage data when exiting

auth: Properly record coverage data when exiting

Update the code coverage m4 to support modern options, hook it up

Merge pull request #13345 from omoerbeek/rec-changelog-sysvinit

rec: Mention removal of sysvinit script in 4.9.x changelog.

Merge pull request #13185 from rgacogne/ddist-parse-ede

dnsdist: Log Extended DNS Errors (EDE) to protobuf

Merge pull request #13311 from omoerbeek/rec-more-log-fixes

rec: more log (level) fixes

Merge pull request #13303 from omoerbeek/rec-log-fixes

rec: fix log messages related to zone to cache function

Mention removal of sysvinit script in 4.9.x changelog.

Fixes #13343

Merge pull request #13339 from rgacogne/building-packages-doc

BUILDING-PACKAGES: Document how to build packages

dnsdist: Apply Otto's suggestions to the EDE code and docs

BUILDING-PACKAGES: Mention that `pdns-builder` requires a working Docker local daemon

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

BUILDING-PACKAGES: Use a supported OS as example

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

Merge pull request #13340 from rgacogne/netmask-fix-invalid-mask

Netmask: Normalize subnet masks coming from a string

Update BUILDING-PACKAGES.md as suggested by Peter (thanks!)

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

Merge pull request #13335 from rgacogne/ddist-web-rule-name

dnsdist: Display the rule name, if any, in the web interface

Netmask: Normalize subnet masks coming from a string

Until now we only normalized too large masks when constructed from a
`ComboAddress` object and a separate mask, but not from a string.

BUILDING-PACKAGES: Document how to build packages

dnsdist: Display the rule name, if any, in the web interface

As suggested by Sébastien Riccio.

Merge pull request #13317 from Habbie/auth-4.8.3-docs-secpoll

auth-4.8.3: changelog and secpoll

clang-tidy.py: We need to output relative paths in annotations

git-filter: We no longer need to check for non pdns/ paths

git-filter.py: Document that we cannot normalize here

build-and-test-all: Fix clang-tidy now that we build in dist dirs

clang-tidy-diff.py: clang-tidy only supports filename in filter-line, not paths

Import clang-tidy-diff.py

Merge pull request #13323 from ekohl/update-lmdb-docs-path

Match the lmdb-filename example with the default

Match the lmdb-filename example with the default

The default config places it in /var/lib, which is a more logical place
for a DB.

Merge pull request #13320 from fredmorcos/cleanup-remote-backend

Some cleanups to the auth remote backend

Merge pull request #13298 from rgacogne/ddist-nghttp2-in-timeout

dnsdist: Fix timeouts on incoming DoH connections with nghttp2

Cleanup remote backend

dnsdist: Apply suggestion from Charles-Henri's code review (thanks!)

dnsdist: Fix timeouts on incoming DoH connections with nghttp2

Merge pull request #13319 from Habbie/sphinxcontrib-update

auth docs: update sphinxcontrib-api hash, add a comment

update sphinxcontrib-api hash, add a comment

Merge pull request #13304 from rgacogne/ddist-async-unit-tests-rae

dnsdist: Fix a race in the Async unit tests

Merge pull request #13314 from rgacogne/ddist-skip-badvers-test-when-needed

dnsdist: Skip our BADVERS regression test on broken Python versions

Merge pull request #13310 from rgacogne/ddist-ebpf-fixes

dnsdist: Fix several cosmetic issues in eBPF dynamic blocks, update documentation

Merge pull request #13287 from rgacogne/dnsscope-custom-port

dnsscope: Add a `--port` option to select a custom port

auth-4.8.3: changelog and secpoll

dnsdist: Skip our BADVERS regression test on broken Python versions

See https://github.com/PowerDNS/pdns/pull/12912 for the longer
explanation.

dnsdist: Fix eBPF metrics in the internal web server

We used to only display eBPF dynamic blocks for the Dynamic BPF
registered via `registerDynBPFFilter()`, which does not play well
with the new DynBlockRulesGroup approach.
This commit fixes it by making exporting / displaying metric for
the default BPF filter (`setDefaultBPFFilter`) as well.

Merge pull request #13080 from jsoref/remove-timestamp

Remove `timestamp` and `test-auth`

Merge pull request #13014 from jsoref/deprecated

Report auth settings deprecated in 4.5

Remove test-auth

Fix typoe and ws

rec: more log (level) fixes

Found while working on a list

Merge pull request #13305 from rgacogne/ddist-proper-log-levels

dnsdist: Set proper levels when logging messages

Merge pull request #13309 from Habbie/refreh

dnsdist kvs docs: fix typo

Merge pull request #13308 from phonedph1/patch-1

dnsdist: Update tuning.rst

dnsdist: Document that `add*DynBlocks()` methods are legacy ones

dnsdist: Log whether newly inserted dynamic blocks are eBPF-based

dnsdist: Display whether blocks are eBPF-based in `showDynBlocks()`

dnsdist kvs docs: fix typo

whitespace error

Update tuning.rst

dnsdist: Set proper levels when logging messages

This PR re-evaluates the log level of all messages in the DNSdist
code base. The rules are simple:
- 'Error' should only be used when a critical service will not function
  properly
- 'Warning' should be used when a non-essential feature may not be
  working as expected
- 'Info' is for everything that should not go unnoticed but also cannot
  be triggered by an outside attacker, otherwise we might be flooding
  our logs
- 'Verbose' can be used for everything else.

There are a few exceptions, something we log at the 'info' level for
events that might be happening a lot, but only when told to do so,
like for example with verbose health-check reporting.

Merge pull request #13302 from phonedph1/imeouts

dnsdist: client timeouts not client imeouts

dnsdist: Fix a race in the Async unit tests

We used to set the `errorRaised` variable from the `AsynchronousHolder`
thread then check its value from the main thread, which is correctly
reported as TSAN as a data race. We do know that we have waited enough,
and that otherwise it's fine to fail, but TSAN cannot know that. Switching
to a `std::atomic<bool>` fixes it.

delint

rec: Fix log messages lacking priority

When we have authoritative NS records in cache that do not lead to
any usable address, wipe the NS records so a query of NS records
to theprant leads to new NS records being inserted into the cache.

Previously the presence of the authoritaivbe records would prevent
that to happen. Queries would *use* the freshly retrieved parent
records, but they would not end up in the cache (the presence of
authoritative NS records prevents that) leading to repeated queries
to the parent.