Update pdns/zoneparser-tng.cc

Co-authored-by: Miod Vallat <miod.vallat@powerdns.com>
Signed-off-by: Jamie (Bear) Murphy <1613241+ITJamie@users.noreply.github.com>

strip semicolon from import files

strip semicolon from import files

Merge pull request #15849 from Habbie/pdnsutil-absolute-upgrading

auth: upgrade note for absolute names in pdnsutil

Merge pull request #15817 from Habbie/docker-images-trixie

switch Docker images to Debian Trixie

Merge pull request #15850 from miodvallat/nsecure

Fix lmdb-nsec3-optout-variant tests

Correctly recognize *-nsec3-optout-variant as "optout".

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

auth: upgrade note for absolute names in pdnsutil

this documents the changes in #14984

Merge pull request #15827 from miodvallat/lycos

Add record search capability to the LMDB backend

Merge pull request #15847 from miodvallat/master-unbreakage

unbreak auth tests

Merge pull request #15836 from ukleinek/doc-zone-rrset-modified_at

docs: zone: `modified_at` is a separate property when updating RRSets

Unbreak pdnsutil invocation after #14984 merge

(pdnsutil {add-record,delete-rrset}: require NAME to be absolute)

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

Merge pull request #14984 from ukleinek/pdnsutil-absolute-names

pdnsutil {add-record,delete-rrset}: require NAME to be absolute

Add record search capability to the LMDB backend.

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

Merge pull request #15786 from miodvallat/let_me_dnsupdate_better

lmdb rfc2136 support

misc: Use boost::ends_with() in isCanonical instead of open-coding

boost:ends_with(qname, ".") behaves exactly as isCanonical(qname)
should. So use the first to implement the latter.

pdnsutil {add-record,delete-rrset}: Require NAME to be absolute

Note this is an incompatible change because calls like

pdnsutil --config-dir=configs/auth add-record example.net . NS 1.2.3.4

which added the NS record to the zone's apex before and is an error now.

Note that also "@" isn't interpreted any more as "at the apex".

Adapt the test suite and docs accordingly to use absolute names.

Closes: https://github.com/PowerDNS/pdns/issues/8595

Merge pull request #15833 from Darwin4053/clean-fix

Fix narrowing conversion on 32-bit systems by using uint64_t instead …

Fix narrowing conversion on 32-bit systems by using uint64_t instead of size_t

Merge pull request #15819 from rgacogne/ddist-yaml-rcode-as-string

dnsdist: Handle named rcodes in the YAML configuration

Merge pull request #15829 from zeha/dnsdist-rustctargetarch

dnsdist-rust-lib: honor RUSTC_TARGET_ARCH

Merge pull request #15835 from Habbie/i-sell-dates-by-the-sea-shore

datefudge: more portable date parsing; py3 fixes

switch scripts to explicit python3

docs: zone: `modified_at` is a separate property when updating RRSets

`modified_at` is a stand alone property, so it warrants its own item in
the RRSet object properties.

datefudge: more portable date parsing

dnsdist-rust-lib: honor RUSTC_TARGET_ARCH

Like for pdns-recursor and also like in the meson build.

Necessary as Debian's cargo always puts the output files into target/<arch>/...

Document DNS Update available for the LMDB backend.

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

dns: Fix another clang-tidy warning

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

Merge pull request #15825 from miodvallat/fewer_mistakes

Fewer mistakes in the auth documentation

Missed test oracle.

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

Document ENABLE-LUA-RECORD metadata.

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

More metadata documentation fixes:

- API-RECTIFY is not visible in the API.
- TSIG-ALLOW-AXFR is writable.
- only list TSIG-ALLOW-DNSUPDATE once.

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

Sort metadata variables. Mention SOA-EDIT is readonly in the API.

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

Document resolver should not point to ourselves.

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

Enable dyndns tests with lmdb.

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

Add dyndns test oracles for lmdb.

Unfortunately, due to the crude adaptation of mysqldiff, we do not
gather all the interesting data and can not check subtle details,
especially NSEC3 chaining details.

Also, a few tests are currently disabled because the way LMDB builds
NSEC3 chains is still not 100% correct.

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

dns: Add short descriptions for extended rcodes as well

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

dns: Fix a clang-tidy warning

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

Add some (limited) zone variant awareness.

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

Make some tests mysql-agnostic...

... by replacing actual SQL operations on the database with proper
pdnsutil command invocations, which can then work regardless of the
backend being used.

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

Correctly remove NSEC3 records when removing ENT records.

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

dnsdist: The ability to use RCode names in YAML will be backported to 2.0

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

dns: Add unit tests for the RCode/ERCode/Opcode helpers

dns: Add short description for the Not Zone rcode

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

dns: Rewrite `RCode::from_short` with `std::find` and `std::distance`

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

Make this truly work with variants...

...by undoing some of the bugs introduced in
ab319bd3eac77cb321d71964ba237ae1e238db49. We would ent up marking
records in variant zones as never authoritative by mistake.

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

Implement listSubZone in LMDB backend.

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

Shuffle/factor internal query logic in preparation for future changes, NFCI

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

Ѕcarce documentation of the list/lookup/get shared state.

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

Heed the domain_id in list rather than try and recompute it. NFCI

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

Remove unused instance members. NFC

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

Merge pull request #15790 from jsoref/refresh-spelling-v0.0.25-2

Refresh spelling v0.0.25 2

Merge pull request #15803 from omoerbeek/rec-nsspeed-share

rec: add Lua hooks to dump and restore measured nameserver speed table

dnsdist: Fix a typo in the upgrade guide

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

dnsdist: Document changed YAML parameters

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

dnsdist: Handle named rcodes in all YAML configuration items

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

dnsdist: Handle named rcodes in the dynamic block YAML configuration

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

dnsdist: Add a Dynamic block rcode rate test using YAML

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

Merge pull request #15806 from miodvallat/entropy

auth: one less discrepancy between lmdb and sql backends

switch Docker images to Debian Trixie

Merge pull request #15783 from omoerbeek/rec-docs-policy-mod

rec: polish docs describing how to modify policy decisions

Merge pull request #15794 from rgacogne/ddist-protocol-selector

dnsdist: Add a selector to match the incoming protocol

Merge pull request #15815 from rgacogne/ddist-fix-source-backend-yaml

dnsdist: Properly process the YAML source parameter for backends

Merge pull request #15813 from rgacogne/ddist-autotools-dlopen

dnsdist: Properly link with `libdl` when building with `autotools`

dnsdist: Properly process the YAML source parameter for backends

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

dnsdist: Properly link with `libdl` when building with `autotools`

Depending on the system we might actually need to link with `libdl`
when our Rust library is used, and the mechanism to do that was not
properly set up when building with `autotools` (we were adding `LIBDL`
to the the libraries we need but the variable was not properly filled).
Unfortunately the systems we are exercising in our CI do not need to
explicitly link with `libdl` so we did not notice.

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

Merge pull request #15807 from omoerbeek/rec-prep-5.3.0-alpha2

Prep for rec-5.3.0-alpha2

Process review comments from rgacogne

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

Add missing files to testrunner sources

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

Formatting

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

name of nsspeed entries can be empty (auth case)

Add test

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

Documentation

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

Reorg sources, split nsspeeds_t out into separate .cc and .hh

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

Add size limit

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

Basic code to fill ns speed table from a dump

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

Expose getNSSPeedsTable() to Lua script

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

Dump ns speed map in protobuf format

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

Prep for rec-5.3.0-alpha2

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

Make updateEmptyNonTerminals conform to its specification...

...by performing the insertions (if any) when all ENT had to be removed
first.

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

Merge pull request #15798 from miodvallat/flaggerbasted

lmdb NSEC3 record handling hygiene: return of the wrath of the seventh son of the phantom of the beast

Merge pull request #15799 from kpfleming/pblogger-rs

Add Protobuf logger written in Rust.

Merge pull request #15759 from elenril/preoutquery_force_tcp

rec: allow forcing TCP from preoutquery()

Tidy

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

Add test

rec: allow forcing TCP from preoutquery()

Merge pull request #15801 from rgacogne/ddist200rc1-changelog-secpoll

dnsdist: Prepare ChangeLog and security polling zone for 2.0.0-rc1

Merge pull request #15639 from Habbie/alma10

builder: add el-10 target, based on rockylinux:10 for now

Merge pull request #15788 from neheb/npd

clang-tidy: replace lock_guard with scoped_lock

clang-tidy: replace lock_guard with scoped_lock

Found with modernize-use-scoped-lock

Signed-off-by: Rosen Penev <rosenp@gmail.com>

docs: add new targets to daily master builds

nit

dnsdist: Use the correct month for the release date

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

Address review feedback.

Eliminate some code duplication.

Improve make_addr_port by letting SocketAddrV4/V6 format themselves.

Add Protobuf logger written in Rust.

This was inspired by the ProtobufLogger.py already present in the
'contrib' directory.

In addition to being written in Rust instead of Python, there are
various other differences:

* Each line of output is prefixed with the sender's 'socket address'
  (IP address and port number).

* Messages from multiple clients will be properly output, they will
  not be mixed.

* Timestamp format is slightly different (full ISO-8601 with UTC
  offset).

* Command-line arguments are handled by a full parser, which can
  generate help text and report the program's version.

* All 'optional' fields in the protobuf messages are checked for
  presence before being read.

* Output to stdout will never block reception/decoding/formatting of
  protobuf messages; if stdout blocks for some reason, incoming
  messages will be stored in memory until they can be printed.

* Summary, meta, and question lines are printed; responses are not,
  nor is OpenTelemetry data. Future work for another contributor!

* 'meta' output is untested.

* A Cargo feature 'opentelemetry' is available to be the starting
  point of OT support.

No AI or LLM tools were used in the creation or testing of this code.

dnsdist: Prepare ChangeLog and security polling zone for 2.0.0-rc1

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

Merge pull request #15792 from miodvallat/auth496

auth-4.9.7 secpoll & changelog

Document ill-fated 4.9.6 so that people aren't surprised too much.

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

Turns out it will be called 4.9.7

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