Another missed one

Missed a few locals starting with d_

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

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

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

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

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()

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

Merge pull request #10031 from Habbie/test-luawrapper

test luawrapper in make check

test luawrapper registerFunction in make check

protozero: make internal includes work

auth docker: find unixodbc on non-x86_64 platforms

dnsdist: Silence a 'performance' warning from cppcheck

Work around false positives from cppcheck

dnsdist: Fix a bug when loading the rings from a file

dnsdist: Correctly handle numerical web passwords and keys

dnsdist: Remove a construct that cppcheck is choking on

Silence false positives about unintialized values

Silence warnings about values set but not used

Pass strings as const references in exceptions

Fix rebase error (which was introduced by git doing a merge the wrong way)

Make deep nesting more shallow by early returns

Process review comments, most importantly refactoring the fd accounting code after sending a TCP reply

Tidy

Adapt regression test to TCP having a packetcache now

Add unit tests and simplify code.

Refactor cache hit path and do cache check for tcp

dnsdist: Add 9997 to the 1.6.0-alpha1 ChangeLog

Merge pull request #9997 from rgacogne/ddist-concurrent-conns

dnsdist: Limit the number of concurrent console and web connections

Add more unit tests directly testing hashQNameWithSalt()

Add a unit test directly testing hashQNameWithSalt()

Only call std::string::reserve() when needed in hashQNameWithSalt()

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

The salt does not change between iterations, and the hash size is
constant, so we can just overwrite the hash instead.

dnsdist: Add unit tests for the connection management code

dnsdist: Limit the number of concurrent web connections

dnsdist: Limit the number of concurrent console connections

Fix typos

dnsdist: ChangeLog and secpoll update for 1.6.0-alpha1

Merge pull request #9950 from pieterlexis/maxtcpclient-error

dnsdist: Improve error with a hint on how to fix it

ddist: improve error with a hint on how to fix it

Merge pull request #10011 from pieterlexis/ddist-doc-deprecation

dnsdist: clean up docs for 1.6.0

Merge pull request #10023 from omoerbeek/dnsdist-vdt-BasicQPSLimiter

dnsdist: Silence clang 12 warning

Title underline too short

Merge pull request #10004 from omoerbeek/rec-2038-unit-tests

rec: fix recursor caches to handle timestamps > 2038

Silence clang 12 warning: destructor called on non-final 'QPSLimiter' that has virtual
functions but non-virtual destructor

Merge pull request #10014 from omoerbeek/rec-validate-after-2038

Rec: validate signatures after 2038 and more

Use named qtype

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

Remove functions deprecated before 1.4.0

remove all pre-1.4.0 versionchanged tags

remove all pre-1.4.0 versionadded tags

Merge pull request #10020 from omoerbeek/rec-taskqueue-metrics

rec: Document taskqueue metrics and add them to SNMP MIB

Merge pull request #10019 from omoerbeek/rec-dnssec-cd-docs

rec: Document CD bit better, reference detailed DNSSEC docs from settings

Typos and spelling

Document taskqueue metrics and add them to SNMP MIB

Document CD bit better, reference detailed DNSSEC docs from settings
and reformat two item lists.