Merge pull request #15915 from omoerbeek/rust-build-env

rec and dnsdist: properly set up env vars for rust build

dnsdist: Fix clang-tidy warnings

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

ci: Don't install Rust or Quiche when testing DNSdist 'least'

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

dnsdist: Ensure the runtime configuration is updated in the SNMP thread

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

dnsdist: Speed up response content matching

This commit introduces a new method to compare a `DNSName`
against a view of raw, wire-format bytes, skipping the
allocation and copy that is usually required to get a
second `DNSName` object to compare against.
This signifitcantly reduces the amount of time matching
a DNS response received from a backend against the content
we expect to find.

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

Do not use countLabels() in chopOff() loop conditions.

Instead, compute it once outside of the loop in a local variable and
decrement that variable.

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

More use of hasLabels

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

Address clang-tidy complains.

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

Simplify some DNSName label processing.

Instead of checking countLabels() against zero, introduce a faster
hasLabels() which does not need to actually count them.

Also replace getRawLabels()[n] with getRawLabel(n), the only difference
being that getRawLabel() will raise an exception if n is out of bounds,
instead of returning garbage.

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

Use ERCode symbolic names instead of their numerical values.

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

Factor logic checking for a zone metadata being equal to "1".

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

Declare global const DNSName variables for frequently-used names

(such as the TSIG key algorithms)

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

Build the DNSName in order in keyUnconv.

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

dnsdist: Test netmask exclusions via YAML

Follow-up to #15822.

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

Merge pull request #15890 from rgacogne/lua-wrapper-std-optional

LuaWrapper: Add support for `std::optional`

Merge pull request #15916 from rgacogne/ddist-rings-update-atomic-outside-lock

dnsdist: Update rings' atomic counter without holding the lock

Merge pull request #15917 from rgacogne/ddist-early-return-empty-chains

dnsdist: Return early when a rule chain is empty

Merge pull request #15918 from rgacogne/ddist-cache-update-atomic-outside-lock

dnsdist: Update a cache's atomic counter without holding the lock

dnsdist: Upgrade Cloudflare's Quiche to 0.24.4

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

dnsdist: Update a cache's atomic counter without holding the lock

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

dnsdist: Return early when a rule chain is empty

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

dnsdist: Update rings' atomic counter without holding the lock

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

rec and dnsdist: properly set up env vars

env.append() appends to an existing value

Should fix the error in #15896, but the reported location of cargo
still might be wrong.

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

Merge pull request #15902 from rgacogne/ddist-fix-qtype-rate

dnsdist: Fix QType rate dynamic block with YAML

dnsdist: Fix invalid "qps" in Quickstart Guide

As noted by Holger Hoffstätte, the quickstart guide has an outdated
YAML configuration sample which is no longer valid. `qps` has been
renamed to `queries_per_second` well before the release candidates.

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

dnsdist: Explicitly update the configuration thread-local copy

This commits ensures that all DNSdist threads are regularly checking
if there is a new version of the runtime-modifiable configuration and
update their local copy if necessary. Regular accesses to the copy
are now fully read-only, meaning they do not invalidate the current copy.
It prevents the case where a function is invalidating the copy that the
caller is holding, and makes accessing the configuration cheaper.

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

Merge pull request #15899 from miodvallat/betamax

auth 5.0.0ß1 changelog + secpoll

dnsdist: Fix restricted network families when building with meson

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

dnsdist: Fix systemd template unit when building with meson

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

Merge pull request #15893 from omoerbeek/rec-rpz-custom-cname-chain

rec: try harder to follow cname chain on RPZ hit with custom CNAME record

dnsdist: Apply Miod's suggestions (thanks!)

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

Changelog and secpoll update for 5.0.0-beta1.

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

Merge pull request #15900 from omoerbeek/rec-prep-5.3.0-beta1

rec: prep for rec-5.3.0-beta1

dnsdist: Fix QType rate dynamic block with YAML

The YAML configuration for the the "QType rate" dynamic block was
totally broken, trying to configure a rcode rate rule instead of a
qtype rate one.
Thanks to HellSpawn for reporting this the issue!

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

test-luawrapper: Appease the clang-tidy gods

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

Typos in comments

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

rec: prep for rec-5.3.0-beta1

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

Use unmodified flag for post policy

rec: if a RPZ hit has a custom CNAME record, we should try harder to follow it

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

LuaWrapper: Add support for `std::optional`

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

Merge pull request #15888 from omoerbeek/rec-ottrace-align

rec: add a reserved/alignment byte in EDNS record

rec: add a reserved/alignment byte in EDNS record

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

Merge pull request #15887 from omoerbeek/rec-version-edns-traceid

rec: add version to edns record for OpenTelemetry data

Better code for setting and getting traceids and spanids from EDNS

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

Merge pull request #15885 from rgacogne/ddist200-eol

dnsdist: Update EOL statements now that 2.0.0 has been released

secpoll: Mark EOL DNSdist versions as such

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

dnsdist: Update EOL statements now that 2.0.0 has been released

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

Merge pull request #15882 from rgacogne/ddist200-changelog-secpoll

dnsdist: Update ChangeLog and security polling zone for 2.0.0

Add a 1 byte version number to edns option OTTRACEIDS

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

dnsdist: Update ChangeLog and security polling zone for 2.0.0

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

Merge pull request #15883 from omoerbeek/recp-prep-20260722

rec: Prep for rec 20250721 releases

Fix spell checking

Fix spell check issues

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

Merge pull request #15859 from aerique/feature/update-repo-test-script

Update repo test script.

Prep for rec 20250721 releases

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

Merge pull request #15824 from rgacogne/ddist-test-built-in-console-client

dnsdist: Test the built-in console client in our regression tests

Merge pull request #15872 from rgacogne/ddist-only-parse-z-once

dnsdist: Only parse EDNS Z once

Merge pull request #15869 from rgacogne/get21z-speedup

dnsdist: Small speedup for `getEDNSUDPPayloadSizeAndZ()`

pdnsutil tsigkey {de,}activate: do not setDomainMetadata with unchanged data.

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

Make AXFR-MASTER-TSIG really a single item metadata.

In pdnsutil zone show, only list the first item if more than one.
In the API, do not allow setting more than one item.

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

Merge pull request #15867 from miodvallat/interesting_times

Provide record timestamps in the API

dnsdist: Only parse EDNS Z once

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

dnsparser: Add a comment explaining that the OPT RR has to be in additional

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

Merge pull request #15871 from rgacogne/ddist-pool-unordered-map

dnsdist: Use an unordered map for pools

Merge pull request #15870 from rgacogne/ddist-only-create-dnsresponse-when-needed

dnsdist: Avoid constructing a DNSResponse object when not really needed

Merge pull request #15868 from rgacogne/ddist-mac-addr-cache-scalability

dnsdist: Improve the scalability of the MAC address cache

dnsdist: Use an unordered map for pools

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

dnsdist: Avoid constructing a DNSResponse object when not really needed

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

dnsdist: Small speedup for getEDNSUDPPayloadSizeAndZ()

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

dnsdist: Improve the scalability of the MAC address cache

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

Check that timestamp exist and increase if LMDB.

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

That ought to silence clang-tidy.

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

Provide rrset timestamps in LMDB.

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

Filter possible timestamps in rrsets.

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

Return record last modification time in API when known.

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

Merge pull request #15865 from zeha/fsfaddress

Remove old FSF address

Merge pull request #15704 from miodvallat/polyglot

Rework pdnsutil command syntax for more consistency and future flexibility

Merge pull request #15863 from miodvallat/sigaction

RFC9615 fallout

Merge pull request #15861 from rgacogne/ddist-fix-out-of-tree-builds-autotools

dnsdist: Fix out-of-tree builds with autotools

Remove old FSF address

Perform case-insensitive checks for _dsboot and _signal.

Reminded by Kees Monshouwer.

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

appease clang-tidy.

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

Add a few "peek" routines...

...which look at specific record fields without performing a complete
deserialization, especially the memcpy of the record payload.

This (hopefully) allows for faster operation, especially during NSEC3
record maintainance.

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

Fix a few lies in comments.

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

dnsdist: Fix out-of-tree builds with autotools

Stolen from Habbie, many thanks!

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

Be smarter and skip deleting NSEC3 entries which we know do not exist.

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

More sugar for <name withheld>.

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

Still mention the old pdnsutil syntax when applicable.

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

Move "zone show" from DNSSEC to main zone command section.

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

Use new pdnsutil syntax.

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

Use singular for views as object name in pdnsutil.

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

Update documentation to reflect new pdnsutil syntax.

Add an old-to-new command table to the 5.0 upgrade notes.

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

Overhaul pdnsutil command name parsing again.

We are moving towards a more consistent command naming scheme, where the
general syntax is:
  pdnsutil object-type command args...
where 'object-type' is a noun and 'command' is a verb.

For example, in this new world order, "pdnsutil list-zone" becomes
"pdnsutil zone list", "pdnsutil set-meta" becomes "pdnsutil metadata
set", etc.

The old world order commands are still recognized and their behaviour is
not modified.

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

Update repo test script.

The changes to the repo test script hadn't been pushed to the repo for
a while.

Merge pull request #15858 from rgacogne/ddist-20-rc2-secpoll-changelog

dnsdist: Update ChangeLog and the security polling for 2.0.0-rc2

Merge pull request #15848 from omoerbeek/rec-chain-ecs

rec: chain and harden ECS enabled queries

Rework updateEmptyNonTerminal remove-all logic to not use too much memory.

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

dnsdist: Update ChangeLog and the security polling for 2.0.0-rc2

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

Merge pull request #15822 from rgacogne/ddist-fix-nmg-objects-yaml

dnsdist: Allow registering NMG objects from YAML

Merge pull request #15855 from ITJamie/delete-matching-domains-v2

delete existing matching zones during (re)import