Sort endpoints by name. NFCI

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

Allow zone edit to work in non-interactive mode (!isatty(0)).

In this case, we will trust EDITOR to point to something able to edit
the zone file, and assume "no" to all prompts in case of warnings or errors.

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

auth: set version in meson.build in autoconf make dist path

Stopgap until auth switches to meson dist.

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

Merge pull request #17560 from rgacogne/ddist-action-by-name

dnsdist: Add "ByName" support for YAML actions

Merge pull request #17559 from rgacogne/ddist-fix-named-and-or-not-selectors

dnsdist: Fix naming And, Or and Not selectors, Continue action from YAML

Merge pull request #17387 from esensar/feature/mmdbkvstore

dnsdist: add MMDB KV store

Put LOC default values in member initializers.

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

Rewrite the LOC parser with something more strict and more readable.

The existing code was copied from the LOC RFC.

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

dnsdist: Tell ruff to shup up

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

dnsdist: Add "ByName" support for YAML actions

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

Merge pull request #17549 from karelbilek/kb/doh-fix-dohtobackend

dnsdist: don't close DoH on timeout, do on release

dnsdist: Fix naming Continue action from YAML

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

dnsdist: Fix naming And, Or and Not selectors from YAML

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

Merge pull request #17535 from omoerbeek/rec-priv-ywh-250-ws-rust-origin

rec: only set ACCESS_CONTROL_ALLOW_ORIGIN if auth succeeded (#YWH-PGM-6095-250)

Merge pull request #17554 from omoerbeek/protozero-order

protozero: extend test to determine endianess

dnsdist: remove unused iostatehandler add function

Signed-off-by: Karel Bilek <kb@karelbilek.com>

dnsdist: TCP multiplexer: update should reset ttd

Fixes #17555

Signed-off-by: Karel Bilek <kb@karelbilek.com>

Merge pull request #17547 from arthurscchan/add-new-fuzzer

OSS-Fuzz: Add new fuzzer targets record handling

dnsdist: don't close DoH on timeout, do on release

Currently, dnsdist closes outgoing DoH on receive
timeout, instead of keeping TCP connection alive.

On the other hand, when DoH max idle time is
reached, the connection is not closed properly and
the filedescriptor is not properly closed.

The second issue was hard to reach before, as
cleanup interval would need to be lower than TCP
read timeout.

There is a new testcase for this, which needed
slight refactor of test DoH threads.

Signed-off-by: Karel Bilek <kb@karelbilek.com>

protozero: extend test to determine endianess

By using symbols defined by modern compilers

Merge pull request #17550 from rgacogne/auth-fix-meson-build_root-warning

auth: Fix deprecated "meson.build_root" warning

Merge pull request #17553 from zeha/auth-meson-test

auth: meson: avoid having boost_test_framework in all programs

auth: meson: avoid having boost_test_framework in all programs

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

Merge pull request #17551 from manuelm/master

auth: declare enable-lua-record-updates unconditionally

Update pdns/auth-main.cc

fix comment.

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>
Signed-off-by: Manuel Mausz <manuel@mausz.at>

auth: declare enable-lua-record-updates unconditionally

The Lua record update restrictions added in #17131 rely on the
enable-lua-record-updates setting. When built without Lua record support,
this setting is not defined, causing errors like:

Undefined but needed argument: 'enable-lua-record-updates'

Add the setting unconditionally.

Signed-off-by: Manuel Mausz <manuel@mausz.at>

fuzzing: Silence clang-tidy warning about a global variable we don't control

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

auth: Fix deprecated "meson.build_root" warning

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

fuzzing: Fix formatting of the recordcontent target

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

fuzzing: Fix clang-tidy warnings in the recordcontent target

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

fuzzing: Build the new recordcontent target with meson

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

dnsdist: add a wrapper type for `MMDBQueryParams` to prevent misuse

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: simplify MMDB functions to always accept vector

The variant with just string is really just to make for an easier user
interface, but MMDB is not used directly, only through Lua functions or
KVS, so this keeps the same interface, but less branching in MMDB code.

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add more types in mmdb tests

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add top level access test for MMDB

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDB regression tests with complex object access

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: Fix one more clang-tidy warning

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

dnsdist: Fix clang-tidy warnings for the new MMDB code

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

dnsdist: add basic MMDB guide in docs

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: move MMDB feature flag to `mmdb.hh`

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: feature flag "mmdb.hh" include in YAML conf

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDB Lua and YAML variant tests

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: fix ordering of MMDB related imports

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: expect MMDBKV in spell checker

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: accept strings in `MMDB:query` and `MMDB:exists` for easier use

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDB docs

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add basic MMDB regression tests

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add mmdb to YAML configuration

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add mmdb functions to console completion

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: replace specific MMDB queries with a single generic query

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add lua bindings for MMDB

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDBKVStore lua bindings

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDBKVStore impl

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

dnsdist: add MMDB class impl

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

Merge pull request #17543 from rgacogne/ddist-parse-cname-exception-ywh-258

dnsdist: Catch exceptions when parsing CNAME via the Lua FFI API

Merge pull request #17548 from romeroalx/update-ci-auth-51

gh actions: added rel/auth-5.1.x and rel/dnsdist-2.1.x to scheduled tests

Merge pull request #17546 from miodvallat/computerswereamistake

auth: give even more slack to the IfurlupMinimumFailures test

gh actions: added rel/auth-5.1.x and rel/dnsdist-2.1.x. Removed old ones

dnsdist: add mmdb feature flag

Signed-off-by: Ensar Sarajčić <dev@ensarsarajcic.com>

OSS-Fuzz: Add new fuzzer targets record handling

Signed-off-by: Arthur Chan <arthur.chan@adalogics.com>

auth docs lmdb: footnote about 5.1.x schema usage

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

auth meson: add html-docs target

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

Merge pull request #17544 from rgacogne/ddist-more-ffi-exception-hardening

dnsdist: Do not let exceptions cross the FFI boundary

Give even more slack to the IfurlupMinimumFailures test.

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

dnsdist: Do not let exceptions cross the FFI boundary

As we learned that on at least some ARM platforms "External frame unwinding"
is not available / working well.
None of these changes should be a security issue as they should not be triggerable
via attacker-controlled data, but they will make debugging of flawed Lua code easier.

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

dnsdist: Log packet bytes after failing to parse a CNAME from them

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

Merge pull request #17536 from miodvallat/axfrdam

auth: stricter lmdb AXFR checks

Merge pull request #17538 from karelbilek/kb/fix-udp

fix: dnsdist drops responses without HAVE_DNSCRYPT

Merge pull request #17537 from miodvallat/911

auth: documentation and secpoll bits for 5.1.1

dnsdist: Clear DoH3 buffers when processing a stream triggered an exception

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

dnsdist: Properly handle exceptions when decoding DoH paths

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

dnsdist: Fix handling of trailing whitespaces in base64 content

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

fix: dnsdist drops responses without HAVE_DNSCRYPT

Signed-off-by: Karel Bilek <kb@karelbilek.com>

auth: meson: fix missing symbols for backend dylibs

Auth backends can be built as dynamic libraries. In the autoconf build,
such backends expect pdns_server to provide all symbols they need
(including sqlite3 for gsqlite3, gsqlbackend* for the gsql*backends,
bindlexer for the bindbackend).

Previously gsqlbackend.cc was in common_sources, thus libpdns-common.
While pdns_server and pdnsutil link libpdns-common, they do so using
link_with. This commit moves gsqlbackend.cc into a libpdns-sqlbackend,
which gets linked using link_whole. Thus the symbols appear in
pdns_server and pdnsutil and are not discarded.

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

Documentation and secpoll bits for 5.1.1

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

If EDNS is mandatory, do it always in the first iteration (#YWH-PGM6095-290)

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

Check that all secondary zones have their timestamp updated after AXFR.

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

rec: only set ACCESS_CONTROL_ALLOW_ORIGIN if auth succeeded (#YWH-PGM6095-250)

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

Make some test failures more visible in stdout.

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

Merge pull request #17503 from rgacogne/ddist-tls-ecdhe-curves

dnsdist: Add support for setting TLS ECHDE curves

Add a defense against invalid input, which should not trigger anymore.

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

Annotate cases where we build DNSZoneRecord with domain_id not known.

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

Initialize domain_id fields of synthesized records whenever possible.

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

Factor code responsible for the NSEC{,3} QType bitmap computation.

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

Remove deceased, long dead code which is not alive.

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

Merge pull request #17533 from zeha/auth-meson-doc

auth: meson: install module_resources into /usr/share/doc/pdns

auth: meson: install module_resources into /usr/share/doc/pdns

This mirrors what the autoconf build does.

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

Merge pull request #17523 from miodvallat/morebrainless

auth lmdb: correctly update timestamps in non-split domain table mode

remote logger: Reconnect if writes have been stalled for too long

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

Merge pull request #17521 from rgacogne/ddist-fix-statnode-fullname-ffi

dnsdist: Prevent an exception when accessing an empty StatNode

Merge pull request #17525 from Habbie/ednsoptions-string

ednsoptions.hh: #include <string>

ednsoptions.hh: #include <string>

gcc-17 (or its libstdc++) is stricter about includes and needs this

closes #17524

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

dnsdist: Fix inconsistent quoting in the documentation

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

libssl: Use `SSL_CTX_set1_groups_list` instead of `SSL_CTX_set1_curves_list`

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

Merge pull request #17502 from omoerbeek/rec-zap-autotools

rec: zap autotools as build system

dnsdist: Fix compilation of the FFI unit tests when dynblock support is off

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

dnsdist: Add unit tests for the FFI inspection API

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

dnsdist: Prevent an exception when accessing an empty StatName

We fixed it for the regular interface in bd21d60 but overlooked
the FFI one.

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

Merge pull request #17520 from romeroalx/bump-requests-dependabot

dependabot/195: bump version of python3 requests to >=2.33.0 in `builder-support/requirements.txt`

Crude test to check notification timestamp is updated after initial AXFR

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