gh actions - replace yq snap in collect job build-and-test-all

Merge pull request #14087 from rgacogne/ddist19-backport-13983

dnsdist-1.9.x: Backport 13983 - Generate Software Bill Of Materials (SBOMs) for our RPM packages

Merge pull request #14105 from rgacogne/ddist19-backport-14104

dnsdist-1.9.x: Backport 14104 - Fix a crash in incoming DoH with nghttp2

dnsdist: Fix a crash in incoming DoH with nghttp2

This fixes an issue in the code dealing with incoming DNS over HTTPS
queries with the nghttp2 provider. In some rare cases, if the incoming
query is forwarded to the backend over TCP and the response comes back
immediately (the `read()` call done just after the `write()` call sending
the query must succeed and yield a complete response), the processing
of the response might end up calling `IncomingHTTP2Connection::readHTTPData()`
down the line, via the `nghttp2` callbacks, while we were already
inside this function. This does not actually work because
`nghttp2_session_mem_recv` is not reentrant, so the internal state of
the `nghttp2_session` object might become inconsistent and trigger
an assertion, for example:
```
nghttp2_session.c:6854: nghttp2_session_mem_recv2: Assertion `iframe->state == NGHTTP2_IB_IGN_ALL' failed.
```

This results in a call to `abort()` and very unlikely to be exploitable,
because there is no memory corruption occurring. It would also be quite
difficult for an attacker to trigger the conditions leading to this event
remotely.

Reported by Daniel Stirnimann from Switch and Stephane Bortzmeyer, many thanks to them.

(cherry picked from commit a4d82c53f714211462ae4da2e16fb13dec51ba05)

Merge pull request #14082 from rgacogne/ddist19-backport-14037

dnsdist-1.9.x: Backport 14037 - Fix "C++ One Definition Rule" warnings in XSK

Merge pull request #14080 from rgacogne/ddist19-backport-13972

dnsdist-1.9.x: Backport 13972 - Move the version (and hashes) of external dependencies to JSON files

Merge pull request #14081 from rgacogne/ddist19-backport-14070

dnsdist-1.9.x: Backport 14070 - Fix DNS over plain HTTP broken by `reloadAllCertificates()`

dnsdist: Fix "C++ One Definition Rule" warnings in XSK

It turns out we need to include the linux specific headers AFTER the
regular ones, because it then detects that some types have already been
defined (`sockaddr_in6` for example) and does not attempt to re-define
them, which otherwise breaks the C++ One Definition Rule

(cherry picked from commit 679360ad842c60e38f4009cecac6e1422c747889)

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)

SBOM: Dynamically generate the list of auth-related packages

(cherry picked from commit 94b262b9709077f7c3a4e8cd74108e2c6991a63e)

SBOM: Fix the builder target variable

(cherry picked from commit b17412c732f8ec98ae47926254828cb45ff825df)

Fix syntax errors

(cherry picked from commit fef1b1c63e150b34a391bc836596657ec068de81)

Disable SBOM generation on el-7 (almost EOL), fix it on el-8

(cherry picked from commit 4b54c7db6822c31d70a5661258b4ed70c58ee163)

Hopefully fix SBOM generation

(cherry picked from commit 5efa3cdfbcf590068006030da64cd5673e11bf8d)

Attempt to generate SBOMs after building packages

(cherry picked from commit 44503ded5e8e6f088d009177f5f951d408467b58)

Add license, publisher information to the external deps data

(cherry picked from commit cb7e7db7990d4a6ea7a873e3ff4d0c63cd2f1291)

Move the version (and hashes) of external dependencies to JSON files

(cherry picked from commit 4ccb0f786bab6e752ce8f6bc8223d428754c0081)

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