dnsdist: Fix DNS over plain HTTP broken by `reloadAllCertificates()`

This was introduced in 1.9.0, with the use of the `nghttp2` library
for incoming DNS over HTTP(S).

(cherry picked from commit e07785a477d8a65d4f3eb4d110395519cf95a4db)

Merge pull request #14040 from rgacogne/ddist19-revert-14004

dnsdist-1.9.x: Revert "dnsdist: Release failed TCP backend connections more quickly"

Revert "dnsdist: Release failed TCP backend connections more quickly"

This reverts commit 7b5f590ee72fecf54c0c40b24e98ba03a406af53.

(cherry picked from commit b3d13db2b0ddf5c5bea4c080330ac1e72475c918)

Merge pull request #14017 from rgacogne/ddist19-backport-14004

dnsdist-1.9.x: Backport 14004 - Release failed TCP backend connections more quickly

Merge pull request #14016 from rgacogne/ddist19-backport-14005

dnsdist-1.9.x: Backport 14005 - Release incoming TCP connection right away on backend failure

Merge pull request #14012 from rgacogne/ddist19-backport-14003

dnsdist-1.9.x: Backport 14003 - Fix a null-deref in incoming DoH w/ nghttp2

Merge pull request #14013 from rgacogne/ddist19-backport-14001

dnsdist-1.9.x: Backport 14001 - Support "no server available" result from Lua FFI LB policies

Merge pull request #14014 from rgacogne/ddist19-backport-13999

dnsdist-1.9.x: Backport 13999 - Fix DNS over HTTP connections/queries counters with `nghttp2`

Merge pull request #14015 from rgacogne/ddist19-backport-14006

dnsdist-1.9-x: Backport 14006 - FDWrapper: Do not try to close negative file descriptors

dnsdist: Release failed TCP backend connections more quickly

After a timeout we cannot reuse the TCP connection to the backend
anyway, so let's release it immediately.

(cherry picked from commit 7b5f590ee72fecf54c0c40b24e98ba03a406af53)

dnsdist: Release incoming TCP connection right away on backend failure

We used to keep a shared pointer to the incoming TCP connection around
in `TCPConnectionToBackend::d_currentQuery.d_sender` even after all queries
sent to the backend failed, which prevented the incoming TCP connection
from being closed as soon as it should have.

(cherry picked from commit 5e8f7194934857b1fd579c2ad68c56523d0e5941)

FDWrapper: Always reset the internal descriptor to -1

Even if it was a different negative value, which happens for example
with the bpf helper functions which return `-errno`.

(cherry picked from commit eefbe57feca72af4a088e8e872e864dc36e9284e)

FDWrapper: Do not try to close negative file descriptors

It turns out that some of the BPF helper functions return
a negative `errno` value in case of failure, and since we
wrap the return value into a `FDWrapper` right away this
led to a warning from Valgrind about trying to close an
invalid file descriptor.

(cherry picked from commit ef1e068d55f2247ff416204cebbbb96063a175c2)

dnsdist: Fix formatting in dnsdist-nghttp2-in.cc

(cherry picked from commit 7f4b2ec776c888f61eb9934363fa41edfaa1ba5e)

dnsdist: Add a regression test for DoH connection counters

(cherry picked from commit 82dc167be78195ebf38dd9f1e64b1ba652c2a4c8)

dnsdist: Properly increase the HTTP connections counter w/ nghttp2

(cherry picked from commit 5fea1763bc6e0e19412e40f029b11ae4d3d8a800)

dnsdist: Remove commented out include directive

(cherry picked from commit 79608ac498be158c8f5a94fde85d90a700963152)

dnsdist: Increase the HTTP/1.1 query counter when DoH with 1.1 ALPN

This way we can keep track of how many HTTP/1.1 connections attempt
we see. We will not actually process the DNS over HTTP/1.1 payload
anyway when the `nghttp2` provider is used.

(cherry picked from commit 3bd0eadc3ae285266c781cbc4bfe53c78723587f)

dnsdist: Delint test-dnsdistlbpolicies_cc.cc

(cherry picked from commit 467380e1287097b26cf47193912c050bca5c77fb)

dnsdist: Fix clang-tidy warnings

(cherry picked from commit 6a2478eaf4652e89b3d40d7af182ecb3832b2e24)

dnsdist: Clarify how to return 'no server available' from the FFI policies

(cherry picked from commit faa96abfe2b025d2322bebb60882e6c775855c11)

dnsdist: Support "no server available" result from Lua FFI LB policies

(cherry picked from commit 08fb08ce087af355c37dfa0bad21efca89c9ee55)

dnsdist: Fix a null-deref in incoming DoH w/ nghttp2

When an incoming DoH connection using the `nghttp2` provider is waiting
for a response from a backend that results in a I/O error or timeout,
and the incoming connection also fails due to a I/O error or timeout,
dnsdist could in some cases try to dereference a null pointer, leading
to a crash.

(cherry picked from commit 0e809f7ec9796cae0e3cc0b6e7407083a22cc157)

Merge pull request #13976 from rgacogne/ddist19-13946

dnsdist-1.9.x: Backport #13946 - Also handle EHOSTUNREACH as a case for reconnecting the socket

dnsdist: also handle EHOSTUNREACH as a case for reconnecting the socket

Manual backport of c42df46af74f9af4158eebb51bfba3d1d8fb1a42 to
rel/dnsdist-1.9.x

Merge pull request #13944 from rgacogne/ddist19-13914

dnsdist-1.9.x: Backport of 13914 - Fix XSK-enabled check when reconnecting a backend

Merge pull request #13943 from rgacogne/ddist19-13927

dnsdist-1.9.x: Backport of 13927 - Fix annoying compiler warnings by introducing and switching to `pdns::UniqueFilePtr`

Merge pull request #13940 from rgacogne/ddist19-13851

dnsdist-1.9.x: Backport of #13851 - tcpiohandler: Use server preference algoritm for ALPN selection

Merge pull request #13945 from rgacogne/ddist19-13834

dnsdist-1.9.x: Backport of 13834 - dnsdist: Properly handle a failure of the first lazy health-check

Merge pull request #13942 from rgacogne/ddist19-13920

dnsdist-1.9.x: Backport of #13920 - Shrink InternalQueryState's size by reordering its fields

Merge pull request #13941 from rgacogne/ddist19-13906

dnsdist-1.9.x: Backport of #13906 - Fix first IPv6 console connection being rejected

Merge pull request #13939 from rgacogne/ddist19-13849

dnsdist-1.9.x: Backport of #13849 - Docker: Only print config if debug flag is set

Merge pull request #13938 from rgacogne/ddist19-13805

dnsdist-1.9.x: Backport of #13805 - Fix compilation warnings

dnsdist: Fix clang-tidy warning

dnsdist: Fix clang-tidy warnings

dnsdist: Properly account the failure to forward a query to a backend

Manually backported from b0b3480b98d41db821f681183f45d5d08db02f93

dnsdist: Fix exponential backoff computation in edge cases

(cherry picked from commit a8856ce717f439832515440af5550ca9d51f4b7d)

dnsdist: Properly handle a failure of the first lazy health-check

Manually backported from 2cd1a8d65bd8bc45458115f5827126aa537906b3

dnsdist: Fix XSK-enabled check when reconnecting a backend

The check was reverted, meaning that we did not properly remove an address:port from the XSK map when reconnecting a backend, and actually tried to remove an entry from a non-existing map when XSK is compiled in but not enabled, triggering a:
```
Error getting the XSK destination addresses map path '/sys/fs/bpf/dnsdist/xsk-destinations-v4'
```
message.

(cherry picked from commit bb218803ee8348a4ce9cd11b97681d6eba4082cf)

Fix clang-tidy warnings, again

(cherry picked from commit 6e58535e4cf4b5d139e66cc3c6f18f7b2e62e64a)

Add `pdns::openFileForWriting()` to control permissions when creating a file

(cherry picked from commit b1564d45616abcd65bf606fa5816449bde41af3e)

Fix clang-tidy warnings

(cherry picked from commit 114b879665a529a093e66a44347e924a3e2ab984)

Mark the `pdns::UniqueFilePtr` deleter `const noexcept`

(cherry picked from commit 07d4785d49eb93a51818dae5fdbe04e7ff4e128c)

auth: Switch to `pdns::UniqueFilePtr`

(cherry picked from commit 46c4985c721840c7d34c9199dc72beab89264836)

rec: Switch to `pdns::UniqueFilePtr`

(cherry picked from commit 01807c646de1af252681794c48a71d7b95f58df8)

dnsdist: Switch to `pdns::UniqueFilePtr`

Manually backported from 931536a53367efbf9be3884b8970498aa92be774

misc: Implement pdns::UniqueFilePtr

Using a deleter instead of `decltype(&fclose)` has two big advantages:
- the deleter is included in the type and does not have to be passed
  when creating a new object (easier to use, less memory usage, in theory
  better inlining)
- we avoid the annoying `ignoring attributes on template argument ‘int (*)(FILE*)’`
  warning from the compiler, which is there because `fclose` is tagged as `__nonnull((1))`

(cherry picked from commit 20b2f2041a04a2e16f6bf4781c177a6aa3adec2b)

dnsdist: Shrink InternalQueryState's size by reordering its fields

Manually backported from 1731b908b4e77b5294aad64a2c2aec750e4a03be

dnsdist: Add a regression test for IPv6 console connections

(cherry picked from commit 8be2b867e08b49cdc11a0452476229ceaa1ce6f6)

dnsdist: Fix first IPv6 console connection being rejected

Manually backported from 1a9b7f5a97e97e371bd4b6043c14d9faccb9013c.

dnsname: Fix formatting issue

(cherry picked from commit 981c43ec9b61a942b0f539084fed1a24ed3ad301)

Add missing views.hh reference in the Makefiles

(cherry picked from commit d4cd065a24fea7fd0270a2eb102f40a06309703a)

dnsdist: Use a view for parsing ALPN data, add a regression test

(cherry picked from commit 2a3c2b444812369cae285e6b67e2a72a5b8fed08)

tcpiohandler: Use server preference algoritm for ALPN selection

This complies with RFC 7301 section 3.2

(cherry picked from commit b599f69b2090a1269cdce49a53297201772c454c)

nits

(cherry picked from commit 688d5dbdba626e3a36af37a8629c6fab6e5d5a1a)

Only print config if debug flag is set

Signed-off-by: Carolin Dohmen <carodohmen@gmail.com>
(cherry picked from commit d773b7bb99418026c3907ebd4b4e994a61fccecd)

dnsdist: Move `linux/*.h` headers to xsk.cc to fix warnings

(cherry picked from commit e3044aca1c21335d3d35d8e2d5cfa830efb04541)

dnsdist: Fix outdated definition for `addMaintenanceCallback()`

(cherry picked from commit be05f14384735756b52756c9160a2ce168da1902)

Merge pull request #13912 from omoerbeek/backport-13909-to-dnsdist-1.9.x

dnsdist: Backport 13909 to dnsdist 1.9.x: update to quiche 0.20.1

dnsdist: update to quiche 0.20.1

(cherry picked from commit 7d5a2792c199611ab26e8821a31a2cd0912f34d9)

Merge pull request #13911 from omoerbeek/dnsdist-branch-build

dnsdist 1.9.x: remove irrelevant targets from branch build

dnsdist 1.9.x: remove irrelevant targets from branch build

Merge pull request #13910 from omoerbeek/backport-13907-to-dnsdist-1.9.x

dnsdist: backport 13907 to 1.9.x: gh actions: wo issue 9491 - actions/runner-images

gh actions: wo issue 9491 - actions/runner-images

(cherry picked from commit e0bf314e472d0c1d4bc1ff82d97cabf87be1e929)

Merge pull request #13760 from rgacogne/ddist-doc-rules-mgmt

dnsdist: Refactor the rules (selectors and actions) documentation

Merge pull request #13780 from rgacogne/ddist-better-wording-setSuffixMatchRule

dnsdist: Clarify that setSuffixMatchRule blocks per suffix, not labels

dnsdist: Clarify that setSuffixMatchRule blocks per suffix, not labels

Merge pull request #13779 from rgacogne/ddist-coverity-20240212

dnsdist: Fix performance inefficiencies reported by Coverity

dnsdist: Fix performance inefficiencies reported by Coverity

Reported by Coverity as 414759 and 414760:
> Performance inefficiencies  (COPY_INSTEAD_OF_MOVE)

Merge pull request #13778 from M0NsTeRRR/master

fix(doc): incorrect setting `query_local_address`

fix: replace missing query_local_address

fix(doc): incorrect setting `query_local_address`

Merge pull request #13774 from rgacogne/ddist-quiche-config-atomic-load

dnsdist: Fix a missing explicit atomic load of the Quiche configuration

Merge pull request #13771 from omoerbeek/dnsdist-lua-align

dnsdist: do not allocate 16-byte aligned objects through lua(jit)

dnsdist: Fix a missing explicit atomic load of the Quiche configuration

Reported by TSAN.

Merge pull request #13759 from rgacogne/ddist-doc-tls-certificates

dnsdist: Improve the documentation around TLS certificates and keys

dnsdist: Prevent useless allocation+copy in `setPoolServerPolicy`

dnsdist: Fix newServerPolicy, add regression tests for custom policies

Merge pull request #13391 from devicenull/master

Add Lua function to pick records via name hash

Experiment

dnsdist: Fix an issue spotted by TSAN: we need to use atomic_load_explicit along with atomic_store_explicit

Merge pull request #13756 from rgacogne/ddist-xsk-doc-typos

dnsdist: Fix typos in the AF_XDP documentation

Merge pull request #13768 from rgacogne/ddist-maintenance-hook

dnsdist: Add a Lua maintenance hook

Merge pull request #13757 from rgacogne/ddist-coverty-warning-20240201

dnsdist: Fix a warning reported by Coverity

dnsdist: make sure we do not allocate 16-byte aligned objects through lua(jit)

luajit aligns only to 8 bytes by default, and some objects require
16 byte alignment.

Fixes #13766

Note that the static assert in LuaContext.hpp is commented out in
one case.  This trips on some platforms, but does not seem to be
harmful right now.

The fundamental solution remains the have luajit agree with C++ on
minimal alignment of its allocators.

dnsdist: Delint the 'reloadCertificates' methods

dnsdist: Add a regression test for DoQ certs/keys reloading

dnsdist: Document certs/keys reloading for DoQ and DoH3

dnsdist: Implementation reloading of TLS certs/keys for DoQ and DoH3

dnsdist: Apply suggestions from code review, delint

Merge pull request #13755 from rgacogne/ddist-quic-unsupported-version

dnsdist: Better handling of short, non-initial QUIC headers

dnsdist: Clear the Lua maintenance callbacks before exiting

Merge pull request #13761 from rgacogne/ddist-test-ebpf

dnsdist: Add regression tests for eBPF blocks (static / dynamic)

Merge pull request #12359 from Habbie/auth-lua-dblookup

auth LUA: add dblookup function

dnsdist: Add a Lua maintenance hook

Having a hook that can accept multiple callbacks makes it a lot easier
to manage the execution of several, independant tasks that needs to be
called regularly.

fix formatting

Update minor code/doc nits

Appease lint

Fix typo

Add Lua function to pick records via name hash

This adds a Lua function to return a record based on a weighted hash
of the DNS record name.  One use case here is to consistently return
the same IP address for a particular cache server based on what
subdomain is requesting the data.