Merge pull request #11596 from rgacogne/ddist-console-redundant-code

dnsdist: Remove dead code in the console

dnsdist: Remove dead code in the console

Reported by Coverity as CID 378254.

Merge pull request #11590 from fredmorcos/11325-rsa-pem-import-export

RSA PEM import/export

Merge pull request #11589 from omoerbeek/coverity-20220429

auth: Set of Coverity fixes

SignerParams PEM contents don't need to be optional anymore

We now always test PEM import/export and we expect any new signers to
implement the necessary methods for that.

RSA PEM test data

Formatting

RSA PEM import

RSA PEM export

Merge pull request #11582 from fredmorcos/refactor-test-signers

Refactor signers tests

Use better get(); writen2(); release() idiom

Add missing include

Fix signer tests for platforms (i.e. OpenSSL forks) without Ed25519

Don't leak on exception and another case that needs an annotation

Coverity 1030062 Uninitialized scalar field

Coverity 1401639 Division or modulo by float zero: avoid div by zero

Coverity 1469729 Explicit null dereferenced: bail is SOA isn't set

Coverity 1469730 Explicit null dereferenced: use a default sleep time if sr isn't set

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

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>

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.

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

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.

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

Merge pull request #11535 from Y7n05h/MergeParams

Merge multiple parameters in newBPFFilter

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

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

Merge pull request #11527 from omoerbeek/prep-rec-4.7.0-beta1

rec: Prep for rec-4.7.0-beta1

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

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

Update pdns/dnsdist-lua-bindings.cc

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

Document more details of root-hints handling

Merge pull request #11536 from omoerbeek/rec-nitems

Fix Coverity 1487923 Out-of-bounds read (wrong use of sizeof)