Fixup dnsbackend.cc and dnsbackend.hh

Format dnsbackend.hh and dnsbackend.cc

Merge pull request #13702 from omoerbeek/rec-log-ref-wrapper

rec: use ref wrapper instead of raw pointer in variant

Merge pull request #13728 from paddg/patch-1

STDOUT instead of STDERR. Probably a typo.

Merge pull request #13725 from chbruyand/auth-tinydnsbackend-build

auth: fix tinydnsbackend compilation issue

Merge pull request #13729 from Habbie/13633-avoid-copy

getAllDomains catalog: avoid useless copy

Merge pull request #13650 from rgacogne/ddist-sym-libcrypto

dnsdist: Fall back to libcrypto for authenticated encryption

Fix test by updating expected_result

Merge pull request #13727 from rgacogne/coverity-20240119

dnsdist, rec: Fix missed optimizations reported by Coverity

Merge pull request #13722 from Habbie/createforward-word-prefix

auth LUA createForward: allow non-hex word prefix

getAllDomains catalog: avoid useless copy

bit of formatting

reference instead of copy, and move some code for clarity

Merge pull request #12891 from Habbie/upgrading-el9-sha1

auth docs: add note about EL9 and SHA1

Merge pull request #13633 from Habbie/gsql-alldomains-catalog

auth: set catalog in gsql getAllDomains

STDOUT instead of STDERR. Probably a typo.

It's not an error message.

note changed query in updating.rst

dnsdist: Fix a missed optimization reported by Coverity

dnstap: Fix a missed optimization reported by Coverity

Merge pull request #13726 from PowerDNS/dependabot/github_actions/actions/cache-4

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

Merge pull request #13649 from chbruyand/auth-delay-notifications

auth: add a configurable delay for notifications

auth: document new delay-notifications setting

auth: a bit of tidy

auth: reformat communicator.hh and communicator.cc

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>

auth: fix tinydnsbackend compilation issue

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

also recognise uppercase hex

Revert "Debian Trixie builder: install fakeroot pkg"

This reverts commit a902e392a38905bec709468a6ebd26902a6c255a.

fix false check positive on modified submodule

tidy: this function is not really complex, just long

tidy: use .empty()

tidy: fix short variable name

tidy: avoid pointer math

small refactor

auth LUA createForward: allow non-hex word prefix

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

Reformat and tidy annotations

rec: Avoid raw pointers by using a referrence wrapper inside a variant

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>