Typos in comments

Co-authored-by: Remi Gacogne <github@coredump.fr>
Signed-off-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>
(cherry picked from commit 284d807746195952a735e342ef2f06119812ff54)

Use unmodified flag for post policy

(cherry picked from commit 26c4f7c44ca739fe4a9446677b532e5b56b35806)

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>
(cherry picked from commit 66c8b8bca9218f6d14a76c830634e1e0d39e8f9f)

Merge pull request #15852 from omoerbeek/backport-ecs-to-5.1.x

rec: Backport to 5.1.x: chain and harden ECS enabled queries

Fix test, it was using a feature only available on master

Backport of Fix two issues with chaining ECS enabled queries

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

Switch off leak detect in CI

Merge pull request #15622 from romeroalx/backport-14565-to-rec-5.1.x

rec-5.1.x: backport 14565 - gh actions - coveralls: avoid CI failure by setting fail-on-error: false

gh actions - coveralls: avoid CI failure by setting fail-on-error: false

Fix test names

Implement metrics

Alwas detect mismatches in outgoing and incoming ECS; add tests for that as well

Refactor: get rid of an unneccesary loop

Run all ECS tests also in hardened mode and add a specific hardened mode test

Add setting and metric

Parse ECS info if relevant and act on it if it mismatches

Moved slowParseEDNSOptions() from dnsdist specific code to common code

Fix invalid scope test to do what I think is actually intended

To match incoming, don't look at subnet

Chain on ECS matching, and consider a mismatch in returned ECS as a spoof attempt

Pass actual (optional) ecs instead of bool to asendto()

Merge pull request #15405 from omoerbeek/backport-15352-to-rec-5.1.x

rec: Backport 15352 to rec 5.1.x: in test_SimpleDot.py make sure we have the root DNSKEY in cache, it might require TCP

Update regression-tests.recursor-dnssec/test_SimpleDoT.py

Co-authored-by: Miod Vallat <miod.vallat@powerdns.com>
(cherry picked from commit 173e64959ab4ea2890d76825be2742601727cc42)

rec: in test_SimpleDot.py make sure we have the root DNSKEY in cache, it might require TCP

Previously we assumed we didn't need TCP at all.

(cherry picked from commit 4025ecc524689b38527e31c1f38aa1ebb23e7bc3)

Merge pull request #15280 from omoerbeek/rec-backport-15254-to-5.1.x

rec: Backport 15254 to 5.1.x: If we see both a CNAME and answer records, follow CNAME and discard the answer records

Merge pull request #15287 from omoerbeek/rec-stop-using-ubuntu-20-on-rec-5.1.x

rec-5.1.x: Stop formatting check and start using Ubuntu-24.04 for fuzzing

Stop formatting check and start using Ubuntu-24.04 for fuzzing

Clang-tidy annotation

If we see both a CNAME and answer records, follow CNAME and discard the answer records

Merge pull request #15215 from omoerbeek/backport-14525-to-rec-5.1.x

rec: Backport 14525 to rec-5.1.x: add 38696 root anchor

recursor: add 38696 root anchor

fetched from https://www.iana.org/dnssec/files after their 2024-07-24 update

(cherry picked from commit 0bbbdd60ab054e7bbfc2b0d0604d1f7e15f0a229)

Merge pull request #14872 from romeroalx/backport-14862-to-rec-5.1.x

rec-5.1.x: Backport 14862 - builder: remove ubuntu lunar+mantic as they are EOL

builder: remove ubuntu lunar+mantic as they are EOL

Merge pull request #14797 from omoerbeek/backport-14766-to-rec-5.1.x

rec: Backport 14766 to rec 5.1.x: bump SNMP ignore from buster to bookworm after buildbot VM upgrade

bump SNMP ignore from buster to bookworm after buildbot VM upgrade

(cherry picked from commit 32ac2627fad9d398756c6ec05ae423eec22eff7e)

Merge pull request #14774 from omoerbeek/backport-14693-to-rec-5.1.x

rec: Backport 14693 to rec 5.1.x: implement rfc6303 special zones (mostly v6 reverse mappings)

Merge pull request #14773 from omoerbeek/backport-14628-to-rec-5.1.x

rec: Backport 14628 to rec 5.1.x: rec: avoid duplicated waiter ids for chained requests

Merge pull request #14772 from omoerbeek/backport-14554-to-rec-5.1.x

rec: Backport 14554 to rec-5.1.x: distinguish OS imposed limits from app imposed limits, specifically on chains

Merge pull request #14771 from omoerbeek/backport-14550-to-rec-5.1.x

rec: Backport 14550 to rec 5.1.x: json11: add include for cstdint

Include more recent version of test_Chain.py

Ensure serve-rfc1918 is enabled when doing serve-rfc6303

(cherry picked from commit 458a9064cc3e6eb56a8d6dfb7d501295dd1cf1dd)

Apply suggestions from code review

Co-authored-by: Remi Gacogne <github@coredump.fr>
(cherry picked from commit 565587b70215b7968c9fa711d0bc1b85938f79ed)

rec: implement rfc6303 special zones (mostly v6 reverse mappings)

(cherry picked from commit 9557d437c239d672c49195847a794bb9537f7223)

rec: distinguish OS imposed limits from app imposed limits, specifically on chains

(cherry picked from commit b22ec811be8c094605df49e015be02e9a85d978e)

Disable tracing for test_Chain, it chokes GH

(cherry picked from commit 3755f2f9fa4aac368ac73dc351a38f2af5e63c7e)

Add basic chain test that hits the case of #14624

We do rely on that OS queueing the UDP packets, hope it will do so long enough

(cherry picked from commit 44b1de9830c4c8d7ceec86799c1bdb5a670d3d3e)

rec: avoid duplicated waiter ids for chained requests

(cherry picked from commit b04c64e61cd7b3440ea014da375f47114fe9364b)

json11: add include for cstdint

fixes #14549, which came with a patch

(cherry picked from commit 8d6d32bef74aa2cba41421645936fade35c71c72)

Merge pull request #14743 from omoerbeek/rec-5.1.2-branch

rec: backport of CVE-2024-25590 to rec-5.1.x: limit maximum size of rr sets in record cache

Optimization for the ADDR case: stop loop if we found 2 (A and AAAA)

rec: refuse to store very large RRSets in the cache and ServFail on retrieval of those

Merge pull request #14571 from omoerbeek/backport-14562-to-rec-5.1.x

rec: Backport to rec-5.1.x: Fix build with boost 1.86.0

Fix build with boost 1.86.0

Boost 1.86.0 changes seem to no longer indirectly include header which
causes build to fail with:
```
uuid-utils.cc:38:58:
error: 'random' is not a class, namespace, or enumeration
```

boost/random/mersenne_twister.hpp has been available since Boost 1.21.2

(cherry picked from commit eed56000b1d68ac083b8e8bea4ff0ea30a1579c4)

Merge pull request #14516 from omoerbeek/backport-14514-to-rec-5.1.x

rec: Backport 14514 to rec 5.1.x: fix maintenanceCalls vs maintenanceCount

Update RECURSOR-MIB.txt

fixed maintenanceCalls vs maintenanceCount
It's anyones guess if it should be called Calls or Count in the end, it should be the same name in the end.

(cherry picked from commit 551641127c26799ff8ef80f56887a6ccd0d2b74f)

Merge pull request #14501 from omoerbeek/backport-14499-to-rec-5.1.x

rec: Backport 14499 to rec-5.1.x: limit the number of async tasks pushed to resolve NS names

rec: limit the number of async tasks pushed to resolve NS names

Plus: as we only use a limited set of NS names for resolving,
processing all additional records does not help.

(cherry picked from commit cd2de2ee7ad55f295a00dfce5488ee3863d974d6)

Merge pull request #14481 from omoerbeek/backport-14471-to-rec-5.1.x

rec: Backport 14471 to rec-5.1.x: dump right SOA into dumpFile and report non-relative SOA for includeSOA=true

Merge pull request #14478 from omoerbeek/backport-14404-to-rec-5.1.x

rec: Backport 14404 to rec 5.1.x: Yahttp router: avoid unsigned underflow in match()

Merge pull request #14477 from omoerbeek/backport-14459-to-rec-5.1.x

rec: Backport 14459 to rec 5.1.x: move default Docker config to YAML

Merge pull request #14488 from omoerbeek/backport-14486-to-rec-5.1.x

rec: Backport 14486 to rec 5.1.x: pin pysnmp to version 5 for regression tests

Also pin pysnmp version for dnsdist regression tests

(cherry picked from commit 5d3db32982e7a06a9e653529baa2fb67e24d7189)

rec: pin pysnmp to version 5 for regression tests

(cherry picked from commit 4d44d3076051fe27ee91e847a3600ad19c44fd72)

rec: dump right SOA into dumpFile and report non-relative SOA for includeSOA=true

(cherry picked from commit 397da738caad01df0da04387083c73a0e941608b)

Fix example command line

(cherry picked from commit 7bfa760eb2a7df849c88252f471fc6626fbd4f97)

rec: move default Docker config to YAML

(cherry picked from commit 0f1374a7689a4ee6adcf9ea1185c6116142708df)

Avoid upos underflowing in match loop

(cherry picked from commit 09462bda7f8da4e59307e0d6a7446d499c1da34c)

no else block after break statement

(cherry picked from commit 9ecf2b37287270be3d06ac3d5570f5d67a0bb364)

Merge pull request #14442 from omoerbeek/rec-5.1.x-daily-el7

rec 5.1.x: daily build for el-7 instead of centos-7

rec 5.1.x: daily build for el-7 instead of centos-7

Merge pull request #14435 from omoerbeek/backport-14425-to-rec-5.1.x

rec: Backport 14425 to rec 5.1.x: fix typo in log message

Fix processsing typo.

(cherry picked from commit c91c84f640e32a7d24925a78cf3609e731d098e9)

Merge pull request #14402 from Habbie/backport-14400-to-rec-5.1.x

rec-5.1: switch el7 builds to Oracle Linux 7

switch el7 builds to Oracle Linux 7

(cherry picked from commit 73a1b98f92c671c590540ac19d74d70499f89066)

Merge pull request #14389 from omoerbeek/backport-14384-to-rec-5.1.x

rec: Backport 14384 to rec-5.1.x: keep Lua config as existing configs might use it

rec: keep Lua config as existing configs might use it

(cherry picked from commit 6b2d9507dab4d64623bbca6fcd3bbd57ddebff8e)

Merge pull request #14375 from omoerbeek/rec-specialize-5.1.x

rec: specialize rel/rec-5.1.x branch to rec only workflows

rec: specialize rel/rec-5.1.x branch to rec only workflows

Merge pull request #14374 from omoerbeek/rec-debian-tweaks

Rec followup to #14265: use correct file suffix for reloading and debian tweaks

Also use variable suffix for file when reloading YAML-Lua config

Merge pull request #14363 from omoerbeek/rec-rename-new-udr-settings

rec: make names of new udr settings consistent with existing scheme

Merge pull request #14373 from omoerbeek/rec-dns64-double-soa

rec: don't send double SOA record in the case of a dns64 CNAME that does not resolve

Add test for duplicate SOA record in the dns64/NODATA case

rec: tweak debian install, as suggested by @zeha

Merge pull request #14265 from omoerbeek/rec-yaml-as-conf

rec: allow recursor.conf file to contain YAML to ease transition to YAML config

Merge pull request #14366 from PowerDNS/dependabot/pip/pdns/keyroller/urllib3-1.26.19

build(deps): bump urllib3 from 1.26.18 to 1.26.19 in /pdns/keyroller

Merge pull request #14360 from rgacogne/auth-fix-distributor-unit-test-leak

auth: Better fix for the leak reported by LSAN in test-distributor_hh.cc

build(deps): bump urllib3 from 1.26.18 to 1.26.19 in /pdns/keyroller

Bumps [urllib3](https://github.com/urllib3/urllib3) from 1.26.18 to 1.26.19.
- [Release notes](https://github.com/urllib3/urllib3/releases)
- [Changelog](https://github.com/urllib3/urllib3/blob/main/CHANGES.rst)
- [Commits](https://github.com/urllib3/urllib3/compare/1.26.18...1.26.19)

---
updated-dependencies:
- dependency-name: urllib3
  dependency-type: indirect
...

Signed-off-by: dependabot[bot] <support@github.com>

Merge pull request #14345 from PowerDNS/dependabot/pip/pdns/recursordist/docs/urllib3-2.2.2

build(deps): bump urllib3 from 2.2.0 to 2.2.2 in /pdns/recursordist/docs

auth: Use [[maybe_unused]] in the distributor unit tests

As suggested by Otto.

Merge pull request #14318 from omoerbeek/rec-nsname-pbout

rec: add nsName into outgoing protobuf request/response messages

Install a small YAML default config instead of one with everything commented out

Document the new approach of reading YAML from a recursor.conf file

Generate no more .conf-dist file

Adapt RH packaging

Adapt Debian packaging

rec: allow recursor.conf file to contain YAML

This should us to work around the packaging issues discussed in #13935.
THe idea is that modify the parsing so that .conf files also *may* contain YAML.

The search for a config file then becomes:

1. Try read recuror.yml if it exists. If valid, done. If it is invalid punt.
2. Try read recursor.conf as YAML. If it is valid, done.
3. If it is invalid, try to read as old-style.

This means that the status of recursor.conf as a config file does not change.

This allows us to install a default YAML config into recursor.conf for new installs.
Of course we leave recursor.conf (and recursor.yml) alone for existing installs.

This is a draft. I will add docs and packaging changes after this is deemed
the way to proceed.

rec: make names of new udr settings consistent with existing scheme

Remove potential double SOA records if the target of a dns64 name is NODATA

Use const ref for string arg

Co-authored-by: Remi Gacogne <github@coredump.fr>

Merge pull request #14359 from omoerbeek/use-ucharview

dns.cc: use pdns::views::UnsignedCharView