Merge pull request #13834 from rgacogne/ddist-fix-first-lazy-health-check-failure

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

Merge pull request #13931 from omoerbeek/rec-build-least

rec ci: also build a rec with all optional stuff disabled

Merge pull request #13932 from omoerbeek/update-boost.m4

Add a bunch of new compiler versions to the compiler list used by boost.m4

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

Merge pull request #13927 from rgacogne/fix-fclose-warnings

Fix annoying compiler warnings by introducing and switching to `pdns::UniqueFilePtr`

Merge pull request #13874 from fredmorcos/remotebackend-test-fixes

Some fixes for the remotebackend tests

Fix clang-tidy warnings, again

Pass 'full' to rec configure arg for codeql & clang-tidy run

Add a bunch of new compiler versions to the compiler list used by boost.m4

Sad we have to maintain this.

Merge pull request #13930 from omoerbeek/sdig-no-xpf

sdig: remove xpf handling

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

rec ci: also build a rec with all optional stuff disabled

Merge pull request #13928 from Habbie/build-for-noble

add ubuntu noble to default build targets

Fix clang-tidy warnings

Fix remotebackend testrunner script's new_api mode

Cleanup remotebackend testrunner script

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

Merge pull request #13926 from omoerbeek/rec-trace-fail

rec: fix trace=fail regression

add pkghashes-ubuntu-noble

sdig: remove xpf handling

add ubuntu noble to default build targets

auth: Switch to `pdns::UniqueFilePtr`

rec: Switch to `pdns::UniqueFilePtr`

dnsdist: Switch to `pdns::UniqueFilePtr`

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))`

Merge pull request #13920 from rgacogne/ddist-reorder-internal-query-state

dnsdist: Shrink InternalQueryState's size by reordering its fields

ZTC regression test does not need auths

rec: fix trace=fail regression and add regression test for it

Merge pull request #13917 from Habbie/auth-4.9.0-docs-secpoll

auth-4.9.0: docs&secpoll

EOL auth 4.6

Merge pull request #13906 from rgacogne/ddist-fix-ipv6-console

dnsdist: Fix first IPv6 console connection being rejected

dnsdist: Shrink InternalQueryState's size by reordering its fields

This saves 524 kB per downstream server in the default configuration
(8 bytes times 65536 slots).

Merge pull request #13919 from omoerbeek/rec-not-throttle-lame

rec: don't throttle lame servers if they are marked as dontThrottle

rec: don't throttle lame servers if they are marked as dontThrottle

Merge pull request #13916 from Habbie/dockerignore

.dockerignore: adjust for https://github.com/docker/buildx/issues/850

auth-4.9.0: docs&secpoll

.dockerignore: adjust for https://github.com/docker/buildx/issues/850

Merge pull request #13913 from Habbie/dnsdist-1.9.1-secpoll-docs

secpoll & docs for dnsdist 1.9.1

Merge pull request #13876 from rgacogne/ddist-simplify-response-chains

dnsdist: Simplify the handling of rule chains

dnsdist: Fix exponential backoff computation in edge cases

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

dnsdist: Add a regression test for IPv6 console connections

dnsdist: Fix first IPv6 console connection being rejected

If we don't set the family of the client IP address,
`ComboAddress::getSocklen()` will return the size of an IPv4 struct
and thus the first IPv6 client address will get truncated.
Subsequent connections will be fine because the family will have been
set to IPv6.

Merge pull request #13894 from omoerbeek/rec-coverity-1534473

rec: fix Coverity 1534473 Unintended sign extension

Merge pull request #13889 from omoerbeek/rec-tidy-lua4

rec: tidy recursor-lua4.cc and recursor-lua4.hh

Explicit cast to do multiply in 64-bit

fix formatting

secpoll & docs for dnsdist 1.9.1

Reformat

Avoid label and goto in loop

Make a few fields (d_gettag and friends and handle) private

rec: Tidy lua-recursor4.cc and lua-recursor4.hh

rec: fix Coverity 1534473 Unintended sign extension

Merge pull request #13909 from omoerbeek/dnsdist-quiche-0.20.1

dnsdist: update to quiche 0.20.1

dnsdist: update to quiche 0.20.1

Merge pull request #13907 from romeroalx/wo-runner-issue-9491

gh actions: Fix CI - `issue 9491` on repository `actions/runner-images`

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

Merge pull request #13877 from rgacogne/remove-harvestCSPFromRecs

validate: Remove unused `harvestCSPFromRecs()`

Merge pull request #13879 from Habbie/auth-lua-filterforward-empty

auth LUA: support returning empty set in filterForward

simplify return type usage

Merge pull request #13860 from Habbie/auth-lua-dblookup-qtype

auth LUA dblookup: switch qtype argument to int

Merge pull request #13873 from Habbie/auth-49-debian-postinst

auth debian: adjust option name in shipped postinst

Merge pull request #13205 from mind04/auth-catalog-group

auth: catalog, include groups in hash calculation

auth LUA: support returning empty set in filterForward

fixes #12436

fix NUL string literal

Merge pull request #13841 from mind04/auth-include-ns

auth: do not disable ns records at apex in consumer zones

dnsdist: Fix clang-tidy warnings

validate: Remove unused harvestCSPFromRecs()

dnsdist: Simplify the handling of rule chains

This PR creates a list of response rule chains so we can deal with
them using a loop instead of having to take care of them one by one.
It makes the code easier to read and reduces the likelihood of introducing
a bug when implementing a new chain.

Merge pull request #13871 from rgacogne/ddist-move-files-3

dnsdist: Move dnsdist specific files to `dnsdistdist`, format and delint them

dnsdist: Return the correct TLS provider for DoQ and DoH3 frontends

dnsdist: Remove dangling sodcrypto.cc and sodcrypto.hh symbolic links

dnsdist: Move dnsdistconf.lua to dnsdistdist/

dnsdist: Delint dnsdist-protobuf.cc

dnsdist: Delint dnsdist.cc

Merge pull request #13867 from omoerbeek/auth-nb-send-obsd

auth: on OpenBSD, try harder to send on a non-blocking socket

auth: catalog, include groups in hash calculation

Merge pull request #13814 from wwijkander/wwijkander-patch-remote-doc

Update remote.rst to reflect that getAllDomains might be mandatory

auth debian: adjust option name in shipped postinst

Merge pull request #13866 from omoerbeek/rec-ztc-wildcard

rec: Don't enter wildcard qnames into the cache in the ZoneToCache function

Merge pull request #13864 from omoerbeek/frmstrm-v6

Support v6 in FrameStreamLogger, including tidy

Merge pull request #13870 from omoerbeek/rec-prep-20240307

rec: prep for 5.0.3, 4.9.4 and 4.8.7

dnsdist: Delint dnsdist-xpf.cc

dnsdist: Delint dnsdist-web.cc

dnsdist: Delint dnsdist-ecs.cc

auth: on OpenBSD, try harder to send on a non-blocking socket

Should fix #13857 in many cases, though you can still argue this
is a OpenBSD bug (or at least an undesirable difference between other
systems and OpenBSD).

rec: prep for 5.0.3, 4.9.4 and 4.8.7

Move #define to static const string

Merge pull request #13865 from omoerbeek/tidy-const-or-ref-field-in-class

common: less strict clang-tidy: allow const and/or ref fields in classes

rec: Don't enter wildcard qname's into the cache in the ZoneToCache function

less strict cling-tidy: allow const and/or ref fields in classes

tidy

Merge pull request #13861 from omoerbeek/rec-rpzloader-tidy

rec: tidy rpzloader.cc and .hh

dnsdist: Fix formatting issues

dnsdist: Delint dnsdist-lua-vars.cc

dnsdist: Delint dnsdist-protocols.cc

dnsdist: Delint dnsdist-lua.cc and dnsdist-snmp.cc

dnsdist: Fix formatting in dnsdist-lua-bindings-dnsquestion.cc

dnsdist: Delint dnsdist-rings.cc

dnsdist: Delint dnsdist-lua-inspection.cc