Merge pull request #12421 from rgacogne/ddist-skip-invalid-ocsp

dnsdist: Skip invalid OCSP files after issuing a warning

Merge pull request #12431 from fredmorcos/gh-actions-docs

Migration of CircleCI documentation generation and upload to Github Actions

Merge pull request #12443 from omoerbeek/rec-prep-4.8.1

rec: Prep for rec-4.8.1

Prep for rec-4.8.1

Merge pull request #12439 from rgacogne/fix-dnsreplay-man

dnsreplay: Fix formatting in the man page

libssl: Use decltype(&SSL_CTX_free) as suggested by Fred

dnsdist: Skip invalid OCSP files after issuing a warning

Contrary to certificates and keys, OCSP files are never required to
provide a working DoT or DoH service, so it's better to start even
if would not load all, or any, OCSP files.

Merge pull request #12435 from rgacogne/ddist-failure-server-ctx

dnsdist: Gracefully handle a failure to create a TLS server context

dnsreplay: Fix formatting in the man page

Remove unused CircleCI documentation generator and upload workflows

Github Actions workflow to generate and upload documentation

Fix heading of Documentation section under Compiling the Recursor

docs/Makefiles: Whitespace cleanup

tasks.py: Missing commas

Gitignore: Add .venv

dnsdist: Use pdns::OpenSSL::error and decltype()

As suggested by Fred (thanks!).

dnsdist: Gracefully handle a failure to create a TLS server context

Merge pull request #12425 from omoerbeek/rec-forward-nord

rec: Change the way RD=0 forwarded queries are handled.

Merge pull request #12428 from rgacogne/moved-credentials

credentials: Explicitly clear credentials once they have been moved

credentials: Explicitly clear credentials once they have been moved

Merge pull request #12415 from rgacogne/ddist-improv

dnsdist: Small improvements

Change the way RD=0 forwarded queries are handled.

Since forever, there has been special case code for forwarded queries
in the RD=0 case.  This special case code does a hardcoded RD=0
query to the specified forwarder.  This code has two consequences:

1. Even if the forwarder is marked recursive it gets a RD=0 query
2. The cache is not consulted at all

The corresponding unit tests actually test this behaviour, but after
historic digging with help from @rgacogne it turns out the the unit
test do not reflect the desired functionality, but the current state
of affairs to help with a refactoring PR.  That is good, since
refactoring should not change functionality.

But now the time has come to change the code to do the desired thing:

1. If an RD=0 query is received, do a cache only-lookup in all cases.
2. Never send a RD=0 query to a recursive forwarder

I already did a similar thing when I wrote the QName Minimization
code, introducing a conditional that only gets set for that case,
to avoid changing unrelated (to QM) functionality.

Merge pull request #12381 from rgacogne/fortify-3

Enable FORTIFY_SOURCE=3 when supported by the compiler

Merge pull request #12420 from omoerbeek/rec-README

rec: README tweaks, .md does not know :program:`...`

Merge pull request #12407 from omoerbeek/rec-ecs-chaining

rec: do not chain ecs enabled queries

Merge pull request #12419 from omoerbeek/rec-close-fd-on-fdopen-fail

rec: negcache dump code: close fd on fdopen fail

rec: README tweaks, .md does not know :program:`...`

Merge pull request #12364 from rgacogne/rec-doc-compiling

rec: In compiling docs, warn about the GH tarballs, add deps from README

Merge pull request #12396 from omoerbeek/stringerror-thread-safe

Introduce a thread-safe version of stringerror()

Close fd on fdopen fail.

Originally suggested by @rgacogne, but not included in the merge of #12374
somehow.

Merge pull request #12412 from fredmorcos/openssl3-tls-mac

OpenSSL 3.0: TLS MAC handling

Merge pull request #12416 from Habbie/variable-mirror

build-and-test-all: add functionality to quickly switch ubuntu mirrors

build-and-test-all: add functionality to quickly switch ubuntu mirrors

Merge pull request #12405 from rgacogne/ddist-frontend-responses

dnsdist: Proper accounting of response and cache hits

Merge pull request #12386 from rgacogne/single-doh-acceptor

dnsdist: Merge the 'main' and 'client' DoH threads in single acceptor mode

Merge pull request #12384 from rgacogne/ddist-cap-ttl

dnsdist: Add the ability to cap the TTL of records after insertion into the cache

dnsdist: Add more unit tests for the packet overlay

m4: Fix interleaved output during `FORTIFY_SOURCE` detection

Merge pull request #12411 from fredmorcos/openssl3-ipcipher-ca6-crypto

Support OpenSSL 3.0 for ipcipher CA6 encryption/decryption

OpenSSL 3.0: TLS MAC handling

libssl: Cleanup TLS-related code

libssl: Formatting TLS-related code

dnsdist: Appease the formatting gods

Merge pull request #12403 from rgacogne/spell-allow-eaca

spell-checker: Allow EACA (from the PGP public keys in the docs)

dnsdist: Use smaller event buffers in our socket multiplexers

dnsdist: Silence a warning about an unused variable in discovery.cc

dnsdist: Remove a duplicated ';' in dnsdist-lua-actions.cc

credentials: Add more unit tests

OpenSSL 3.0: decryptCA6

OpenSSL 3.0: encryptCA6

spell-checker: Allow prometheus values

dnsdist: Fix a typo in the prometheus type

dnsdist: Update the prometheus sample in the documentation

It was very old and not up-to-date.

dnsdist: Only record one hit or miss per query in the cache metrics

The scope-zero feature and the DoH paths can actually do more than
one lookup per query, and until now this led to an increase of the
per-cache metric for every lookup, while the global `cache-hits`
and `cache-misses` metrics were only updated once per query.
This has led to several questions and misunderstandings, so we now
only update the per-cache metrics once per query as well.

dnsdist: Properly record cache-hits as responses

For a very long time we have not been adding cache-hits to the
responses counter, which is wrong. Let's fix it now.

Cleanup ipcipher.hh

dnsdist: Merge the 'main' and 'client' DoH threads

When we are in "single acceptor thread" mode, merge the 'main' and
'client' DoH threads into a single one. We use separate threads to
reduce the separate the handling of the HTTP/2 traffic from the DNS
handling, to reduce latency, but that does not really make sense on
small devices with a single, limited CPU core. On these we prefer
using as few threads as possible to reduce the context switches and
the memory usage.

dnsdist: Remove ttlCapTypes from the internal state

dnsdist: Use unordered sets to speed things up a bit

dnsdist: Implement the ability to cap TTLs for some record types only

dnsdist: Add MaxReturnedTTLResponseAction to cap the TTL after packet cache

Merge pull request #12383 from rgacogne/ddist-stronger-udp-path

dnsdist: Stronger guarantees against data race in the UDP path

Merge pull request #12401 from fredmorcos/openssl-3-prep

Openssl 3 prep work

dnsdist: Really fix the formatting this time

dnsdist: Fix formatting

dnsdist: Apply suggestions from Otto's code review (thanks!)

dnsdist: Set a proper HTTP error code on Proxy Protocol failure

dnsdist: Stronger guarantees against data race in the UDP path

Merge pull request #12410 from rgacogne/unbreak-ci

CI: Work-around the "503  Service Unavailable" erros on azure.archive.ubuntu.com

Merge pull request #12402 from rgacogne/ddist-dq-time

dnsdist: Add bindings for the current and query times in DQ/DR

CI: Work-around the "503  Service Unavailable" erros on azure.archive.ubuntu.com

Merge pull request #12400 from rgacogne/ddist-reduce-ttl

dnsdist: Add SetReducedTTLResponseAction

dnsdist: Add Lua bindings for the current time and query time

Print compiler version at the end of configure

dnsdist: Better description of SetReducedTTLResponseAction

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>

Include failure message when creating DSRCs

Lint cleanups of test-signers.cc

OpenSSL signers test for getPubKeyHash()

Cleanup DNSCryptoKeyEngine

Print checkKey() error messages in signer tests

Cleanup test_generic_signers in test-signers.cc

Cleanup of ipcipher.cc

Cleanup includes in misc.cc

Add OpenSSL error handler

Merge pull request #12385 from rgacogne/ddist-ffi-metrics

dnsdist: Add a Lua FFI interface for metrics

Reformat

Better wording of reason to not chain ECS enabled queries

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

Typo in comment

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

rec: do not chain ecs enabled queries.

asyncresolve() assumes the recieved ecs info corresponds to the one sent out.

Merge pull request #12387 from rgacogne/ddist-oom-udp

dnsdist: Handle out-of-memory exceptions in the UDP receiver thread

Merge pull request #12399 from omoerbeek/rec-threadnames

rec: Name recursor threads consistently with a "rec/" prefix.

Merge pull request #12392 from omoerbeek/bind-errno-cleanup

Be more careful saving errno in makeClientSocket() and closesocket()

Merge pull request #12367 from jsoref/pdns_control-ccounts-comma

Add missing comma to pdns_control ccounts

spell-checker: Allow EACA (from the PGP public keys in the docs)

Formatting of various files like ipcipher and opensslsigners

Relax clang-tidy regarding magic numbers

Add _build dir to docs/.gitignore

Name recursor threads consistently with a "rec/" prefix.

Use thread names without capitals, as they look a bit ugly otherwise.

Threads started by libfstrm are not named, as they are created internally by the lib.

Partly supsersedes #11138

Merge pull request #12397 from rgacogne/fix-cachecleaner

cachecleaner: Add cmath, needed for std::ceil and std::round

Introduce a thread-safe version of stringerror()

It turns out we already have the code, so call it.