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 #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>

clang-tidy: add reserve

Found with performance-inefficient-vector-operation

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

clang-tidy: use const reference

Found with performance-for-range-copy

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

clang-tidy: use references

Found with performance-unnecessary-copy-initialization

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

clang-tidy: use '' with single character find

Found with performance-faster-string-find

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

clang-tidy: use raw string literals

Found with modernize-raw-string-literal

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

clang-tidy: use nullptr

Found with modernize-use-nullptr

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

clang-tidy: remove pointless void

Found with modernize-redundant-void-arg

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

clang-tidy: use bool literals

Found with modernize-use-bool-literals

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

clang-tidy: convert loops to range based

Found with modernize-loop-convert

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

Merge pull request #10048 from Habbie/oss-dns-vulns

docs: link to OARC oss-vulns list; add security policy to dnsdist; include GPL license; some minor refactoring

Merge pull request #10043 from Habbie/build-oraclelinux-8

builder: support oraclelinux-8; verify build in GH Actions on push&PR

Merge pull request #10042 from pieterlexis/dockerfiles-version

dockerfiles: Ensure version number is correct for releases

Merge pull request #10028 from Habbie/docker-odbc

auth docker: find unixodbc on non-x86_64 platforms

Merge pull request #9636 from wojas/proto-file-license

dnsmessage.proto: make CC0 Public Domain

Merge pull request #8999 from kpfleming/add-error-responses-to-api-docs

Add error response objects to all API operations in spec file

Merge pull request #10046 from rgacogne/doc-nmt-fork

Make NetmaskTree::fork() a bit easier to understand

Merge pull request #10050 from rgacogne/ddist-web-log

dnsdist: Remove forgotten debug line in the web server

Choose provider add a CAStore on command line

Cert validation

There is life in TLS...

docs: add GPL license to all 3 product docs

dnsdist docs: add Security Policy to TOC

dnsdist: Remove forgotten debug line in the web server

dnsdist docs: avoid warnings about rst files in .venv

dnsdist docs: add security reporting policy

Baseline for DoT integration into sdig, taken from Habbie/sdig-dot-pin

allow "vulns" in spelling

docs security policy: steal OARC link&text from https://www.isc.org/security-report/

Merge pull request #10037 from mind04/pdns-delete-domain

auth: run deleteDomain() inside a transaction

shorten title - improves readability in GitHub Checks interfaces

Make NetmaskTree::fork() a bit easier to understand

Merge pull request #10044 from aerique/feature/dnsdist-16-repo-script

Add dnsdist 1.6.x to repo script.

Add dnsdist 1.6.x to repo script.

docker: use unbuffered output in init

builder: support oraclelinux-8; verify build in GH Actions on push&PR

Merge pull request #9990 from omoerbeek/rec-tcp-packetcache

Rec: packetcache for tcp queries

Merge pull request #10040 from Habbie/dnsdist-registerwebhandler-docs

dnsdist docs: formatting + spelling in registerWebHandler explanation

auth: run deleteDomain() inside a transaction

dockerfiles: Ensure version number is correct for releases

Merge pull request #10025 from rgacogne/ddist-160-a1-changelog-secpoll

dnsdist: ChangeLog and secpoll update for 1.6.0-alpha1

dnsdist: Set 1.6.0-alpha1 to "1 OK" in the secpoll zone

dnsdist docs: formatting + spelling in registerWebHandler explanation

dnsdist: Move 10035 in the right section of the ChangeLog

update pdns-builder

dnsdist: Set the release date to today

Merge pull request #10038 from Habbie/test-luawrapper

luawrapper test: improve comment

luawrapper test: improve comment

dnsdist: Add 10035 to the ChangeLog

Merge pull request #9859 from rgacogne/hashqnamewithsalt-optim

Don't copy the same salt for all iterations in hashQNameWithSalt()

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

We used to wait until the first invocation of a MTask to set the
start of the stack, but that sometimes resulted in passing the nullptr
address to ASAN when calling a task for the first time.
It resulted in ASAN skipping the stack switch, logging something like:
```
WARNING: ASan is ignoring requested __asan_handle_no_return: stack top: 0x000000020000; bottom 0x7f18f174a000; size: 0xffff80e70e8d6000 (-139745106763776)
False positive error reports may follow
```
Then almost right away complaining about a stack-use-after-scope,
or a stack-based overflow.

This changes sets the end of the memory allocation before the first
invocation, so that we always notify a valid value. A closer
approximation is still set during the first invocation, as before.

Merge pull request #10035 from rgacogne/ddist-static-fixes

dnsdist: Fix a few warnings reported by clang's static analyzer and cppcheck

Merge pull request #10030 from Habbie/protozero-includes

protozero: make internal includes work