dnsdist: Apply suggestions from code review

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Document the new behaviour for custom LB policies written in Lua

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Fix CodeQL warnings

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Simplify the least outstanding policy

The servers can no longer be updated under our feet, so we only need
one pass.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Refactor load-balancing policies

Since we no longer need to increase the reference counter of the
returned backend (the runtime configuration cannot be updated be
updated under our feet anymore), we can return the position of the
selected backend in the initial array instead, significantly
reducing the performance cost of the load-balancing policies.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Only set the DNSQuestion packet cache after a cache miss

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Server pools are no longer ref counted

Since the refactoring of the runtime configuration, the content of
a Server Pool is now in effect immutable, we have to create a new
copy and update it, so we no longer have to lock and reference count
Server Pools and their content.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16191 from rgacogne/ddist-test-meson-dist

build-and-test-all: Build dnsdist and recursor via a release tarball

Merge pull request #16205 from rgacogne/ddist-fix-settag-dynblock-doc

dnsdist: The dynamic block tag name is set via `tagName`, not `tagKey`

Merge pull request #15929 from miodvallat/cement

grow auth source tarballs fatter

Merge pull request #16204 from omoerbeek/rec-fix-dist

rec: we need a usable cargo for sdist so do not download (and install) rust but use package

rec: Fix "Avoid generating doc files in a sdist based build" check in out-of-tree builds

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

build-and-test-all: Test rec release tarball via meson instead of autotools

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16200 from omoerbeek/rec-coverity-20251002

rec: fix a few issues reported by coverity, all minor

rec: Fix out-of-tree builds with autotools

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Actually do not install rust at all

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

dnsdist: The dynamic block tag name is set via `tagName`, not `tagKey`

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16192 from rgacogne/ddist-health-check-latency-metric

dnsdist: Add a metric for the latency of the latest health-check

rec: we need a usable cargo for sdist so download (and install) rust later

The install is needed for meson setup, which does not run the found executable,
so the error of a missing lib was never spotted.

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #16195 from miodvallat/cnt_spl_a_dmn

auth: missing space in error message

rec: fix a few issues reported by coverity, all minor

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #16198 from Smoothengineer/master

Fixed grammar in README.md

Fixed grammar in README.md

Signed-off-by: Smoothengineer <160827599+Smoothengineer@users.noreply.github.com>

Be sure to add the meson build files to the contents of `make dist'.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

sort EXTRA_DIST

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

This does not try to generate the man pages if they are found in the tarball

Same method as dnsdist and rec

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #15267 from romeroalx/store-packages-additional-repo

gh actions: upload packages to an additional repository

Merge pull request #16194 from omoerbeek/rec-commands

rec: rework command dispatcher to be table based

build-and-test-all: Fix code coverage generation from meson dist tarball

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

build-and-test-all: Fix bugs

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Fix out-of-tree autotools builds

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

build-and-test-all: No longer run dnsdist autotools build from distdir

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

build-and-test-all: We always need Rust to build the dnsdist release tarball

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

build-and-test-all: Build dnsdist via a release tarball

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16153 from rgacogne/ddist-docs-eol

dnsdist: Simplify EOL page

Merge pull request #16193 from miodvallat/air_freshness

lmdb: optional memory-only freshness check timestamp

Add missing space in error message.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

dnsdist: Update the EOL text as suggested by Otto

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>
Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16132 from omoerbeek/rec-docs-eol

rec doc: simplify EOL page

Merge pull request #16151 from miodvallat/overscore

auth: knob to allow underscores in hostnames

Be more careful when initializing TransientDomainInfo.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Handle freshness check timestamps similarly to notified serials.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

dnsdist: Make the health-check latency atomic

Otherwise TSAN rightfully complain that it is written from one thread
and read from several others.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Fix typo spotted by Miod

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Invoke Namingway and change a bunch of data structure names. NFCI

This will make them look less awkward in the near future.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Underscores may appear in hostnames if RFC112-CONFORMANCE metadata is set to 0.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16185 from miodvallat/endless_negative_trouble

auth: handle ENT in pipe backend

Merge pull request #16179 from miodvallat/pdnsutil_shaken_not_stirred

pdnsutil: add zone copy command

Merge pull request #16113 from miodvallat/cmos

auth: remove confusing message

rec: rework command dispatcher to be table based

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

s/following/subsequent/

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

dnsdist: Add a metric for the latency of the latest health-check

This is useful for monitoring purposes, since this latency is usually
not impacted by the content of the backend cache.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16189 from miodvallat/nothing_to_see_here_move_along

boring PR to move a single file

Merge pull request #16188 from omoerbeek/rec-lib-version-followup

rec: Fix release builds by updating the locked Rust lib version

Merge pull request #16175 from omoerbeek/rec-help

rec: move help text to a table so it's easier to maintain

Merge pull request #16178 from rgacogne/ddist-pass-meta-keys-to-response

dnsdist: Fix setting meta keys on response, pass them from question to response

Move logr.hh to common directory. NFC

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16070 from rgacogne/ddist-skip-padding-by-default

dnsdist: Skip EDNS padding content by default in the packet cache

rec: Fix release builds by updating the locked Rust lib version

Compare #16180 for dnsdist

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #16142 from pieterlexis/add-HHIT-BRID

feat: Add DRIP records (HHIT and BRID)

gh actions: upload packages to an additional repository

Handle ENT records (with no content) in pipe backend protocol.

Fixes: #15027
Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16184 from miodvallat/flush_your_domain's_teeth

auth: backend flush command

Add a pdns_control flush command, to instruct backends to write in-flight data.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16183 from miodvallat/error_message_obscurity

auth: Provide more context in GeoIPbackend YAML parsing failures

Sort registration of pdns_control actions. NFC

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Provide more context in GeoIP YAML parsing failures.

Fixes: #13064
Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16180 from rgacogne/ddist-fix-release-rust-update

dnsdist: Fix release builds by updating the locked Rust lib version

dnsdist: Fix release builds by updating the locked Rust lib version

Since we are now dynamically setting the version of our internal Rust library
when generating the release tarball, `cargo` needs to update the `Cargo.lock`
file to reflect the new version, which is not possible if we are passing `--locked`:
```
error: the lock file /pdns/dnsdist-2.1.0-alpha0.870.master.gc64b979bc/dnsdist-rust-lib/rust/Cargo.lock needs to be updated but --locked was passed to prevent this
If you want to try to generate the lock file without accessing the network, remove the --locked flag and use --offline instead.
```
This commit fixes that also updating the `Cargo.lock` file when generating the
release tarball so that `cargo` no longer needs to update the `Cargo.lock`.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Add a pdnsutil "zone copy" command, and suggest its use for views.

Fixes: #5798
Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

dnsdist: Fix setting meta keys on response, pass them from question to response

This commit fixes setting Protocol Buffer meta keys on DNS response via Lua FFI:
the existing code was assuming it was possible to use the question methods on a
response object which is not true and would likely have ended in a crash at some
point.
It also propates meta keys set on a DNS question to the corresponding DNS response.
Before this commit the values were not passed along to the response which was quite
unexpected, especially for self-answered responses.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16176 from miodvallat/lmdb_polish

auth: some more lmdb polish

Use two-line format as pdnsutil does

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Sort includes.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Address some coverity-reported warnings.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Merge pull request #16140 from rgacogne/fix-tcp-downstream-freebsd

dnsdist: Fix reentry issue in TCP downstream I/O on macOS/BSD

Merge pull request #16162 from rgacogne/documentation-remove-netlify

documentation: Stop pushing to netlify

Remove .netlify from .gitignore

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Remove redundant assignment noticed by Otto

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Free more resources at the end of a lookup operation.

In particular, lookupEnd() could keep unprocessed record data in memory
until the next lookup operation, for no good reason.

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

Group the various fields used for lookups in their own struct. NFC

Signed-off-by: Miod Vallat <miod.vallat@powerdns.com>

rec: move help text to a table so it's easier to maintain

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

chore: clang-tidy does not like boilerplate_conv

feat: Add DRIP records (HHIT and BRID)

These are defined in draft-ietf-drip-registries-33.

Merge pull request #16168 from Habbie/auth-4.9.9-docs

auth-4.9.9: docs & secpoll

Merge pull request #16167 from omoerbeek/rec-update-rust-version-on-dist

rec: Update the Rust library version when generating a tarball

Merge pull request #16169 from rgacogne/build-docker-images-tags-perms

build-docker-images-tags: Grant enough permissions to sign images

docker: Grant enough permissions to sign images

build-docker-images-tags: Grant enough permissions to sign images

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

auth-4.9.9: docs & secpoll

Signed-off-by: Peter van Dijk <peter.van.dijk@powerdns.com>

rec: Update the Rust library version when generating a tarball

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #16155 from rgacogne/ddist-update-rust-version-on-dist

dnsdist: Update the Rust library version when generating a tarball

Merge pull request #16166 from rgacogne/ddist-fix-unused-rust-struct

dnsdist: Fix 'warning: struct `UnusedStruct` is never constructed' warning

dnsdist: Fix 'warning: struct `UnusedStruct` is never constructed' warning

```
warning: struct `UnusedStruct` is never constructed
  --> src/lib.rs:10:8
   |
10 | struct UnusedStruct {}
   |        ^^^^^^^^^^^^
   |
   = note: `#[warn(dead_code)]` on by default
```

The warning that this struct was introduced to fix seems to be gone
anyway.

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

Merge pull request #16163 from omoerbeek/link-ai-policy

PR template: Make the mention of the AI policy a link

dnsdist: Error out in meson-sdist-script.sh when BUILDER_VERSION is unset

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Add comment about the Rust library version being automatically updated

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

update-rust-library-version: Not DNSdist-specific

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

dnsdist: Update the Rust library version when generating a tarball

Signed-off-by: Remi Gacogne <remi.gacogne@powerdns.com>

PR template: Make the mention of the AI policy a link

Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>