build(deps): bump actions/cache from 3 to 4

Bumps [actions/cache](https://github.com/actions/cache) from 3 to 4.
- [Release notes](https://github.com/actions/cache/releases)
- [Changelog](https://github.com/actions/cache/blob/main/RELEASES.md)
- [Commits](https://github.com/actions/cache/compare/v3...v4)

---
updated-dependencies:
- dependency-name: actions/cache
  dependency-type: direct:production
  update-type: version-update:semver-major
...

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

Merge pull request #13716 from chbruyand/dnsdist-protocol-logging

dnsdist: properly set protocol when logging

Merge pull request #13481 from berthubert/lua-localwho

Add and document a `localwho()` function for LUA records

dnsdist: revert some clang-tidy related changes

Merge pull request #13719 from omoerbeek/rec-coverity-20240116

rec: coverity 20240116

Merge pull request #13555 from neheb/clann

clang-tidy cleanups

dnsdist: test protobuf protocols for DoHTTP/3 and DoQ

dnsmessage.proto: fix naming consistency

dnsdist: add a test for new protobuf field httpVersion

dnsdist: clang-tidy fixes

Merge pull request #13720 from Habbie/builder-update-fakeroot

pdns-builder: update to newest, removes fakeroot usage

Revert "Debian Trixie builder: install fakeroot pkg"

This reverts commit a902e392a38905bec709468a6ebd26902a6c255a.

fix false check positive on modified submodule

pdns-builder: update to newest, removes fakeroot usage

1524944 Use of 32-bit time_t

1524911 Use of 32-bit time_t

1524889 COPY_INSTEAD_OF_MOVE
1524930 COPY_INSTEAD_OF_MOVE

format pdns/dnsdist-lua-actions.cc pdns/dnsdist-protobuf.cc pdns/dnsdist-protobuf.hh pdns/dnstap.cc pdns/dnstap.hh

dnsdist: properly set protocol for dnstap and protobuf logging when using DoQ or DoH3

Merge pull request #13706 from omoerbeek/rec-coverity-20240112

Fix a few coverity reports

Merge pull request #13711 from omoerbeek/snmp-cleanup

rec: Cleanup of code doing SNMP OID handling

Fix typos in annotations: 1524912 COPY_INSTEAD_OF_MOVE and 1524842 COPY_INSTEAD_OF_MOVE

Merge pull request #13654 from zeha/rec-make-outoftree

rec: allow out-of-tree builds

Merge pull request #13713 from chbruyand/dnsdist-issue-13690

dnsdist: DoH3 add content-type header information in responses

Merge pull request #13714 from spheron1/patch-1

Fix country()/countryCode() mixup in example Lua Record documentation

Fix country()/countryCode() mixup in example Lua Record

The country() function returns true if the country code for the query source IP address matches the country code passed to the function.
The countryCode() function doesn't take any arguments and just returns the country code of the query source IP address.

For this example, we're checking if the country for the query source IP address is "US" and expecting a true/false response, so country() is the correct function to use.
Alternatively, this would be something like "if countryCode() == 'US' then return...".

Update pdns/dnsdistdist/doh3.cc

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

dnsdist: add content-type header information in http/3 responses, add relayed response size in http/3 vinfologs

Merge pull request #13580 from Habbie/ubuntu-noble

add ubuntu-noble build target and test it daily

Cleanup of code doing SNMP OID handling

Merge pull request #13707 from Habbie/auth-4.9.0-alpha1-docs

docs&secpoll for auth-4.9.0-alpha1

docs&secpoll for auth-4.9.0-alpha1

1524908 Use of auto that causes a copy

1524920 COPY_INSTEAD_OF_MOVE

1524930 COPY_INSTEAD_OF_MOVE

1531280 COPY_INSTEAD_OF_MOVE

1528464 COPY_INSTEAD_OF_MOVE

Merge pull request #13441 from Habbie/auth-alias-ecs

auth: forward EDNS Client Subnet option during ALIAS processing

cleanups from clang-tidy

Merge pull request #13683 from fredmorcos/cleanup-clang-tidy

Small cleanups of the Github clang-tidy helper scripts

format dnsproxy.{cc,hh}, ednssubnet.{cc,hh}, stubresolver.cc

remove isValid

add testECSNone

only use the scope from the answer

auth-py tests: test ECS in ALIAS forwarding

regression-tests.auth-py/clientsubnetoption.py: fix equality operator

ixfrdist tests: refactor one function a bit

forward EDNS Client Subnet option during ALIAS processing

typo

Merge pull request #13692 from romeroalx/fix-builder-trixie

GH Acttions: fix builder workflow for Debian Trixie

Merge pull request #13693 from omoerbeek/iputils-unused

iputils: avoid unused warnings on !linux

iputils: avoid unused warnings on !linux

Debian Trixie builder: install fakeroot pkg

Merge pull request #13684 from omoerbeek/rec-prep-5.0.0

rec: Prepare for rec-5.0.1 final release

Merge pull request #13689 from chbruyand/dnsdist-issue-13687

dnsdist: buffer HTTP/3 headers until the query has been dispatched

dnsdist: buffer h3 headers until query has been dispatched

Merge pull request #13688 from rgacogne/fix-provenance-artifacts

build-packages: Fix the handling of provenance artifacts

build-packages: Fix the handling of provenance artifacts

The current version of the SLSA framework uses upload-artifact v3
which is not compatible with download-artifact v4 (don't ask me).

Prepare for rec-5.0.1 final release

Merge pull request #13680 from rgacogne/rec-mtasker-1533199

rec: Fix a potential null deref in `MTasker::schedule()`

Merge pull request #13662 from PowerDNS/dependabot/cargo/pdns/recursordist/settings/rust/unsafe-libyaml-0.2.10

build(deps): bump unsafe-libyaml from 0.2.9 to 0.2.10 in /pdns/recursordist/settings/rust

Merge pull request #13679 from rgacogne/update-upload-download-artifact

ci: Update upload-artifact and download-artifact to version 4

rec: Fix a clang-tidy warning in test-mtasker.cc

Fixup Github clang-tidy helper scripts

rec: Fix a potential null deref in MTasker::schedule()

The bug is located in a part of the code that we never actually
use since we always pass the current time to the function, so
I decided to reduce the complexity by making this parameter mandatory.

Reported by Coverity as CID 1533199.

std::get removals

Can be replaced by structured bindings.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

fix use after move

instead of calling append(), just construct a new string.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: pass function by ref

Found with performance-unnecessary-value-param
Found with performance-unnecessary-copy-initialization

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: avoid recursive include

Found with misc-header-include-cycle

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: add noexcept for move stuff

Found with performance-noexcept-move-constructor

fix error type

runtime_error does not take rvalue parameters. It's a copy anyway.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use override

Found with modernize-use-override

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: remove void

Found with modernize-redundant-void-arg

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use delete

Found with modernize-use-equals-delete

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use make functions

Found with modernize-make-*

Signed-off-by: Rosen Penev <rosenp@gmail.com>

clang-tidy: use equals default

Found with modernize-use-equals-default

Signed-off-by: Rosen Penev <rosenp@gmail.com>

Merge pull request #13667 from rgacogne/ddist-coverage-cleanup-lua-on-exception

dnsdist: Clean up the Lua objects before exiting

Merge pull request #13676 from rgacogne/ddist-quic-pmtu

dnsdist: Enable PMTU discovery and disable fragmentation on QUIC binds

Merge pull request #13678 from rgacogne/ddist-doh3-uni-streams

dnsdist: Grant unidirectional HTTP/3 streams for DoH3

ci: Update upload-artifact and download-artifact to version 4

dnsdist: Grant unidirectional HTTP/3 streams for DoH3

While unidirectional streams are not needed for DNS over QUIC, they
are required by the HTTP/3 RFC and thus needed for DNS over HTTP/3.
This change makes curl and Firefix happy with dnsdist's DoH3
implementation.

dnsdist: Enable PMTU discovery and disable fragmentation on QUIC binds

Merge pull request #13674 from dmachard/dockerfile-dnsdist-add-doqdoh3

Enable doq and doh3 in dockerfile-dnsdist

Merge pull request #13652 from omoerbeek/mtasker-cleanup

rec: MTasker cleanup and move to recursordist

Merge pull request #13675 from rgacogne/rec-fix-alabaster-doc-error

Fix documentation building error for dnsdist and recursor

dnsdist: Fix the version of alabaster when building the doc

Fixes
```
The alabaster extension used by this project needs at least Sphinx v3.4; it therefore cannot be built with this version.
```

rec: Fix the version of alabaster when building the doc

Fixes
```
The alabaster extension used by this project needs at least Sphinx v3.4; it therefore cannot be built with this version.
```

Merge pull request #13670 from chbruyand/dnsdist-doq-acl

dnsdist: doq,doh3 make sure we enforce any ACL

enable doq and doh3 in dockerfile-dnsdist

Merge pull request #13664 from chbruyand/dnsdist-udp-buffers

dnsdist: increase receive and send buffers to max

dnsdist: doq,doh3 make sure we enforce any ACL

dnsdist: tidy variable name

dnsdist: Clean up the Lua objects before exiting

When code coverage is enabled, we try to call `exit()` instead of our
usual use of `_exit()` to get more accurate coverage. It does however
trigger a race condition between the destruction of Lua objects and
the Lua context(s) they belong to, so we try to explicitly clean up
the objects before calling `exit()`.

Merge pull request #13666 from rgacogne/ddist-congestion-2

dnsdist: Optimize the DoQ packet handling path

Merge pull request #13653 from rgacogne/openssf-compiler-options-hardening-guide

CI: Enable more compiler hardening options during our CI run

dnsdist: try to increase receive and send buffers to max

dnsdist: Avoid a few more allocations in the DoQ code

dnsdist: Re-format doh3.cc and doq.cc

dnsdist: Delint DoQ and DoH3

dnsdist: Try flushing egress data after processing readable streams

dnsdist: Handle early data in DoQ/DoH3