Coverity 1469728 Resource leak by using annotation

Merge pull request #11587 from omoerbeek/rec-outgoing-pb-race

rec: Avoid a race in test code between processing the first query and the priming code

Merge pull request #11586 from Y7n05h/doc

dnsdist: fix wrong parameter type in document

Merge pull request #11543 from rgacogne/ddist-console-max-size-log

dnsdist: Log when a console message exceeds the maximum size

Merge pull request #11578 from rgacogne/ddist-relayed-msg

dnsdist: Include the address of the backend in 'relayed to' messages

dnsdist: Support link-level address retrieval for IPv6 as well

The existing implementation used /proc/net/arp which only contains
MAC addresses for IPv4 addresses. Switching to netlink provides
support for IPv6 addresses as well, and roughly the same performance
(it seems a tiny bit faster but the difference is in the error margin).

Avoid a race in test code between processing the first query and the priming code.

It can happen the root updating houskeeping code has not run yet when
processing the first query.  In that case, we do not have root DNSKEY
records in cache.  When validating the answer, rec will then go out and
fetch those, so no real issue, apart form the fact that it interferes
with our expected outgoing protobuf messages.

Same make sure the root DNSKEY records are cached before doing the actual
test query.

dnsdist: fix wrong parameter type in document

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

dnsdist: add parameter in blockRange

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

Use IPNetwork instead of IPAddress

Test Ed448 as part of the generic mechanism for testing signers

This also automatically tests a few things that weren't being tested before.

Merge pull request #11514 from rgacogne/ddist-api-noauth

dnsdist: Add an option for unauthenticated access to the API

Avoid using magic numbers for algorithms

Clean-up signer params

test-signers: Use boost fixtures and print info about signers/tests

This also prints out information about what signers are available and what tests are
getting enabled. Use the `--log_level=message` argument to the boost `testrunner` to see
that information.

Lint fixes (consts, boolean conditions)

dnsdist: Include the address of the backend in 'relayed to' messages

Not all backends have a name.

dnsdist: Scan the UDP buckets only when we have outstanding queries

Merge pull request #11574 from rgacogne/ddist-1.7.1-changelog-secpoll

dnsdist: ChangeLog and secpoll update for dnsdist 1.7.1

dnsdist: Hopefully we will release 1.7.1 today

Formatting

Merge pull request #11493 from fredmorcos/11325-eddsa-import-export-pem

Support PEM import and export of EdDSA keys

Merge pull request #11562 from fredmorcos/find-libdecaf-headers

Try harder to find libdecaf headers

Merge pull request #11573 from rgacogne/ddist-no-downstream

dnsdist: Better log message when no downstream server are available

Update document

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

rename BPFFilter:unblockRange() to BPFFilter:allowRange()

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

add BPFFilter::getRangeStats()
add BPF_MAP_TYPE_LPM_TRIE support in getStats()

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

Revert "add BPF_MAP_TYPE_LPM_TRIE support in getStats()"

This reverts commit a8b0100a28b4367d8038726382e007f727d23ad1.

Update bpf_create_map type declare

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>
Co-authored-by: Remi Gacogne <github@coredump.fr>

change name

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>
Co-authored-by: Remi Gacogne <github@coredump.fr>

change code style

Co-authored-by: Remi Gacogne <github@coredump.fr>
Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

Update pdns/dnsdistdist/docs/reference/ebpf.rst

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

dnsdist: ChangeLog and secpoll update for dnsdist 1.7.1

Merge pull request #11570 from omoerbeek/rec-codeql-j4

rec: Reduce make -j parameter from 8 to 4 for CodeQL action, as dnsdist does.

dnsdist: Better log message when no downstream server are available

Reduce make -j parameter from 8 to 4, as dnsdist does.

This might fix the occasional build issues with the CodeQL GH Action.

dnsdist: Use the right function name (setVerbose) in the documentation

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

dnsdist: Add setVerbose() to switch the verbose mode at runtime

And also from the configuration, without having to edit the arguments
passed to the program.

add BPF_MAP_TYPE_LPM_TRIE support in getStats()

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>
Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

update document

use BPF_MAP_TYPE_LPM_TRIE map in XDP program to block a range

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

add blockRange and unblockRange

Support create bpf map with BPF_MAP_TYPE_LPM_TRIE type

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

Merge pull request #11565 from rgacogne/ddist-fix-ebpf-qnames

dnsdist: Fix wrong eBPF values (qtype, counter) being inserted for qnames

Merge pull request #11544 from rgacogne/ddist-bpf-block-action

dnsdist: Properly use eBPF when the DynBlock is not set

Merge pull request #11547 from rgacogne/ddist-add-address-accessor-to-frontends

dnsdist: Add a 'getAddressAndPort()' method to DOHFrontend and TLSFrontend objects

Merge pull request #11545 from rgacogne/ddist-fix-backend-proto

dnsdist: Use the correct outgoing protocol in our ring buffers

Merge pull request #11561 from omoerbeek/dnsdist-include-openbsd

dnsdist: include <unordered_map>

dnsdist: Fix wrong eBPF values (qtype, counter) being inserted for qnames

Headers in /usr/local/include/decaf are now automatically detected

Add autoconf checks for libdecaf headers

This adds a few things:
- Passing a user-defined header location through LIBDECAF_CFLAGS=-I...
- Otherwise, check the following directories for decaf.hxx:
  - /usr/include
  - /usr/include/decaf
  - Add whichever one is picked to LIBDECAF_CFLAGS
- Then AC_CHECK_HEADER decaf.hxx using CXXFLAGS with LIBDECAF_CFLAGS

The reason for this change is that some package systems have decaf.hxx
under /usr/include and others have it under /usr/include/decaf.

Whitespace

Reimpl sodium ed25519 sign & verify using the simpler detached mode

Libsodium expects the pub key concatenated to the secret key buffer

Print data in base64 for better debugging experience when tests fail

Formatting

Cleanup

Libdecaf impl of Ed448 PEM import and export

Libdecaf impl of Ed25519 PEM import and export

Guard libsodium PEM methods with libcrypto

These methods depend on OpenSSL for PEM import and export.

Sodium impl of EDDSA PEM import

OpenSSL impl of EDDSA PEM import

Test data for EDDSA PEM import and export

Sodium impl of EDDSA PEM export

There's no real way to serialize keys in Sodium, so we fall-back to
using OpenSSL which is available in our Sodium-based builds.

Cleanup: Unused local storage

OpenSSL impl of EDDSA PEM export

Whitespace

Merge pull request #11564 from Habbie/circleci-docs-no-texlive

circleci: revert docs build to specific debian image, instead of the debian:testing underlying the texlive image

revert docs build to specific debian image, instead of the debian:testing underlying the texlive image

Include <unordered_map>

On OpenBSD you don't get std::unordered_map for free

Merge pull request #11541 from omoerbeek/rec-fix-dot-probe

Fix DoT port, as noted by ph1 on IRC

Decide to use Dot earlier.

To keep the nsspeed table good, we need to decide to use DoT earlier.
Now the lookup and updats of the speed table occur in a proper way
(using the port that is actually used for the connetion) and when
we switch from/to DoT, the old nsspeeds are cleared by the already
existing code.

Merge pull request #11546 from rgacogne/ddist-fix-0-sized-cache

dnsdist: Raise the number of entries in a packet cache to at least 1

Merge pull request #11539 from rgacogne/rec-fix-dnssec-counters-documentation

rec: DNSSEC counters track responses sent, not actual validations performed

Simplify code

Removes a superfluous `if` and folds both branches which had identical behavior.

Merge pull request #11535 from Y7n05h/MergeParams

Merge multiple parameters in newBPFFilter

Update pdns/dnsdistdist/docs/reference/ebpf.rst

spell-check: Allow UNPRIV from BPF_UNPRIV_DEFAULT_OFF

dnsdist: Document external eBPF programs and maps

dnsdist: Better documentation of the eBPF requirements

dnsdist: Fix formatting

dnsdist: Add a 'getAddressAndPort()' method to DOHFrontend and TLSFrontend objects

dnsdist: Raise the number of entries in a packet cache to at least 1

And make sure that we cannot create such an object.

dnsdist: Use the correct outgoing protocol in our ring buffers

dnsdist: Properly use eBPF when the DynBlock is not set

When the DynBlock rule does not set a specific action we use the
default one, set with `setDynBlocksAction()`, so we should follow
the same logic when determining whether to insert an eBPF block.

fix d_type in MapConfiguration

rec: Fix DNSSEC counters description in web/prometheus as well

fix errer key in access convertParamsToConfig

Signed-off-by: Y7n05h <Y7n05h@protonmail.com>

dnsdist: Prevent leaking the console's socket descriptors

I don't see any case where that could happen but better wrap the
socket descriptors in FDWrapper objects so that we cannot forget to
close them if an exception is raised, for example.

dnsdist: Log when a console message exceeds the maximum size

Otherwise one cannot know whether the server has closed the connection
or sent a too large message.

Missing newline in dump output, noted by ph1

Fix port, as noted by ph1 on IRC

Without this, probed DoT actually becomes regular TCP.

Update pdns/dnsdistdist/docs/reference/ebpf.rst

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

rec: DNSSEC counters track responses sent, not actual validations performed

Since 4.1 these counters are updated for every response sent, even if the DNSSEC
status was fetched from the records cache and did not involve any actual
validation.

Update pdns/dnsdistdist/docs/reference/ebpf.rst

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

Merge pull request #11381 from omoerbeek/rec-dynamic-root-refresh-period

rec: Adaptive root refresh interval, normally at 80% of max-cache-ttl

Update document

Simplify the parameters of newBPFFilter

Merge pull request #11518 from fredmorcos/add-libdecaf-to-github-actions

Add libdecaf (auth) tests to Github Actions

Refer to new root hint handling info from settings docs