Add test

rec: don't drop partial result on exception in cache lookup

Fixes #14310

Should be checked for unwanted side-effects, which is pretty tricky.

Merge pull request #14807 from omoerbeek/rec-packetid-ref

rec: drop ref in mtasker when it is no longer needed

Merge pull request #14817 from oregonpillow/master

Add dnsdist documentation example for using DoH behind nginx

Merge pull request #14768 from chbruyand/dnsdist-load-tickets-key

dnsdist setTicketsKeyAddedHook(): adjust type of the key to fix potential truncate

fix spell-check. allow grpcs

typo fix

Merge pull request #14796 from omoerbeek/rec-cache-dump-undump

rec: provide a few more Lua functions, most importantly record cache dump and load

Update dns-over-https.rst

remove 'include' directives from nginx conf example

Add a unit test checking the ref counts (fails in master)

Merge pull request #14809 from omoerbeek/rec-deprecate-old-style

rec: actively deprecate old style settings

Update dns-over-https.rst

Apply suggestions from code review

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

Replace a few asserts with throws and make sure we've seen the version and type fields in putRecordSet.

An exception will be handled by the catch at the end of putRecordSets()

Prompted by @rgacogne

Apply suggestions from code review

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

Remove the move call in two instances where it really does not add anything

Docs tweaks (not neccesarily related)

Docs of new behaviour: do not do old-style by default

Give a more sensible message for rec_control show-yaml if recursor.conf already contains YAML

rec: disable parsing of old-style settings by default

Merge pull request #14816 from omoerbeek/rec-periodic-stats

rec: periodic stats cleanup and rename outqueries-per-query to outqueries-per-query-perc

Merge pull request #14801 from omoerbeek/rec-valgrind

rec: make valgrind build and testrunner run work again

Missing quotes as noted by @chbruyand

Co-authored-by: Charles-Henri Bruyand <charleshenri.bruyand+github@gmail.com>

rec: periodic stats cleanup and rename outqueries-per-query to outqueries-per-query-perc

Also
- cleans up two TCounters that where unused and/or redundant.
- group the log lines more naturally
- print doubles with two decimal places

Merge pull request #14792 from rgacogne/ddist-coverity-202410

dnsdist: Fix Coverity warning 445614: " Argument cannot be negative"

Merge pull request #14794 from phonedph1/no-makerule

dnsdist: stop using makeRule, update Qname to QName so links should work

Reformat

Correct initial value of d_used

Tidy

Two more case, assert on inconsistency

Same move treatment for value

Here no refs being lowered (for the rec case) but it might save a few cycles

rec: release ref to key in MTasker::waitEvent

If we do not do that, only the next call of sendEvent will release
the shared pointer to the TCPConnectionHanlder in PacketID. This
might delay cleaning TCP objects up on not-so-busy threads.

Fixes #13422, thought it can still take a while on idle recursors,
as the housekeeping function is not called often in those cases.

Allow zero for no limit in getRecordCacheRecords()

Also better distinction between the term record and record set in code and docs

rec: make valgrind build and testrunner run work again

The testrunner run becomes very slow, a non-zero inception skew is needed.

Merge pull request #14787 from fredmorcos/lmdb-typed-cleanups

Cleanup lints & macros in lmdb-typed.hh

clang-tidy tweaks

Refactor prompted by CodeQL

Reorganize a few things, add a Lua hook that's called on startup and (nice) shutdown

Add a test and a fix a few things in getRecordCacheRecords/putIntoRecordCache

Document putIntoRecordCache and getRecordCacheRecords

Refactor some code, fix RU scanning (it produced LRU instead of MRU entries before)

Make Lua startup script customizable

Extend pdnslog to allow for structured key-value pairs

Cleanup of PB encoding, including logging and some meta info

Better (faster) encoding/decoding

Basic functions to dump and undump record cache content

stop using makeRule, update Qname to QName so links should work

dnsdist: Fix Coverity warning 445614: " Argument cannot be negative"

Cleanup lints & macros in lmdb-typed.hh

Merge pull request #14784 from fredmorcos/small-cleanups

Small cleanups

Merge pull request #14714 from Habbie/el8-gcc-toolset-13

el-8: use gcc-toolset-13

Merge pull request #14785 from rgacogne/fix-lmdb-safe-except

dnsdist: Add missing `stdexcept` include in `lmdb-safe.hh`

Merge pull request #14777 from fredmorcos/fix-clang-tidy-again

Fix clang-tidy (again)

dnsdist: Add missing `stdexcept` include in `lmdb-safe.hh`

It was only included for non-dnsdist contexts, but `std::runtime_error`
is used in all contexts and gcc version 14.2.1 20240910 (at least) is
not happy about it:
```
In file included from ext/lmdb-safe/lmdb-safe.cc:2:
ext/lmdb-safe/lmdb-safe.hh: In member function ‘int MDBROTransactionImpl::get(MDB_dbi, const MDBInVal&, MDBOutVal&)’:
ext/lmdb-safe/lmdb-safe.hh:379:18: error: ‘runtime_error’ is not a member of ‘std’
  379 |       throw std::runtime_error("Attempt to use a closed RO transaction for get");
      |
```

Some meson cleanup work before lmdb-safe tests

Minor formatting and cleanup

Merge pull request #14782 from fredmorcos/rework-mdb-inout-vals

Rework MDBIn/OutVal

Merge pull request #14658 from phonedph1/moredump

dnsdist: dump more packet cache

Statically ensure MDBInVal can store requested values

Merge pull request #14781 from bjacquin/dev/beber/pdns-version-string

pdns: allow empty string in version-string

Fixup includes

Rework MDBInVal

Rework MDBOutVal

Fix std::string include in lmdb-typed header

LMDBLS Cleanups

lmdb-safe header lint & formatting cleanups

Merge pull request #14778 from fredmorcos/lmdb-safe-cleanups

Some cleanups for lmdb-safe

Merge pull request #14718 from emmanuel-ferdman/master

regression-tests.auth-py: remove duplicate method

Cleanup includes in lmdbbackend

Refactor shards cmdline handling

Cleanup LMDBBackend::upgradeToSchemav5

pdns: allow empty string in version-string

This change allow an empty string to be returned to CHAOS version.bind
request. version-string by default is set to "full" through
pdns/auth-main.cc declareArguments(), however does not allow an empty
string to be returned as PowerDNS Recursor does.

Some cleanups in lmdbbackend

Cleanup TypedDBI

Cleanup struct nullindex_t

Cleanup struct index_on and index_on_function

Cleanup LMDBIndexOps

Cleanup makeCombinedKey

Use uint32_t in makeCombinedKey

Specialize MDBOutVal::get/getNoStripHeader for uint32_t

Use uint32_t in MDBGetMaxID and MDBGetRandomID

Clang-tidy cleanups of lmdb-typed

Fix clang-tidy (again)

Merge pull request #14775 from omoerbeek/rec-zap-pdns_hw.cc

rec: Delete dangling symlink pdns_hw.cc

rec: Delete dangling symlink pdns_hw.cc

Merge pull request #14770 from omoerbeek/rec-docs-metrics

rec: generate metrics docs when publishing

Fix a few refs

Generate metrics doc file

Merge pull request #14621 from fredmorcos/meson-lmdb-regtests

Add lmdb regression tests to meson

dnsdist setTicketsKeyAddedHook: pass a std::string to the hook to avoid luawrapper to truncate content at potential null chars

Merge pull request #14766 from Habbie/rec-ignore-snmp-bookworm

bump SNMP ignore from buster to bookworm after buildbot VM upgrade

bump SNMP ignore from buster to bookworm after buildbot VM upgrade

fix name of notify binary

Merge pull request #14764 from chbruyand/dnsdist-load-tickets-key

dnsdist: add the ability to load a given TLS tickets key

Merge pull request #12752 from kpfleming/issue-12714

auth doc: Document 2-step process for consumer zone updates.

Merge pull request #14665 from Habbie/sdig-stdin-improvements

auth sdig: some improvements for stdin mode

fix Remi's comments and some formatting

dnsdist: add a global loadTicketsKey function

dnsdist: add the ability to load a given tickets key from lua

Merge pull request #14721 from omoerbeek/followup-14705

Followup to #14705: clang-tidy does not pick up last option listed