Fix most comments on the basic-database docs

Update docs/guides/basic-database.rst

Co-Authored-By: Pieter Lexis <pieter@plexis.eu>

Update docs/guides/basic-database.rst

Co-Authored-By: Pieter Lexis <pieter@plexis.eu>

Update docs/guides/basic-database.rst

Co-Authored-By: Pieter Lexis <pieter@plexis.eu>

Update docs/guides/basic-database.rst

Co-Authored-By: Pieter Lexis <pieter@plexis.eu>

a word

auth: change installation example from MySQL to SQLite

auth doc nit

Merge pull request #8617 from pieterlexis/drop-el6

Drop EL 6, Debian Stretch and Ubuntu Xenial support in the builder

Merge pull request #9872 from mind04/pdns-direct-dnskey

auth: fix direct-dnskey in AXFR-out

Merge pull request #9520 from mind04/pdns-detect

auth: detect possible metadata cache pollution

Merge pull request #9921 from rgacogne/ddist-sni-resumption

dnsdist: Fix SNI on resumed sessions by acknowledging the name sent by the client

Merge pull request #9924 from omoerbeek/rec-fix-unittest-time

rec: Force the sig generation for test_dnssec_bogus_bad_sig to use the right time

Force the sig generation for test_dnssec_bogus_bad_sig to use the
right time to avoid signature not yet valid issues on slow machines.

Merge pull request #9907 from Habbie/auth-doc-lanf

auth docs: correct local-address-nonexist-fail default

remove stretch leftover

reinstate touch /var/lib/rpm/*

remove useless directory change

improve language

reinstate missing .sql files

dnsdist: do not try to ship empty contrib/ dir

Add "old OS" builders fro EL7 and Ubuntu 18.04

Stop adding locally built stuff to docker

Drop Debian Stretch Package builds

Remove dropped OS's from generate-repo-files

Drop non-LTS Ubuntu

Drop Ubuntu Xenial and Debian Jessie from packages

pkgs: drop EL6

auth: drop EL6 pkg support

rec: drop EL6 pkg support

dnsdist: drop EL6 pkg support

auth: some cleanup in the AXFR-out code

auth: test direct-dnskey in AXFR-out

auth: fix direct-dnskey in AXFR-out

auth: detect possible metadata cache pollution

Merge pull request #9836 from Habbie/circleci-extend-2

circleci: extend coverage some more and disable travis

dnsdist: Add a unit test for SNI routing after session resumption

dnsdist: Fix SNI on resumed sessions by acknowledging the name sent by the client

Otherwise `SSL_get_servername()` only returns true when the session
has been freshly established, and will return `nullptr` when it is
resumed.

stop using travis

configure: set --enable-option-checking=fatal

geoipbackend: use const_iterator

==8618==ERROR: AddressSanitizer: stack-use-after-scope on address 0x7ffd8c5a1420 at pc 0x55bee0a5d899 bp 0x7ffd8c5a0710 sp 0x7ffd8c5a0708
READ of size 1 at 0x7ffd8c5a1420 thread T0
    #0 0x55bee0a5d898 in YAML::Node::begin() const /usr/include/yaml-cpp/node/impl.h:284:8
    #1 0x55bee0a3f2f0 in GeoIPBackend::initialize() /opt/project/modules/geoipbackend/geoipbackend.cc:163:27
    #2 0x55bee0a3d0d3 in GeoIPBackend::GeoIPBackend(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) /opt/project/modules/geoipbackend/geoipbackend.cc:86:5
    #3 0x55bee0a784eb in GeoIPFactory::make(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) /opt/project/modules/geoipbackend/geoipbackend.cc:1008:16
    #4 0x55bedff6cb18 in BackendMakerClass::all(bool) /opt/project/pdns/dnsbackend.cc:200:46
    #5 0x55bee07d3abe in UeberBackend::UeberBackend(std::__cxx11::basic_string<char, std::char_traits<char>, std::allocator<char> > const&) /opt/project/pdns/ueberbackend.cc:479:28
    #6 0x55bee060579c in PacketHandler::PacketHandler() /opt/project/pdns/packethandler.cc:65:32
    #7 0x55bedfee1748 in std::unique_ptr<PacketHandler, std::default_delete<PacketHandler> > make_unique<PacketHandler>() /opt/project/pdns/./misc.hh:571:35
    #8 0x55bee079376d in TCPNameserver::go() /opt/project/pdns/tcpreceiver.cc:85:9
    #9 0x55bedfecf7b7 in mainthread() /opt/project/pdns/common_startup.cc:656:7
    #10 0x55bee0655f67 in main /opt/project/pdns/receiver.cc:654:5
    #11 0x7f374b4b809a in __libc_start_main (/lib/x86_64-linux-gnu/libc.so.6+0x2409a)
    #12 0x55bedfc9a989 in _start (/opt/pdns-auth/sbin/pdns_server+0x2736989)

Address 0x7ffd8c5a1420 is located in stack of thread T0 at offset 3040 in frame
    #0 0x55bee0a3d5ef in GeoIPBackend::initialize() /opt/project/modules/geoipbackend/geoipbackend.cc:115

lmdbbackend: avoid unaligned uint32_t reads by using memcpy

lmdbbackend.cc:942:27: runtime error: load of misaligned address 0x604000b827aa for type 'uint32_t' (aka 'unsigned int'), which requires 4 byte alignment

remotebackend: use 0 for denoting unknown serial which is uint32_t

remotebackend.cc:610:51: runtime error: -1 is outside the range of representable values of type 'unsigned int'
SUMMARY: UndefinedBehaviorSanitizer: undefined-behavior remotebackend.cc:610:51 in

add symbolizer to auth tests

disable asan leak sanitizer

auth: asan+ubsan; rec: ubsan

install LLVM symbolizer

Two bulk test steps were named identically, rename

Test on HAVE_FIBER_SANITIZER for stack enlargement, __SANITIZE_ADDRESS__
is not defined everywhere.

Reomve tmp debug code and fix two nits in the bulk script

Crash happens earlier than I thought

cat recursor.log before trying to kill rec, if the kill fails
(because rec died) we at least have the log.

Revert "disable recursor asan"

This reverts commit 2799109d800a368a4aebdf1fb1841602ad4ff3b4.

disable recursor asan

circleci: extend coverage some more

* switch to clang-8
* test Recursor SNMP, DNSTAP, Protobuf
* enable asan for Recursor

Merge pull request #9917 from rgacogne/rec-pb-zero-length

rec: Set the Protocol Buffer 'inBytes' field for zero-length messages

rec: Set the Protocol Buffer 'inBytes' field for zero-length messages

Th existing Protocol Buffer code used to do that, so existing receivers
might expect that field to be present.

Merge pull request #9914 from Habbie/pthread-symbols

autoconf: correctly detect pthread symbols

Merge pull request #9911 from omoerbeek/dnsname-macos-openbsd

MacOS (both Intel and arm64) and OpenBSD have no issues any more with boost::containter::string

autoconf: correctly detect pthread symbols

Merge pull request #9843 from rgacogne/dnsdist-protozero

Use protozero for Protocol Buffer operations in dnsdist, and dnstap/outgoing for the rec

Merge pull request #9782 from rgacogne/ddist-dynblocks-metrics-ebpf

dnsdist: Use an eBPF filter for Dynamic blocks when available

Merge pull request #9883 from rgacogne/rec-ds-cname-cache

rec: Lookup DS entries before CNAME entries

Merge pull request #9906 from Habbie/rec-doc-la

rec docs: correct local-address default

Merge pull request #9826 from Habbie/edit-zone-zpt-exc

pdnsutil edit-zone: do not exit on ZoneParser exception

Merge pull request #9910 from nzlosh/zone_xfr_rewording

Reword zone xfr log message to remove ordinal number naming errors.

MacOS (both Intel and arm64) and OpenBSD have no issues any more
with boost::containter::string

Merge pull request #9909 from omoerbeek/macos-sanitizer

Enable address sanitizer on MacOS

Merge pull request #9908 from omoerbeek/auth-build-testrunner

auth: If unit tests are enabled, build testrunner from the main target

If unit tests are enabled, build testrunner from the main dir

auth docs: correct local-address-nonexist-fail default

Reword zone xfr log message to remove ordinal number naming errors.

rec docs: correct local-address default

Enable address sanitizer on MacOS

Merge pull request #9900 from rgacogne/ddist-rmserver-lock

dnsdist: Fix a hang when removing a server with more than one socket

protozero: Declares names instead of using numerical values for fields

dnstap: Declares names instead of using numerical values for fields

rec: Apply formatting again

rec: Apply formatting, as 'already formatted to perfection' is relative

We still need the protobuf compiler in a few places for our tests

rec: Apply formatting

RemoteLogger: Fix astring-plus-int issue

Remove left-over protobuf from build files, CI, documentation

dnsdist: Use the generic dnstap m4 again

rec: Use protozero for outgoing queries Protocol Buffer operations

RemoteLogger: Use the simpler lock_guard instead of unique_lock

dnspcap2protobuf: Use protozero for Protocol Buffer operations

dnsdist: Use protozero for Protocol Buffer operations

rec: Remove unneeded 'static'

dnsdist: Remove unused fields when eBPF is not available

Merge pull request #9857 from rgacogne/crypto-engine-unique-ptr

Return a unique_ptr instead of a shared one for DNSCryptoKeyEngine

Merge pull request #9905 from omoerbeek/copyright-2021

It's 2021

It's 2021

It's 2021

Merge pull request #9904 from yantarou/dnsdist_typo

dnsdist: Fix typo in tuning documentation

dnsdist: Fix typo in tuning documentation

Merge pull request #9894 from omoerbeek/zap-incbin

Remove incbin from ext and references to it

dnsdist: Fix a hang when removing a server with more than one socket

There was a lock starvation issue when removing a server with more
than one socket in use (`sockets` greater than 1 on the corresponding
`newServer` directive), because the mutex protecting the sockets array
would never be released long enough by the responder thread to allow
the thread stopping the server to acquire it.
This commit fixes that by marking the server as stopped right away,
before acquiring the lock, and also making sure that the responder
thread is woken up regularly (every second, even without any query
to process) and that it checks whether the server has been stopped
just after that.

The issue was introduced in be55a20ce9bb7140071279d70bcb460f1f2b7b7d,
and backported to 1.5.1 in f0d48318cce0dd80ae73c529362bdb2921d8c5c9.

Merge pull request #9841 from rgacogne/pb-logger-short-read

ProtobufLogger: Better handling of short-reads, errors

Merge pull request #9895 from mortenstevens/patch-4

Fix missing #include for gcc-11