rec: Correctly count NSEC3s considered when chasing the closest encloser

We need to count the number of NSEC3s that are present in the response,
not the number of times we have to consider possible NSEC3s when
looking for the NSEC3 closest encloser, label by label.

Merge pull request #13921 from omoerbeek/rec-system-resolve

Rec: add feature to allow names (resolved by system resolver) in forwarding config

Merge pull request #12849 from PowerDNS/dependabot/pip/regression-tests.api/requests-2.31.0

build(deps): bump requests from 2.20.0 to 2.31.0 in /regression-tests.api

Merge pull request #13981 from rgacogne/fix-rec-builds

Fix recursor packages builds

Fix recursor builds: fix invalid path for builder-support/helpers

Fix recursor builds: we now require `jq` during the `dist` phase

build(deps): bump requests in /regression-tests.api

Bumps [requests](https://github.com/psf/requests) from 2.20.0 to 2.31.0.
- [Release notes](https://github.com/psf/requests/releases)
- [Changelog](https://github.com/psf/requests/blob/main/HISTORY.md)
- [Commits](https://github.com/psf/requests/compare/v2.20.0...v2.31.0)

---
updated-dependencies:
- dependency-name: requests
  dependency-type: direct:production
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #10933 from swegener/64-bit-time-support

m4: Enable 64-bit time_t on 32-bit systems with glibc-2.34

Merge pull request #13972 from rgacogne/external-libs-to-json

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

Update pdns/recursordist/settings/docs-new-preamble-in.rst

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

Merge pull request #13844 from omoerbeek/rec-sl-only

rec: remove the possiblility to disable structured logging

Make resolve check interval and self-resolve check settable

Merge pull request #13770 from Assumeru/require-tsig

auth: Add setting to make TSIG required for DNS updates

Remove changes var and releted method, they are unused

Apply suggestions from code review

Co-authored-by: Fred Morcos <fm@fredmorcos.com>
Co-authored-by: Remi Gacogne <github@coredump.fr>

more clang-tidy

Merge pull request #13979 from omoerbeek/rec-netmask-docs

rec: fix netmask docs wrt deprecatd isIpX() functions

Add license, publisher information to the external deps data

rec: fix netmask docs wrt deprecatd isIpX() functions

Fixes #13978

clang-tidy: more descriptive variable name

process review comments; move toTimestampStringMilli() to Logging namespace

Merge pull request #12769 from neilcook/patch-1

Update Dynamic DNS Update Docs with GSS-TSIG

remove unused branch in SLOG macro (ifdef RECURSOR)

rec: remove the possiblility to disable structured logging

Sort result vector of getaddrinfo

Apply suggestions from code review: typos in comments

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

Add test to also test changing forwarder. Needs ENABLE_SUDO_TESTS

Todo: modify tasks.py to add ENABLE_SUDO_TESTS to test run, like dnsdist does

Simple test of named forward, a bit ugly as it depends on external servers

Comments & docs

Avoid race setting serverID

Enable explicitly only

Search -lresolv (if applicable) the proper auto* way

selfresolve check wip

rec: facility to resolve names via system resolver

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

Merge pull request #13975 from omoerbeek/more-if-vs-ifdef-have

Fix the remaining cases of #if vs #ifdef HAVE_XYZ

Merge pull request #12989 from Habbie/stderr

auth, rec: update option text and docs to clarify logging is to stderr, not stdout

Merge pull request #13914 from rgacogne/ddist-fix-xsk-error

dnsdist: Fix XSK-enabled check when reconnecting a backend

Fix the remaining cases of #if vs #ifdef HAVE_XYZ

Merge pull request #13971 from omoerbeek/check-have-with-ifdef

Always check HAVE_XYZ flags with #ifdef

Merge pull request #13973 from jsoref/clarify-job-description

Clarify workflow name

Merge pull request #13548 from PowerDNS/dependabot/pip/docs/urllib3-2.0.7

build(deps): bump urllib3 from 2.0.3 to 2.0.7 in /docs

Clarify workflow name

The workflow performs two very unrelated tasks:
* Verify source code formatting
* Check Makefile.am sort order

auth, rec: update option text and docs to clarify logging is to stderr, not stdout

accept the complexity of this function for now

Merge pull request #13970 from jsoref/build-tags-permissions

Allow build-tags to run on forks

Always check HAVE_XYZ flags with #ifdef

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

Allow build-tags to run on forks

build-tags uses: PowerDNS/pdns/.github/workflows/build-packages.yml@master
As of f107ec62467b8779db9bbdb175721ef232ed52e5, that workflow requires:

    permissions:
      actions: read   # To read the workflow path.
      id-token: write # To sign the provenance.
      contents: write # To be able to upload assets as release artifacts

Per https://docs.github.com/en/actions/using-workflows/reusing-workflows
in order for this to work, the calling job (in build-tags) needs to
have the maximum required permissions in order for the calling workflow
to be run.

Merge pull request #13968 from PowerDNS/dependabot/github_actions/slsa-framework/slsa-github-generator-1.10.0

build(deps): bump slsa-framework/slsa-github-generator from 1.9.0 to 1.10.0

Merge pull request #13964 from rgacogne/ddist-document-cache-hits-ring-buffer

dnsdist: Document that cache hits go into the ring buffer since 1.8.0

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

build(deps): bump urllib3 from 2.0.3 to 2.0.7 in /docs

Bumps [urllib3](https://github.com/urllib3/urllib3) from 2.0.3 to 2.0.7.
- [Release notes](https://github.com/urllib3/urllib3/releases)
- [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst)
- [Commits](https://github.com/urllib3/urllib3/compare/2.0.3...2.0.7)

---
updated-dependencies:
- dependency-name: urllib3
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

build(deps): bump slsa-framework/slsa-github-generator

Bumps [slsa-framework/slsa-github-generator](https://github.com/slsa-framework/slsa-github-generator) from 1.9.0 to 1.10.0.
- [Release notes](https://github.com/slsa-framework/slsa-github-generator/releases)
- [Changelog](https://github.com/slsa-framework/slsa-github-generator/blob/main/CHANGELOG.md)
- [Commits](https://github.com/slsa-framework/slsa-github-generator/compare/v1.9.0...v1.10.0)

---
updated-dependencies:
- dependency-name: slsa-framework/slsa-github-generator
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <support@github.com>

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.

dnsdist: Document that cache hits go into the ring buffer since 1.8.0

And the effects it has on Dynamic Blocks.

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