Fix typos in comment

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

rec: fix daemonize(), followup to #12836

Originally the code did not distinguish between parent return and error.

Merge pull request #12927 from omoerbeek/rec-zonetocache-local

rec: fix documentation of zoneToCache localAddress attribute

Merge pull request #12913 from omoerbeek/dns_random

dns random: add method to get full 32-bits of randomness

rec: fix documentation of zoneToCache localAddress attribute

Fixes #12919

Merge pull request #12911 from rgacogne/ddist-increment-dyn-blocked-ebpf

dnsdist: Increment the "dyn blocked" counter for eBPF blocks as well

Convert a couple of calls from dns_random() to dns_random_uint32()

Don't close the urandom file descriptor

Merge pull request #12783 from rgacogne/ddist-fix-pool-cache-metrics

dnsdist: Fix cache hit and miss metrics with DoH queries

Merge pull request #12918 from omoerbeek/rec-prep-4.9.0-rc1

rec: Prep for rec-4.9.0-rc1

Merge pull request #12916 from Houtworm/master

edited webserver config template for dnsdist docker container.

Prep for rec-4.9.0-rc1

Update dockerdata/startup.py

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

delint

fixed the webserver config line?

Fix loops

dnsdist: Increment the "dyn blocked" counter for eBPF blocks as well

Regular, userspace blocks increment the "dyn blocked" counter for every
dropped query. The eBPF blocks are executed in kernelspace and thus do
not increment that counter at all, which makes it challenging for
reporting to do its job. On the other hand we want our eBPF code to
be as efficient as possible since it is used when performance really
matters.
This commit updates the counter when a eBPF dynamic block is removed,
which is a compromise between the performance impact and a slight
reporting delay.

Merge pull request #12811 from rgacogne/ddist-health-check-better-messages

dnsdist: Add the query ID to health-check log messages, fix nits

Actually start testing this on !gcc and refactor

Add basic test for dns_random_uint32

Introduce dns_random_uint32()

To be used when a full 32 bits of randomness are needed, also avoiding
the modulo dance

Replace boost lexical casts, including the header produces a lot of warnings on some platforms for some time

We now produce a string runtime, but since the to_string() call should never be executed, it does not matter.

delint

Reformat

Merge pull request #12909 from omoerbeek/fix-speedtest

Fix build of speedtest and make sure it gets built automatically to avoid bitrot

Add GGS_LIB to speedtest link if needed

Fix build of speedtest and make sure it gets built automatically to avoid bitrot

Merge pull request #12804 from rgacogne/ddist-health-check-resize-response

dnsdist: Properly set the size of the UDP health-check response

Merge pull request #12892 from rgacogne/yahttp-fuzz-overflow

YaHTTP: Prevent integer overflow on very large chunks

Merge pull request #12802 from rgacogne/ddist-health-check-wait-time

dnsdist: Account for the health-check run time between two runs

Merge pull request #12808 from omoerbeek/args-delint

Reformat and delint arguments.cc and arguments.hh

dnsdist: Add the query ID to health-check log messages, fix nits

dnsdist: Fix a boolean nit as suggested by Otto

Typo in var name spotted by @Habbie

Merge pull request #12904 from omoerbeek/rec-gid_t-uid_t-can-be-unsigned

rec: uid_t and gid_t can be unsigned, so doing > on an -1 value is tricky

Merge pull request #12906 from omoerbeek/rec-sdjournal-escape

rec: systemd-journal backend: escape keys that are special

Merge pull request #12893 from omoerbeek/rec-dnssec-alg-setting

rec: add feature to switch off unsupported DNSSEC algos

Tidy and process review comments

Merge pull request #12905 from rgacogne/ddist-async-test-racy

dnsdist: Remove a racy test in the AsynchronousHolder unit tests

Merge pull request #12881 from Habbie/dispatch-bookworm

builder-dispatch: add debian-bookworm target to defaults

Merge pull request #12900 from omoerbeek/rec-serve-stale-dup-cname

rec: Prevent duplicate C/DNAMEs being included when doing serve-stale

Don't double print and delint

dnsdist: Remove a racy test in the AsynchronousHolder unit tests

We are adding an expired event so the worker thread of the
AsynchronousHolder can pick it up immediately, even before we come
back from the call to push(), which leads to a racy test.
This was observed on GitHub Actions when running with TSAN:
```
FAIL: testrunner
================

Running 170 test cases...
test-dnsdistasync.cc(156): error: in "test_dnsdistasync/test_AddingExpiredEvent": check !holder->empty() has failed

*** 1 failure is detected in the test module "unit"
FAIL testrunner (exit status: 201)
```

Merge pull request #12896 from omoerbeek/rec-nod-metrics

rec: expose NOD/UDR metrics

rec: uid_t and gid_t can be unsigned, so doing > on and -1 value is tricky

Merge pull request #12883 from omoerbeek/rec-rpz-soa

rec: add SOA to RPZ modified answers if configured to do so

delint

Process review comments

Merge pull request #12840 from phonedph1/patch-39

Update dnsdist-console.cc

Add tests to see if CNAME records are not included multiple times

Merge pull request #12839 from phonedph1/patch-38

Update rules-actions.rst

rec: Escape (by prepending "PDNS") message keys that are special to systemd-journal

Delint arguments.cc and arguments.hh

Reformat

rec: Prevent duplicate C/DNAMEs to be included when doing serve-stale

This can happen if the CNAME record itself was found, but its target not

Merge pull request #12898 from omoerbeek/rec-depth

rec: keep track of max depth reached and report it if !quiet

Prometheus does not like counter names ending in -count

(Partial) Tidy

rec: keep track of max depth reaches and report it if !quiet

This is enough for now, mostly a metric only interesting to devs

Extend SNMP test to include new OIDs

Add Prometheus and SNMP version of NOD/UDR metrics

Keep track of metrics for NOD and UDR events.

While there, change level of ndr logs to Notice, it was Debug before

Add docs

Add rec_control command to list supported algo names

Distinguish auto and manual disabling in logging

Impelement verification of algos 5 and 7

YaHTTP: Prevent integer overflow on very large chunks

If the chunk_size is very close to the maximum value of an integer,
we trigger an integer overflow when checking if we have a trailing
newline after the payload.
Reported by OSS-Fuzz as:
https://oss-fuzz.com/testcase-detail/6439610474692608
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=56804

rec: implement a way to disable specific DNSSEC algorithms

This could be needed when runing RHEL9, to avoid having zones signed
with algo 5 or 7 going Bogus. RHEL9 does not support these algorithms,
unless the globalsecurity policy is modified.

Merge pull request #12889 from Habbie/auth-4.8-eol-update

auth 4.8: EOL update

auth 4.8: EOL update

delint

Merge pull request #12793 from omoerbeek/assorted-delint

Another set of delinting

Merge pull request #12867 from omoerbeek/rec-prep-4.9.0-beta1

rec: Prep for rec-4.9.0-beta1

Prep for rec-4.9.0-beta1

Merge pull request #12880 from Habbie/swagger-no-docker

swagger-syntax-check: run outside of Docker

builder-dispatch: add debian-bookworm target to defaults

swagger-syntax-check: run outside of Docker

Merge pull request #12875 from Habbie/upgrade-notes-46-47-master

auth upgrading: fix 4.6/4.7 header; update LMDB text

Merge pull request #12856 from romeroalx/enable-ipv6-actions-container

Enable ipv6 for actions running on docker containers

auth upgrading: fix 4.6/4.7 header; update LMDB text

Merge pull request #12874 from Habbie/auth-4.8.0-docs-secpoll

auth-4.8.0: docs and secpoll

auth-4.8.0: docs and secpoll

Merge pull request #12873 from aerique/feature/add-bookwork-to-repo-test-script

Add Bookworm to repo test script.

Add Bookworm to repo test script.

Add Debian Bookworm for `auth-master` & `auth-48` to repo test script.

Merge pull request #12866 from Habbie/auth-lmdb-dup-delete

auth lmdb: delete duplicate domain entries in deleteDomain

Add test and fix TC=1 adding of SOA record

rec: add SOA to RPZ result if configured to do so

Fixes #8232

Merge pull request #12860 from Habbie/auth-ent-wildcard

auth: do not answer with broken TYPE0 data when expanding an ENT wildcard

Merge pull request #12859 from Habbie/pdnsutil-edit-zone-wrong-key

pdnsutil: if user pushes unknown key in response to "problem with zone" prompt, do not throw away their changes

auth: do not answer with broken TYPE0 data when expanding an ENT wildcard

Process review comments

Reformat

More delinting

A set of random files made clean. During this process .clang-tidy.full
was also amended a bit.

document shards=1 enforcement

format

simplify

Merge pull request #12862 from omoerbeek/rec-recursion-bound

rec: bound maximum recursion depth to 16.

Merge pull request #12861 from omoerbeek/rec-no-refresh

rec: Introduce a way to completely disable root-refresh

use d_transactiondomain[id]

Update modules/lmdbbackend/lmdbbackend.cc

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>