Merge pull request #10081 from pieterlexis/issue-7203-pdnsutil-load-zone

pdnsutil load-zone: reject zones with broken rrs

Merge pull request #10083 from omoerbeek/rec-doc-trace-fail

rec: Document trace=fail better and warn against its shortcomings.

Merge pull request #10087 from pieterlexis/check-MX-to-CNAME

pdnsutil: Warn on CNAME targets for NS, MX and SRV

Merge pull request #10094 from Habbie/speedtest-nsec3

speedtest: add NSEC3 hashing test

Merge pull request #10095 from rgacogne/ddist-fix-crash-dbpf-console

dnsdist: Prevent a crash with DynBPF objects in client mode

Merge pull request #10092 from omoerbeek/docs-server-proxy

docs: Avoid some confusion about server objects

fix Makefile _SOURCES order

speedtest: fix compiler warnings

speedtest: add NSEC3 hashing test

dnsdist: Prevent a crash with DynBPF objects in client mode

Quoting

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

spelling

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Avoid some confusion about server objects

Merge pull request #10088 from rgacogne/ddist-start-tcp-workers-before-acceptors

dnsdist: Create TCP worker threads before acceptors ones

Merge pull request #10053 from aerique/feature/python-repo-script

Replace shell-based repo test script with a Python version

Merge pull request #9468 from omoerbeek/rec-control-pass-fd

rec: Rec control pass fd

Wrap the fd so it will be closed automatically

Python version of repo test script.

This replaces the Bourne shell version.

pdnsutil: Warn on CNAME targets for NS, MX and SRV

Fixes #10085

dnsdist: Create TCP worker threads before acceptors ones

Otherwise we might crash when the first TCP query arrives because
we are dividing by zero when trying to select a worker.

Merge pull request #10075 from omoerbeek/rec-setting-ecs-never-cache

rec: Introduce settings to never cache EDNS Client (v4/v6) Subnet carrying replies

Apply suggestions from code review

Co-authored-by: Remi Gacogne <rgacogne+github@valombre.net>

Actually define the two new args

Introduce settings to never cache EDNS Client (v4/v6) Subnet carrying replies.

While there, add a few entries to the upgrade guide.

Merge pull request #10086 from jsoref/pip-wheel

Add wheel to pip

Add wheel to pip

Document trace=fail better and warn against its shortcomings.

Merge pull request #10082 from pieterlexis/issue-8517-LUA-record-split

docs: notes on LUA record splitting

Add a general note to the manpage on how file handling is changed.

Also, remove the snippets about where the files are written in
case of chroot for each file-writing command.

AlLow - for stdout, does need a tweak in the argument parser...

Handle all commands that dump to file

Apply suggestions from code review

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

docs: notes on LUA record splitting

Process review comments; most importantly a separate type for
RecursorControlChannel::Answer

Merge pull request #10073 from rgacogne/ddist-spoof-multiple-from-lua

dnsdist: Add 'spoof' and 'spoofRaw' Lua bindings

pdnsutil load-zone: reject zones with broken rrs

Return command status and output as two separate items

dump-cache x is two args, not one

Allow dump-cache stdout
(a hyphen does not work due to the "funny" command line parser)

Open file in rec_control

Rough draft of fd passing for rec_control

Merge pull request #10076 from omoerbeek/rec-docs-dnsname

rec: Take the DNSName docs from auth, they are more complete

Merge pull request #9 from AntagonistHQ/ddist-spoof-multiple-from-lua

dnsdist: add documentation about DNSQuestion:spoof

dnsdist: add documentation about DNSQuestion:spoof

Merge pull request #10077 from omoerbeek/rec-spoof-nearmiss-max-default

Change spoof-nearmiss-max default to 1.

Merge pull request #9819 from omoerbeek/better-qtype

Improve the way QType behaves by defining proper conversion operators and a hash function.

Update pdns/recursordist/docs/settings.rst

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Change spoof-nearmiss-max default to 1.

Fixes #9845.

Take the DNSName docs from auth, they are more complete.

We really should be having one source for all common Lua docs.
Fixes #9608.

Also convert te recursor cache, adding the needed hash function for boost

Keep const QType in definitions and introduce a nicer way to build
the name to number mapping and vice versa.

Simplify boilerplates a bit by deriving the QType from the RNAME by using
QType::RNAME, enforcing that one must exist.

Improve the way QType behaves by defining proper conversion
operators and a hash function.

I converted syncres.* for starters to stop using uint16_t for qtype
and use QType everywhere.  Note that I also changed const QType&
in arg lists by QType, since it makes little sense to pass a 16 bit
value by const reference.

Merge pull request #10022 from omoerbeek/rec-missing-prometheus-entries

rec: Move prometheus table to web server and add missing entries.

Merge pull request #8 from AntagonistHQ/ddist-spoof-multiple-from-lua

Merge dq:spoof and dq:spoofRaw and add regression tests for dq:spoof

Merge dq:spoof and dq:spoofRaw and add regression tests for dq:spoof

dnsdist: Add 'spoof' and 'spoofRaw' Lua bindings

This commits makes it possible to spoof multiple values from Lua:

```
function myfunc(dq)
  local ret = { newCA("192.0.2.1"), newCA("192.0.2.2") }
  dq:spoof(ret)
  return DNSAction.HeaderModify
end

addAction(AllRule(), LuaAction(myfunc))
```

Barely tested, need tests and documentation.

Merge pull request #10063 from AntagonistHQ/spoof_multi

dnsdist: Add option to spoofRawAction to spoof multiple answers

Merge pull request #10005 from Habbie/auth-4.4.1-docs

changelog&secpoll for auth-4.4.1

Merge pull request #8813 from RobinGeuze/checkForDuplicateMetadata

Make check-zone also check whether there are duplicate key value pair metadatas for the zone

Merge pull request #10070 from omoerbeek/clang-tidy-followup

rename a few `d_*` locals

Do an exact comapre, since set.count can only return 0 or 1

Update pdns/pdnsutil.cc

Accept textual change

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Make check-zone also check whether there are duplicate key value pair domaindatas for the zone

add last two PRs and update date

auth-4.4.1 changelog: add #10039

spelling

fix pullreq links in auth-4.4.0 docs

changelog&secpoll for auth-4.4.1

Merge pull request #10068 from Habbie/no-more-yum

docker builds: stop using yum on EL8 derivatives

Merge pull request #10016 from Habbie/builder-archs

builder: add arch-specific targets

Merge pull request #10064 from omoerbeek/rec-fix-empty-cname-followup

rec: Return current rcode instead of 0 if there are no CNAME records to follow

some d_* locals

Merge pull request #9672 from neheb/clng

clang-tidy fixes

dnf install yum so Dockerfile.rpmbuild can stay simple

Merge pull request #10069 from mnordhoff/dns-over-https-colon-colon

dnsdist docs: Fix formatting of response map example

dnsdist docs: Fix formatting of response map example

docker builds: stop using yum on EL8 derivatives

Merge pull request #10007 from mind04/auth-tcp-counters

auth: fix tcp answer counters

Merge pull request #10029 from Habbie/actions-docker

gh actions: build Docker images

Another missed one

Merge pull request #10060 from omoerbeek/sdig-add-dot

sdig: add DoT

Return current rcode instead of 0 if there are no CNAME records to follow.

Note that this is a change in behaviour. While it is for the good, it might
be existing code depends on the old 0 value...

Missed a few locals starting with d_

Catch exception on stats entries init and exit on that

Merge pull request #10036 from rgacogne/rec-asan-null-stack

rec: Set the start of the stack right away to avoid an ASAN issue

gh actions: build Docker images

dnsdist: Add option to spoofRawAction to spoof multiple answers

Stray comma and document some older provider libs do no do validation at all.

Do not use local vars that start with d_, our naming convention is that
those are fields of a class.

Merge pull request #10061 from omoerbeek/format-option-fix

fix a clang-format issue difference in opinion

Add dolog.hh to sdig sources

Fix compilation on CentOS-7, which lacks gnutls_session_set_verify_cert().

Make explicit gnutls only validates the certificate if a server hostname is given.

Apply suggestions from code review

Co-authored-by: Remi Gacogne <rgacogne+github@valombre.net>

Setting this flag appears to fix one difference between clang-format-8 and 10

Tweaks and man page updates

clang-tidy: pass by value and use std::move

Found with modernize-pass-by-value

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

clang-tidy: remove pointless std::move

Found with performance-move-const-arg

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

clang-tidy: several reference conversions

Found with performance-unnecessary-value-param

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

clang-tidy: replace bind with lambdas

Found with modernize-avoid-bind

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