Merge pull request #9070 from rgacogne/boost-173

Boost 1.73 moved boost::bind placeholders to the placeholders namespace

Merge pull request #9079 from omoerbeek/coverity-1428667

Avoid throwing an exception in Logger::log().

Merge pull request #9080 from omoerbeek/coverity-1401969-sqllite

auth: better (actual) fix for mem leak in SSQLite3::execute()

Merge pull request #9076 from rgacogne/clang-10-warnings

Fix useless copies in loop reported by clang++ 10

Merge pull request #9084 from pieterlexis/dnsdist_latency_prometheus_help

dnsdist: add the unit to the help for latency buckets

sqlite3_exec does not return SQLITE_DONE or SQLITE_ROW plus one forgottten throw
conversion

Apply suggestions from code review

Co-authored-by: Remi Gacogne <rgacogne+github@valombre.net>

Merge pull request #9078 from omoerbeek/coverity-1419400

NetmaskTree: do not test node for null, the loop guarantees node is not null.

Merge pull request #9087 from kpfleming/patch-1

Correct doc for ecs-add-for default

Correct doc for ecs-add-for default

The default setting for `ecs-add-for` includes "::/0", not "::".

dnsdist: add the unit to the help for latency buckets

Better version that also plug the leaks for errors != the ones tested for.

Better (actual) fix for leak reported by Coverity.

Always free errmsg; use two diffferent string vars to avoid shadowing.
Coverity 1401969.

Avoid throwing an exception in Logger::log().

Throwing an exception in the logger, almost cetainly will cause
trouble. Coverity 1428667 shows this can happen in
LdapBackend::~LdapBackend().

Do not test node for null, the loop guarantees node is not null.

Found by Coverity (1419400), which concludes that if node could be
null, the following node->node.first.getBits() would deref a nullptr.

auth: Remove unused variable in the LDAP backend

auth: Fix 'loop variable [...] creates a copy' warnings

Reported by clang++ 10.

rec: Fix 'loop variable [...] creates a copy' warnings

Reported by clang++ 10.

Merge pull request #9067 from rgacogne/wrap-pthread-objects

Wrap pthread objects

Make the constructors taking a pthread_rwlock_t * private.

This is to avoid re-introducing code using the unwrapped pthread_rwlock_t's.
While there, reorganize the classes to make the order more natural.

Replace boost's placeholders with the ones from the std namespace

Boost 1.73 moved boost::bind placeholders to the placeholders
namespace which did not exist before 1.60, let's switch to
std::bind and std::placeholders instead of doing per-version handling.

Lock: Remove the unused g_singleThreaded variable

Remove the now unused Lock class wrapping pthread_mutex_t objects

auth: Wrap a few more missed pthread_ objects

auth: Wrap pthread_ objects

rec: Wrap pthread_ objects

dnsdist: Wrap pthread_ objects

Merge pull request #9069 from omoerbeek/test-destroy-with-waiters

Avoid "pthread_rwlock_destroy on rwlock with waiters!" on OpenBSD

Merge pull request #9068 from rgacogne/ddist-http2-counters

dnsdist: Fix duplicated HTTP/1 counter in 'showDOHFrontends()'

Avoid "pthread_rwlock_destroy on rwlock with waiters!" on OpenBSD

dnsdist: Fix duplicated HTTP/1 counter in 'showDOHFrontends()'

Merge pull request #9057 from rgacogne/ddist-socket-removal

dnsdist: Gracefully handle a failure to remove FD on (re)-connection

Merge pull request #9056 from omoerbeek/decl-warnings

Add -Wmissing-declarations -Wredundant-decls

Merge pull request #9060 from Habbie/bind-extstatus-offbyone

BIND-DOMAIN-EXTENDED-STATUS: don't look for a domain called BIND-DOMAIN-EXTENDED-STATUS

BIND-DOMAIN-EXTENDED-STATUS: don't look for a domain called BIND-DOMAIN-EXTENDED-STATUS

Merge pull request #9059 from omoerbeek/dnsdist-warnings

dnsdist: a few warnings fixed, reported by clang on OpenBSD

Fix warnings about dup/missing prototypes

A few warnings, reported by clang on OpenBSD

dnsdist: Gracefully handle a failure to remove FD on (re)-connection

When more than one socket per backend is used, we might not have
added all the socket descriptors to the multiplexer and that's fine.

Merge pull request #9044 from rgacogne/auth-never-read-clang

auth: Fix warnings reported by clang++ 10

Merge pull request #9053 from rgacogne/devpoll-no-naked-pointer

Get rid of a naked pointer in the /dev/poll event multiplexer

Merge pull request #9054 from rgacogne/ddist-more-flags

dnsdist: Build with -Wmissing-declarations -Wredundant-decls

Add -Wmissing-declarations -Wredundant-decls

Merge pull request #9016 from omoerbeek/random-engine

random engine to allow for future ++17 compatibility

Merge pull request #9050 from aerique/feature/add-rec-44-to-repo-script

Add `rec-44` to repo test script.

Get rid of a naked pointer in the /dev/poll event multiplexer

dnsdist: Build with -Wmissing-declarations -Wredundant-decls

dnsdist: Fix warnings about dup/missing prototypes

Including a wrong FFI prototype.

Merge pull request #9052 from cyclops1982/docs

Small doc changes

Add notes on logging ability in dnsdist

Merge pull request #9046 from omoerbeek/rec-prep-4.4.0-alpha1

Prep rec-4.4.0-alpha1

Add `rec-44` to repo test script.

In dnsdist, (which is not a dns_random consumer), use sdt::default_random_engine
for shuffle.

Avoid self-assignment of DNSName, some version of boost::container::string
do not like that and throw an assertion.

Utilize std::shuffle and introduce pdns::dns_random_engine

std::random_shuffle is deprecated and removed starting from C++17, so we
might better use std::shuffle which uses URBG:
https://en.cppreference.com/w/cpp/named_req/UniformRandomBitGenerator
under the hood for a better random generation.

Prep rec-4.4.0-alpha1

Merge pull request #8910 from omoerbeek/rec-routing-tag

Rec: routing tag

Test tweaks

Simpify 'variable' condition: it boils down to having a reply from
the auth including an ednsmask, routingTag does not play a role.

Process rgacogne's comments wrt entryMatches().

This simplifies the code, but has the silent assumption that
entryMatches() only gets fed matching routingTags (if present).
The getEntries() code does guarrantee this so we should be safe.

Basic routing tag test

Routingtag is now handled by gettag() and not part of dq anymore.

ffi interface to be done. Includes attempt at documenting the return
value of gettag() a bit better.

Process comments from rcagogne; mostly better unit tests.

entryMatches() logic not yet adapted.

Introduce routingTag, settable from lua and passed to SyncRes if set.

SyncRes passes it on to the record cache methods that do not use
it yet.

auth: Make sure we don't divide by zero in Lua's pickwhashed()

auth: Prevent reading uninitialized memory in Lua's getLatLon()

Merge pull request #9043 from omoerbeek/github-do-not-use-depracted-api

Do not use github deprecated API for changelog script

auth: Fix 'value is never actually read' warnings from clang++ 10

Do not use github deprecated API for changelog script

See https://developer.github.com/changes/2020-02-10-deprecating-auth-through-query-param/

Merge pull request #9042 from rgacogne/ddist-clang-10

dnsdist: Avoid copies in for loops

Merge pull request #9034 from aerique/feature/dnsdist-support-for-repo-script

Add support for dnsdist-15 to repo script.

Merge pull request #9041 from rgacogne/ddist-changelog-year

dnsdist: Fix the year (2010 -> 2020) of the latest changelog entries

Add Remi's usage example.

dnsdist: Avoid copies in for loops

Reported by clang's '-Wrange-loop-construct' since version 10.

Merge pull request #9024 from Habbie/mysql-errors

auth smysql: mimic error message format from mysql tooling

dnsdist: Fix the year (2010 -> 2020) of the latest changelog entries

Small note around Spoof and TTL

Small note on ACL editing

Merge pull request #9031 from rgacogne/ports-event-fix

Fix compilation of the ports event multiplexer

Merge pull request #9033 from pieterlexis/el8-mariadb-dep

EL8 pkgs: Build mysql backend against mariadb-connector-c-devel

Fix `dnsdist-15` on CentOS 8.

Generate proper Docker files for `dnsdist-15` on CentOS 8.

Merge pull request #9028 from rgacogne/ddist-150-rc1

dnsdist: Update ChangeLog and secpoll for 1.5.0-rc1

Add support for dnsdist-15 to repo script.

EL8 pkgs: Build mysql backend against mariadb-connector-c-devel

This matches EPEL downstream.

Closes: #9026

dnsdist: Document the cert/key permissions in DoT and DoH guides

dnsdist: Document that permissions on external files need to be fixed

Fix compilation of test-dns_random_hh.cc w/ getrandom AND arc4random

Fix compilation of the ports event multiplexer

regression-tests: update docker run postgres line

dnsdist: Remove 8985 from the changelog, it did not impact dnsdist

dnsdist: Update ChangeLog and secpoll for 1.5.0-rc1

Merge pull request #9027 from omoerbeek/rec-internals-qname-min

rec: Tell a bit about qname minimization

Apply suggestions from code review

Typos

Co-Authored-By: Remi Gacogne <rgacogne+github@valombre.net>

Tell a bit about qname minimization

auth smysql: mimic error message format from mysql tooling

Merge pull request #9017 from Habbie/prefixe-s

rec: fix typo (prefixe -> prefixes)

Merge pull request #9005 from Habbie/auth-4.2.2-docs

changelog&secpoll for auth 4.2.2

fix typo

Merge pull request #8853 from jsoref/spelling

Spelling

spelling: [API] deserialize

spelling: across

spelling: acted

spelling: added

spelling: address

spelling: advantageous

spelling: against

spelling: algorithms

spelling: answers

spelling: authoritative

spelling: availability

spelling: boundaries

spelling: boundary

spelling: cannot

spelling: canonical

spelling: capability

spelling: choices

spelling: combination

spelling: comboring

spelling: comparison

spelling: constraints

spelling: constructor

spelling: contain

spelling: convenience

spelling: current

spelling: delegation

spelling: dependency

spelling: distribution

spelling: doesnotexist

spelling: dropping

spelling: everything

spelling: example

spelling: existence

spelling: explicitly

spelling: extremely

spelling: from

spelling: housekeeping

spelling: idonotexist

spelling: ipfilter

spelling: issuing

spelling: logging

spelling: message

spelling: method

spelling: modified

spelling: must

spelling: name

spelling: negative

spelling: obtaining

spelling: occurred

spelling: omitted

spelling: overridden

spelling: password

spelling: policy

spelling: positives

spelling: possibility

spelling: preresolve

spelling: probabilistic

spelling: prohibitively

spelling: protocol

spelling: provider

spelling: public

spelling: repeatedly

spelling: repositories

spelling: responding

spelling: response

spelling: several

spelling: should

spelling: sizing

spelling: supplied

spelling: support

spelling: synthesized

spelling: the

spelling: transaction

spelling: usually

spelling: visitor