Merge pull request #13387 from omoerbeek/rec-b-root-servers

rec: Update new b-root-server.net addresses in built-in hints.

Merge pull request #13493 from romeroalx/cleanup-actions-setup-script

Remove unused pip packages from gh-actions-setup

removed unused pip pkgs from gh-actions-setup. removed azure apt reference

Merge pull request #13488 from rgacogne/ddist-fix-last-rule-removal-by-uuid-or-name

dnsdist: Fix the removal of the last rule by name or UUID

Merge pull request #13486 from romeroalx/rec-regression-tests-auth-48

auth-4.8 for regression tests in recursor. updated sdig output format

dnsdist: Add a regression test for rules removal

dnsdist: Prevent a copy of the UUID when scanning rules for removal

Merge pull request #13489 from rgacogne/ddist-rings-api

dnsdist: Add a 'rings' endpoint to the REST API

dnsdist: Add a 'rings' endpoint to the REST API

Merge pull request #13459 from ukleinek/manpage-link-fix

docs/migration: Fix reference to zone2sql(1) manpage

Merge pull request #13023 from mind04/pdns-legacy

Remove legacy terms from the codebase

dnsdist: Fix the removal of the last rule by name or UUID

auth: remove legacy doc updates

auth: cleanup deprecated arguments

auth: rename files

auth: remove old aliasses

replace slave by secondary in as many places as possible

auth: test nits

Revert "auth: remove deprecated master and allow-unsigned-supermaster options"

This reverts commit 2d93a81211979f09e1810ffc8ff5569adeb99ddf.

replace master by primary in as many places as possible

auth-4.8 for regression tests in recursor. updated sdig output format

Merge pull request #13191 from rgacogne/use-unique-pointer-for-dir

Wrap `DIR*` objects in unique pointers to prevent memory leaks

dnsdist: Hopefully make clang-tidy happy

Apply suggestions from Fred's review (thanks!)

Merge pull request #13397 from Habbie/auth-version-exit-0

auth: exit(0) on --version, like dnsdist and recursor do

Delint opendir/readdir related functions

ixfrutils: Set a strong umask before writing a zone file

Fix warnings from clang-tidy

Wrap DIR* objects in unique pointers to prevent memory leaks

Merge pull request #13473 from rgacogne/ddist-ede

dnsdist: Add support for setting Extended DNS Error statuses

Merge pull request #13480 from omoerbeek/rec-outgoing.tcp_max_queries

rec: correctly apply outgoing.tcp_max_queries bound

Merge pull request #13478 from omoerbeek/rec-nsec3-iter-50

rec: change default of nsec3-max-iterations to 50

Merge pull request #13477 from omoerbeek/rec-trace-nb

rec: warn if truncation occurred dumping the trace

Fix regression test that used iter 100

Merge pull request #13475 from omoerbeek/rec-prep-5.0.0-beta1

Prep for rec-5.0.0-beta1

rec: correctly apply outgoing.tcp_max_queries bound

Fixes #13467

rec: change default of nsec3-max-iterations to 50

rec: warning if truncation occurred dumping the trace.

This can happen when writing to e.g. a terminal. While the detection is reliable
the message to stderr is not (it also might return EAGAIN). But often enough it works.
The existing log entry *is* reliable.

Prep for rec-5.0.0-beta1

dnsdist: More clang-tidy fixes

Merge pull request #13468 from omoerbeek/rec-ubsan-settings

rec: fix ubsan error: using a value of 80 for bool

dnsdist: Test existing EDE in backend response

dnsdist: More clang-tidy fixes

rec: fix settings/cxxsupport.cc:525:40: runtime error: load of value 80, which is not a valid value for type 'bool'

Mostly harmless

Merge pull request #13462 from omoerbeek/rec-rpz-mem2

rec: be more memory efficient handling RPZ updates

Merge pull request #13436 from rgacogne/ddist-doc-getStatisticsCounters

dnsdist: Document getStatisticsCounters()

dnsdist: Fix some lints in dnsdist-lua-bindings-dnsquestion.cc

Merge pull request #13472 from rgacogne/ddist-doq-latencies

dnsdist: Add missing DoQ latency metrics

Merge pull request #13437 from rgacogne/ddist-require-quiche-0.15.0

dnsdist: Require Quiche >= 0.15.0

dnsdist: Add support for setting Extended DNS Error statuses

This PR adds support for adding EDNS Extended DNS Error statuses
from DNSDist, via the following mechanisms:
- `SetExtendedDNSErrorAction`
- `SetExtendedDNSErrorResponseAction`
- `DNSQuestion:setExtendedDNSError(infoCode [, extraText])`
- `DNSResponse:setExtendedDNSError(infoCode [, extraText])`
- `dnsdist_ffi_dnsquestion_set_extended_dns_error(...)`

dnsdist: Add missing DoQ latency metrics

Merge pull request #13464 from omoerbeek/rec-edns-errors-default

rec: change default of extended-resolution-errors setting to true

auth: disable clang-format for statements in gsqlbackend

auth: remove deprecated master and allow-unsigned-supermaster options

rec: change default of extended-resolution-errors setting to true

Merge pull request #13455 from omoerbeek/rec-settings-cats

rec: move a few settings from recursor to outgoing section

rec: be more memory efficient handling RPZ reloads

There were two things non-optimal:

- The old zone contents is being kept in memory during a sleep for the
next iteration and then thrown away. Fix: throw away before the sleep.

- The Lua config is kept alive. This causes refs to old config
data to stay around until all RPZ threads have cleared that config.
Fix that by making the config loop-local and load it after the sleep
call.

Above claims confirmed by adding some debug code showing the ref counts
for a config with many RPZs.

Merge pull request #13457 from omoerbeek/rec-prep-4.9.2

rec: Prep for rec-4.9.2

docs/migration: Fix reference to zone2sql(1) manpage

Without :doc: the sentence is rendered as

See its manpage <manpages/zone2sql.1> for more information.

in the html output as can currently be seen on
https://doc.powerdns.com/authoritative/migration.html#migration-zone2sql
. Adding :doc: makes this a proper link.

Merge pull request #13458 from omoerbeek/rec-sdist-install-rust

rec: Install cargo for the sdist step as well now that configure expects it

Install cargo for the sdist step as well now that configure expects it

Merge pull request #13452 from omoerbeek/rec-docs-dup-in-vector

rec: duplicates in YAML vectors have setting specific effects

Merge pull request #13454 from omoerbeek/warnings-followup

Followup to 13442: avoid some warnings

rec: move a few settings from recursor to outgoing section

Avoid using a clang specific -Wxxx, go for a more general one that exists
both in gcc and clang

Use #pragma option GCC to avoid warnings, it covers both clang and gcc

rec: duplicates in YAML vectors have setting specific effects

Fixes #12235

Prep for rec-4.9.2

Merge pull request #13446 from omoerbeek/rec-slogging-port

rec: for structured logging always log addresses including port

Merge pull request #13445 from rgacogne/ddist-dolog-forward

dnsdist: Fix Coverity CID 1523748: Performance inefficiencies in dolog.hh

Merge pull request #13438 from omoerbeek/rec-conf-cargo

rec: Teach configure to check for cargo version and require >= 1.64

Typo in comment

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

dnsdist: Better fix for CID 1523748, args does not have to be mutable

Merge pull request #13342 from rgacogne/ddist-dbrg-remove-range

dnsdist: Add a DynBlockRulesGroup:removeRange() binding

Merge pull request #13344 from rgacogne/ddist-doc-stek-rotation-frontends

dnsdist: More words on TLS session encryption keys setup and rotation

Merge pull request #13226 from fredmorcos/ueberbackend-cleanup

Cleanup `ueberbackend.cc` a bit

rec: for structured logging always log addresses including port

As documented. The way this is done is by defined a specific
log function if SL is used. Could potentially be used for other
cases.

Merge pull request #13443 from omoerbeek/rec-coverity-20231101

rec: fix coverity 1523746 and 1523745

Merge pull request #13442 from omoerbeek/warning-cleanup

Kill a bunch of warnings emitted by clang

Merge pull request #13405 from rgacogne/update-slsa-generator

build-packages: Update the version of the generic SLSA generator

dnsdist: Fix Coverity CID 1523748: Performance inefficiencies in dolog.hh

Coverity reports:
```
Performance inefficiencies (COPY_INSTEAD_OF_MOVE): "args" is
passed-by-value as parameter to "dolog" when it could be moved instead.
```
This PR implements perfect forwarding for the variadic template parameters
of DNSdist's logging methods to address it, even though the performance of
these functions should not matter much.

rec: fix coverity 1523746 and 1523745

Fixing a use_after_move and a copy_instead_of_move

Tidy

Kill a bunch of warnings emitted by clang

A few are coming from boost, I disabled those by adding #pragma's

Merge pull request #13439 from Habbie/auth-py-pytest

auth-py tests: switch to pytest

Merge pull request #13385 from omoerbeek/assorted-delint20231017

Assorted delinting, mark do while loops as ok

auth-py tests: switch to pytest

rec: Teach configure to check for cargo version and require >= 1.64

Merge pull request #13435 from rgacogne/coverity-fixes

Fix a few coverity warnings

Merge pull request #13414 from rgacogne/clang-tidy-with-codeql

CI: Move clang-tidy checks to the CodeQL workflow

Mark function argument using in DLOG() call as maybe_unused

dnsdist: Require Quiche >= 0.15.0

Based on the release notes the API we use did not change after that.

dnsdist: Document getStatisticsCounters()

rec: Fix a warning from Coverity (CID 1522877)

dnsdist: Silence a false positive reported by Coverity as CID 404742

ixfrdist: Default initialize the 'not implemented' metrics

Reported by Coverity as CID 1504405.

Merge pull request #13392 from omoerbeek/rec-sl-dict

rec: Describe the keys and values produced by structured logging in more detail

Merge pull request #13410 from omoerbeek/rec-cache-tidy

rec: tidy cache and only copy values if non-expired entry was found

Fix a few whitespace consistency issues

Merge pull request #13409 from omoerbeek/rec-rootnxtrust-serve-stale

rec: handle serve stale logic in getRootNXTrust()