Merge pull request #9160 from Habbie/spelling-only-docs

limit spell checking to docs

Merge pull request #9166 from cmouse/patch-1590648655

opensslsigners: Add missing 'static' keyword

Merge pull request #9162 from jsoref/clarify-docs

Clarify docs

opensslsigners: Add missing 'static' keyword

openssl_pthreads_locking_callback and openssl_pthreads_id_callback are
local functions, so they need static.

auth: fetch all metadata at once

Optimize IXFR-to-AXFR fallback path

Avoid making new backends when we are going to either deny the XFR, or
fall back to AXFR anyway.

This cuts down the number of new backends from four (three for IXFR
pre-checks plus one for AXFR) to one (just the AXFR one).
When replying in IXFR mode, we keep making _one_ new backend, which is
also better than before.

While we now hold the s_plock for a while longer, we only take it once
in doIXFR; before we took it twice -- for TSIG retrieval, which now
re-uses the IXFR backend.

rewrite pdns-distributes-queries

spellcheck: only run when docs have been changed

clarify: reuseports behavior re worker threads

Wishy-washy

Update recursordist: reuseport

we keep allowing these words so that we can do incidental spellchecks later

remove spelling cron; limit spelling to docs

Add support for FreeBSD's SO_REUSEPORT_LB

On FreeBSD, SO_REUSEPORT "permits multiple instances of a program to each
receive UDP/IP multicast or broadcast datagrams destined for the bound port",
while SO_REUSEPORT_LB distributes them "among the sharing processes based on
a hash function of local port number, foreign IP address and port number".

Add `--nobest` when installing PDNS software.

This is for testing in Docker images, we just want them to build.

Make sure we can install unsigned packages.

Sometimes we need to install unsigned packages from our own ad-hoc repo,
installing `apt-transport-https` makes sure we can do this (at least on
Debian Stretch).

rec: Defer the NOD lookup until after the response has been sent

If the NOD lookup is slow, for example because the destination
authoritative server is down, doing the NOD lookup before the response
has been sent increases the latency a lot.
This commit moves the actual NOD lookup after the response has been
sent, so we can still use the existing mthread (we might actually need
to do a proper DNS resolution to find the target authoritative server)
without keeping the client waiting.

rec: Fix formatting in the "root DS" unit test

Merge pull request #9141 from rgacogne/rec-remove-getquerylocaladdress-stub

rec: Remove unused getQueryLocalAddress stub in the unit tests

rec: Fix the handling of DS queries for the root

Merge pull request #9143 from Habbie/travis-unbreak-trusty

travis: install pdns package from direct download

travis: install pdns package from direct download

rec: Remove unused getQueryLocalAddress stub in the unit tests

The real function moved to the pdns namespace anyway.

Merge pull request #9137 from phonedph1/patch-21

rec: Update syncres.cc

Merge pull request #9138 from PowerDNS/omoerbeek-patch-1

rec: Typos and ref to rec instead of auth in security advisory

Typos and ref to rec instead of auth

Update syncres.cc

Merge pull request #9134 from omoerbeek/secpoll-cleanup

Secpoll cleanup, mark prereleases with vulnerabilities as such.

Secpoll cleanup, mark prereleases with vulnerabilities as such.

Merge pull request #9131 from mnordhoff/patch-6

rec: docs: Update the allow-from setting default

rec: docs: Update the allow-from setting default.

The default is LOCAL_NETS, but the copy of it in the documentation was incomplete.

Merge pull request #9111 from omoerbeek/dnsreplay-loop

Support LOOP link type in dnsreplay

Merge pull request #9126 from omoerbeek/rec-4.1.16-secpoll

Prepare rec 4.1.16 secpoll update

Prepare rec 4.1.16 secpoll update

Correct spelling error

Render examples as preformatted text

Signed-off-by: Kevin P. Fleming <kevin@km6g.us>

Address review feedback.

Signed-off-by: Kevin P. Fleming <kevin@km6g.us>

Clarify local-address documentation

The 'local-address' setting accepts both IPv4 and IPv6 addresses,
and also accepts port numbers. This patch documents all of those
behaviors.

Signed-off-by: Kevin P. Fleming <kevin@km6g.us>

Merge pull request #9124 from rgacogne/powerdns-advisory-2020-002

rec: powerdns-advisory-2020-002 -> powerdns-advisory-2020-02

rec: powerdns-advisory-2020-002 -> powerdns-advisory-2020-02

Merge pull request #9118 from pieterlexis/fix-rec-docs

Fix LaTeX-pdns recursor docs

Merge pull request #9119 from omoerbeek/fix-formatting

fix formatting

fix formatting

Add names and vars to spelling whitelist

Fix LaTeX-pdns recursor docs

Merge pull request #9114 from pieterlexis/rec-may-2020

Recursor fixes May 2020

Merge pull request #9113 from pieterlexis/rec-prep-may-2020

rec: prep for May 2020 releases

rec: prep for May 2020 security releases

Merge pull request #9106 from omoerbeek/release-cycles

Tell about the release cycles.

Merge pull request #8940 from pieterlexis/resolver-query-local-address6

Merge query-local-address6 into query-local-address

"Adress" is actually prural in query-local-address

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

Merge pull request #9073 from pieterlexis/runtime-dirs-virtual-hosting

Ensure runtime dirs for virtual services differ

Merge pull request #9085 from zeha/config

builder: improve shipped config files

Tell about the release cycles.

Merge pull request #9107 from jsoref/spell-check

Update check-spelling to 0.0.16-a

Update expects

spelling: Wshadow

Ignore c(xx)flags

spelling: with

spelling: target

spelling: should

spelling: record

spelling: milliseconds

spelling: improvements

spelling: first

spelling: answer

Update check-spelling action

Merge pull request #9018 from jsoref/spell-check

Set up spell-checking action

Support LOOP link type

Merge pull request #9105 from omoerbeek/auth-4.0-eol

Mark auth 4.0.x as EOL, plus some even older versions.

Mark auth 4.0.x as EOL, plus some even older versions.

Merge pull request #9099 from PowerDNS/omoerbeek-patch-1

rec: don't suggest to add a TA for powerdns.com

Merge pull request #9100 from omoerbeek/secpol-err-msg

Less negatives in error messages improves readability.

Also update surrounding text as noted by phonedph1

Less negatives in error messages improves readability.

don't suggest to a TA for pwerdns.com

Don't read potentially uninitalized memory if gethostname() failed

If the buffer is smaller than `HOST_NAME_MAX` (64 on Linux but up to
255 bytes in POSIX, which FreeBSD, MacOS etc honor) gethostname()
might return -1 without null-terminating the buffer, causing an
out-of-bounds read.
As we look for the first '.' using `strchr()`, replacing it with a
null byte, we also have a one-byte out-of-bounds write which might
result in a crash or, albeit very unlikely, arbitrary code execution.

rec: Fix DNSSEC validation of completely empty NXDomain answers

If the answer has no SOA and no NSEC inside a DNSSEC-secure zone,
we should go Bogus.

rec: Add a regression test for duplicate entries in a RPZ XFR update

rec: Don't crash on a duplicate RPZ entry received over XFR

Mention versionadded for the three branches

Limit the number of queries sent out to get NS addresses per query.

Merge pull request #9095 from rgacogne/dnsdist-1.5.0-rc2-changelog

dnsdist: Update the ChangeLog and secpoll for 1.5.0-rc2

TSIG Keys are also used for DNSUPDATE, so update API docs

Signed-off-by: Kevin P. Fleming <kevin@km6g.us>

replace boost:bind() with lambdas

lambdas are standard C++. bind is also recommended against by LLVM.

Signed-off-by: Rosen Penev <rosenp@gmail.com>

dnsdist: Update the ChangeLog and secpoll for 1.5.0-rc2

receiver: fix help output for pdns current-config

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 "::".

builder: avoid config change prompts on each pdns-recursor update

builder: use new --config=default for building config templates

dnsdist: add the unit to the help for latency buckets