Use get_view() instead of get_data()

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

Use std::copy instead of memcpy

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

Process review comments by rgacogne and Habbie

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

Distinguish argument value and return value

Having full attribute names is something for later (maybe).

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

Tidy

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

Use std::copy and std::fill, as suggested by rgacogne

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

Encode TraceID and SpanID into a single EDNS option record, as suggested by Habbie

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

Rewrite comment that no longer reflects reality

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

Typos in comments from miod

Co-authored-by: Miod Vallat <miod.vallat@powerdns.com>
Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Fix autotools build and avoid a few g++ warnings by initing all members explicitly

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

Rename sdig argument from otid to traceid

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

docs plus some cleanup

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

Remove helper files used for development

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

Better processing of parent info of event traces (and Spans)

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

Optionally pass OT trace to a collector in ProtobufLogger.py

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

sdig adding OT IDs and rec incoming processing of OT IDs

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

Framework to add opentelemetry trace data in rec

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

Teach contrib/ProtobufLogger.py code to decode opentelemetry protobuf data

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

Added comments on the semantics of the various fields, added forgotten flags field to Span

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

Added more extensive test, fixed a few small things

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

OpenTelemetry trace data protobuf encode/decode using protozero

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

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().

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

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.

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