Merge pull request #15609 from miodvallat/shard_removal

Packet Cache fixes for views

Merge pull request #15641 from Habbie/auth-2136-test-script-failure

dyndns test: report script failure

Merge pull request #15667 from miodvallat/my_counter_is_rich

minor tsan fixes

Use atomic types for the latency metrics.

Use atomic types for "time for next cache cleanup" variable.

Fixes: #11816

Merge pull request #15665 from jpmens/patch-18

pdnsutil: clarify provenance of default TTL

pdnsutil: clarify provenance of default TTL

Signed-off-by: Jan-Piet Mens <jp@mens.de>

Merge pull request #15663 from miodvallat/blind_as_a_mole

End sentences with dots, not commas.

End sentences with dots, not commas.

Merge pull request #15656 from miodvallat/deadbool

Remove never used argument

Merge pull request #15605 from miodvallat/random_random

Boring coverity-induced changes

Merge pull request #15532 from nokia/master

dnsdist: add option to support cache sharing between different payload sizes

Remove never used third argument of xfrName().

Purge views more thoroughly.

Add a few more std::move() when possible (coverity)

When emptying a view, purge all its packetcache data.

Let AuthZoneCache::removeFromView return whether the view is now empty.

Add the ability to purge an entire view from the packetcache.

Lock the topmost map (view->cache) in the packet cache.

Address PR comments of reword and remove rsp payload size check

dnsdist: Gracefully handle list of integers in YAML

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

add missing empty condition check on payload rank option

Use payload size ranking for cache sharing

dnsdist: support skip hashing AR section for caching

Use case arise that two clients with different udp payload size,
a customized client uses 4096 while glibc resolver uses 1232.
User would like to share cache result for the same query name,
type and class in this case. The downstream servers does not use
ECS and would not return different answers upon other EDNS
options so this is to add an option to support such use case.

Merge pull request #15650 from edmonds/fix-yaml-bind-enable-proxy-protocol

dnsdist: YAML: Correct `enable_proxy_protocol` value for TCP/UDP binds

Merge pull request #15634 from rgacogne/rec-fix-asan-stack-use-after-scope

rec: Fix stack-use-after-scope reported by ASAN

dnsdist: Update documentation for `binds.enable_proxy_protocol`

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

dnsdist: YAML: Correct `enable_proxy_protocol` value for TCP/UDP binds

The YAML `enable_proxy_protocol` property of `BindConfiguration` is
documented as defaulting to "true", but is actually defined to default
to false. This commit redefines this property to default to "true",
which matches the documentation. It also matches the documentation of
the `enableProxyProtocol` option to the Lua `addLocal()` function.

However, when `loadBinds()` constructs the `ClientState` for a TCP
or UDP bind, it hardcodes the last parameter (`enableProxyProtocol`)
to false, i.e. it ignores the configured value. In this commit, the
`enableProxyProtocol` parameter to the `ClientState` constructor is
taken from the corresponding value loaded by the YAML parser.

dyndns test: report script failure

Merge pull request #15630 from rgacogne/ddist-generated

dnsdist: Better handling of generated files

Merge pull request #15633 from Habbie/dnsdist-pp-doc-nit

dnsdist settings docs: fix derailed sentence

Merge pull request #15342 from romeroalx/regression-tests-meson

GH Actions - build-and-test-all: use meson builds for regression tests

Merge pull request #15637 from omoerbeek/dnsdist-docs-rules-toc

dnsdist docs: Place "Rule Selectors" in more appropriate place in TOC

dnsdist docs: Place "Rule Selectors" in more appropriate place in TOC

Merge pull request #15636 from omoerbeek/rec-meson-packages-followup3

rec meson followup: extend unittest max time once more and add trixie daily builds

rec meson followup: extend unittest max time once more, ubicloud is slow in some cases

And add trixie to dnsdist and rec daily package build targets

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

Merge pull request #14473 from Habbie/centos-10-stream

builder: add centos-10-stream target

rec: Fix a leak reported by LSAN in the mtasker unit tests

dnsdist: Ignore generated dnsdist-rust-lib/rust/src/lib.rs

Merge pull request #15632 from omoerbeek/rec-meson-packages-followup2

rec: Disable fortify on package builds, it is supplied db package build infra

dnsdist: Add a missing change in the YAML response actions docs

dnsdist: Add missing boost dependencies spotted by Otto (thanks!)

dnsdist: Hopefully fixes consistency issues in our build

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

dnsdist: Also ignore dnsdist-rust-lib/dnsdist-configuration-yaml-items-generated.cc

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

dnsdist: Ignore generated "body" header files

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

dnsdist: Remove generated files from git

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

dnsdist: Rename dnsdist-configuration-yaml-items-generated-pre-in.cc

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

dnsdist: Always process `dnsdist-rust-lib/meson.build`

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

dnsdist: Update stale YAML selectors documentation

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

dnsdist: Automatically generate YAML settings documentation

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

dnsdist: Fix paths in the YAML settings documentation generator

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

dnsdist: Move the YAML settings documentation generator

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

Merge pull request #15631 from rgacogne/ddist-fix-spurious-test-failure

dnsdist: Fix a flaky Proxy Protocol regression test

Merge pull request #15610 from ns-sjorgedeaguiar/ddist-add-set-edns-response-action

dnsdist: add SetEDNSOptionResponseAction

dnsdist settings docs: fix derailed sentence

rec: Fix stack-use-after-scope reported by ASAN

Technically we are indeed using a stack that is no longer the active
one, because we just switched to a different context, but since the
previous stack still exists that I do not believe it is a real issue.
After this commit we access the previous stack before notifying ASAN
that the stack switch is finished, so we are still allowed to do that.
Note that clang 19 ASAN is fine with both behaviour, but GCC 14 and 15
ASAN aren't.

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

Merge pull request #15389 from miodvallat/wittlching_hour

[tools] Honour default-ttl in pdnsutil load-zone

build-and-test-all: Enable coverage back on our repository

build-and-test-all: Fix working directory when generating coverage for auth's unit tests

build-and-test-all: We no longer build from a distdir w/ meson

meson: Add support for generating code coverage data in clang format

tasks.py: meson install requires elevation

gh actions: temporarily disable coverage

build-and-test-all: run coverage on meson builds

gh actions: fix condition to check if coverage is enabled as suggested by @rgacogne

build-and-test-all: use meson binaries in regression tests

Merge pull request #15607 from miodvallat/too_much_sugar

Try harder matching command names in pdnsutil

dnsdist: use "replace" instead of "overwrite"

regression-tests.nobackend - soa-edit: use datefudge instead of faketime

dnsdist-meson: enable yaml configuration

Merge pull request #15581 from miodvallat/editors'cut

Do not use system(3) in pdnsutil

Disable fortify on package builds, it is supplied db package build infra

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

Merge pull request #15604 from miodvallat/a_zone_with_a_view

Suggest create-zone after view-add-zone of a nonexisting zone

Merge pull request #15628 from omoerbeek/rec-meson-packages-followup

rec: meson packages followup

dnsdist: Fix a flaky Proxy Protocol regression test

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

Merge pull request #15626 from omoerbeek/rec-el9-luajit

rec: build el-9 package with luajit on arm64

Update builder-support/specs/pdns-recursor.spec

Co-authored-by: Remi Gacogne <github@coredump.fr>
Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #15629 from rgacogne/fix-documentation-workflow

documentation.yml: Fix a typo

documentation.yml: Fix missing `$`s spotted by Alexis (thanks!)

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

documentation.yml: Fix a typo

Merge pull request #15600 from rgacogne/zizmor

ci: Fix various warnings reported by zizmor

enable xsk for every EL except arm64/8

builder: add centos-10-stream target

Set fortify to auto, avoids warning on ubuntu-noble

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

Disable some deprecation warnings on rhel, boost generates a tonne of them

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

Build daily packages for trixie

Longer timeout for unit test run

rec: build el-9 package with luajit on arm64

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

Merge pull request #15592 from omoerbeek/rec-meson-packages

rec: build packages using meson

Merge pull request #17 from romeroalx/zizmor

CI: added digest (index when possible) for external images used in gh actions

Also generate Dockerfile with meson

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

ci: added digest (index when possible) for external images used in gh actions

Merge pull request #15615 from rgacogne/ddist-update-cargo-lock

dnsdist: Update our Rust dependencies

Merge pull request #15616 from zeha/dnsdist-debian-lua

dnsdist: debian: use luajit when possible

Merge pull request #15614 from rgacogne/ddist-rust-lib-locked

dnsdist: Explicitly use the versions present in `Cargo.lock` when building

dnsdist: debian: use luajit when possible

Use luajit on arm64 amd64, otherwise keep lua5.3. luajit in Ubuntu jammy and
Debian bookworm should be new enough to not need version constraints anymore.

Signed-off-by: Chris Hofstaedtler <chris.hofstaedtler@deduktiva.com>

Better logic for luajit use on debian

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

dnsdist: Update our Rust dependencies

dnsdist: Explicitly use the versions present in `Cargo.lock` when building

The documentation states that only `cargo update` and `cargo install`
should update the dependencies present in the `Cargo.lock` file, but
it still seems safer to explictly ask `cargo build` to not update them.

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

Fix systemd user for debian packages