Always check HAVE_XYZ flags with #ifdef

Meson uses #define HAVE_FEATURE (without 1), so it matters there

Merge pull request #13966 from dwfreed/patch-1

rec-main: let NetmaskGroup parse dont-throttle-netmasks

Merge pull request #13840 from Habbie/daily-auth-4.9

add auth-4.9.x and dnsdist-1.9.x to daily build test

Merge pull request #13842 from omoerbeek/rec-json-log

rec: add structured logging backend that uses JSON representation

add auth-4.9.x and dnsdist-1.9.x to daily build test

Typo

Merge pull request #13816 from jsoref/lua-records-rst

Improve rst for lua-records

Merge pull request #13951 from Habbie/auth-api-ds-sha1

auth: deprecate output of SHA1, GOST hashes

longer variable name for clang-tidy

Merge pull request #13955 from jsoref/rewrite-ssqlite3-destructor

Clean up `SSQLite3::~SSQLite3`

Merge pull request #13954 from Habbie/smysql-unused-var

auth smysql: remove unused var

auth smysql: remove unused var

Merge pull request #13949 from Habbie/ixfrdist-v6only

ixfrdist: use IPV6_V6ONLY on listening sockets, closes #13878

Merge pull request #13965 from rgacogne/auth-ssql-unique-ptr

auth: Wrap SSql pointers in a unique pointer earlier

Merge pull request #13947 from omoerbeek/rec-yaml-type-fixes

rec: yaml type fixes for incoming.proxy_protocol_from and incoming.edns_padding_from

rec-main: let NetmaskGroup parse dont-throttle-netmasks

This allows dont-throttle-netmasks to have negations.

Merge pull request #13904 from jsoref/issue-menu

Improve issue template options

add NOLINT because showZone is big

auth: Wrap SSql pointers in a unique pointer earlier

Using a unique pointer from the beginning makes the ownership of the
pointer more clear, and prevents leaking the database connection and
associated memory if an exception is raised right away.

pdnsutil: move GOST DSes behind verbose flag, like SHA1

auth API cryptokeys: skip SHA1 and GOST DSes, closes #13950

Merge pull request #13957 from romeroalx/fix-misc-dailies-rec-features

gh actions - misc-dailies workflow: add feature option to ci-rec-configure task

Apply suggestions from code review

Co-authored-by: Fred Morcos <fm@fredmorcos.com>

fix misc-dailies: add feature for configuring rec build

Clean up SSQLite3::~SSQLite3

* The loop will run once or twice
* There used to be some cleanup that was done in the failure case but it has been gone for a while
* Clarify logic
* Report error message from sqlite3

Improve rst for lua-records

ixfrdist: use IPV6_V6ONLY on listening sockets, closes #13878

Merge pull request #13060 from Habbie/mariadb-ssl

auth gmysql: modernise implementation of gmysql-ssl flag

Upgrade guide entry

auth gmysql: deprecate gmysql-ssl flag, enable connection timeout

closes #12528
closes #12978

rec: fix types of two YAML settings that should be sequences of subnets

In particular, incoming.proxy_protocol_from and incomig.edns_padding_from

Merge pull request #13946 from omoerbeek/dnsdist-thost-unreach

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

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

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

Improve issue template options

* Link to GitHub Discussions Q&A category
* Treat descriptions as sentences

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