Beter naming of closure

Modify journal availability test.

To both test for NOTIFY_SOCKET (to see if we're runnign underst systemd)
and availbility of the journal socket (to see if systemd-journal is
active)

Use systemd-journal for structured logging if it is available

Merge pull request #11642 from omoerbeek/structured-logging2

rec: Structured logging for housekeeping

Merge pull request #11646 from omoerbeek/rec-prep-4.7.0

Rec: prep 4.7.0

Secpoll and changelog

Merge pull request #11650 from mnordhoff/patch-11

auth: Clarify 4.3.0 upgrade note

auth: Clarify 4.3.0 upgrade note

It was a little unclear that it doesn't apply to something like 4.2.0-beta2.

Merge pull request #11644 from omoerbeek/rec-deprecate-xpf-settings

rec: Deprecation warning for XPF settings.

Deprecation warning for XPF settings.

Plus a tweak in the associated structured logging.

Merge pull request #11643 from rgacogne/ddist-mac-large-buffer

dnsdist: Accept large buffers in getMACAddress()

dnsdist: Accept large buffers in getMACAddress()

Use a name for `const std::shared_ptr<Log::Logger>&`, call it `Logr::log_t`

Move housekeeping and associated code to strcutured logging

EOL update for 4.7.0

Merge pull request #11591 from rgacogne/ddist-mac-netlink

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

Merge pull request #11639 from Habbie/packages-sdig-dot

auth packaging: add DoT support to sdig

Merge pull request #11631 from omoerbeek/structured-logging1

rec: use structured logging for startup and config reporting

auth packaging: add DoT support to sdig

Apply suggestions from code review

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

dnsdist: Proper handling of scope IDs for MAC addresses

Thanks to Peter for the correct check!

dnsdist: Document the return value of getMACAddress()

As suggested by Otto.

dnsdist: Better descripton of getMACAddress() in the documentation

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

dnsdist: Better descripton of getMACAddress() in the console help

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>

Merge pull request #11637 from Habbie/dnsdist-getverbose

dnsdist: add getVerbose() function

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

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

dnsdist: add getVerbose() function

Merge pull request #11606 from rgacogne/ddist-lua-binding-dns-payload

dnsdist: Add Lua bindings to access the DNS payload as a string

dnsdist: Fix typo reported by Peter (thanks!)

Co-authored-by: Peter van Dijk <peter.van.dijk@powerdns.com>

Merge pull request #11609 from omoerbeek/rec-api-allow-empty

Rec: api allow empty allow-from

More conversions, startup should be done now

Try to be smart: if to_string() or toLogString() is there, use it.

More smart logging of iterables and inline a few methods

Some more conversions to structured logging

Tweaks: use name for prio, change some prio's, use setter for arguments

Merge pull request #11627 from omoerbeek/rec-rpz-dump-docs

Clarify naming of RPZ zones loaded by rpzFile.

Tsan complained about strftime(), it manulipates locale and/or tz info.
So guard if needed.

Formatting

Convert startup and some of config to use structured logging

Tweaks wrt explicit priority.

It should be made less easy to make mistakes here.

Reorg to be able to use structured logging in shared files.

Using arguments.cc as a test-case.

Merge pull request #11626 from omoerbeek/zonemd-zap-binary-function

Zap binary function, not needed and deprecated in c++11, removed in c++17

Clarify naming of RPZ zones loaded by rpzFile.

Also
- don't include local refs, they end up wrong in the man page. Replace them by URLs.
- add links to general docs on both maual pages.

Zap binary function, not needed and deprecated in c++11, removed in c++17

Merge pull request #11620 from Y7n05h/fix11619

 dnsdist: remove implicit type conversion

Merge pull request #11621 from rgacogne/ddist-fix-proxyprotocol-tc-doh+ddist-fix-proxyprotocol-tc-doh

dnsdist: Fix a crash on a invalid protocol in DoH forwarded-for header

dnsdist: Test for an exception raised when adding proxy protocol payload to a DoH query

dnsdist: Fix a crash on a invalid protocol in DoH forwarded-for header

(cherry picked from commit f84fbd58b150fe6b69a7af27e23502f58f68eee5)

Merge pull request #11604 from rgacogne/ddist-fix-proxyprotocol-tc-doh

dnsdist: Fix invalid proxy protocol payload on a DoH TC to TCP retry

Merge pull request #11614 from xorander00/docs-loglevel-fix

Changed loglevel wording to be more accurate for both recursor and au…

Merge pull request #11607 from rgacogne/rec-doc-missing-dq-attributes

rec: Document a few missing DNSQuestion attributes

 dnsdist: remove implicit type conversion (fix: #11619 )

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

Changed loglevel wording to be more accurate for both recursor and authoritative.

Document meaning of empty allow-from

Merge pull request #11605 from omoerbeek/rec-prep-4.7.0-rc1

Prep for rec-4.7.0-rc1

rec: Document a few missing DNSQuestion attributes

dnsdist: Add Lua bindings to access the DNS payload as a string

Merge pull request #11602 from rgacogne/ddist-missing-prometheus-descriptions

dnsdist: Add missing descriptions for prometheus metrics

Merge pull request #11603 from rgacogne/ddist-upgrade-guide-formatting

dnsdist: Fix formatting issues in the upgrade guide

Add tests for empty allow-from and allow-notify-from case

Fix API issue when asking config values for allow-from or allow-notify-from

Prep for rec-4.7.0-rc1

dnsdist: Fix invalid proxy protocol payload on a DoH TC to TCP retry

dnsdist forwards incoming DoH queries to its backend over UDP, and
retry over TCP if the response is truncated (TC=1).
When the proxy protocol is used between dnsdist and its backend, the
second query, over TCP, needs to take into account that the proxy
protocol payload has already been handled. This was not properly done
in that exact case because the proxy protocol payload length was not
propagated to the code handling the TCP communication, leading to
the query ID being edited at the wrong offset in the packet and thus
to an invalid proxy protocol payload.

Merge pull request #11592 from omoerbeek/auth-test-incomplete-xfr

Auth: fix race in incomplete xfr test

dnsdist: Fix formatting issues in the upgrade guide

dnsdist: Add missing descriptions for prometheus metrics

Merge pull request #11567 from rgacogne/ddist-set-verbose

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

dnsdist: Add a 'getMACAdress(ip)' Lua binding

dnsdist: Hopefully prevent a CodeQL warning in the netlink code

Merge pull request #11598 from omoerbeek/rec-shared-throttlemap

Rec: shared throttle map

Merge pull request #11599 from omoerbeek/auth-coverity-1488422

auth: Tweak for Coverity 1488422

Upgrade guide entries and small nit

Use wrappers, as suggested by rgacogne@ in his review.

Merge pull request #11577 from rgacogne/ddist-skip-scan-no-outstanding

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

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

dnsdist: Remove dead code in the console

Tweak for Coverity 1488422.

The spot where the warning happens changed, plus appease clang-tidy.

Throttle table cleanup task is now handler specific.

Also use QType consistently.

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

Use emplace instead of insert

Shared throttle map

So the race is between setting the new serial to return and the message coming in
Make sure we set the new serial asap

Add debug print lines to the new test_XFRIncomplete that fails occasionally on GH Actions.

Using a slow loaded VM I could reproduce these failures once in a while.
Adding the print statements made the failures go away. Maybe some form
of synchronization is going on with the extra debug I/O?  Committing this
to see what happens on GH Actions.

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