Merge pull request #9583 from Habbie/dnsdist-make-fixes

dnsdist make: two fixes

auth: remove always true conditions in zone2sql

Merge pull request #9581 from zeha/auth-readme

auth: update bionic packages in README

dnsdist make: two fixes

* make sure dnsdist-lua-ffi-interface.inc is built before dnsdist-lua-ffi.o
* cleaner output while building that .inc

by Pieter Lexis

auth: update bionic packages in README

Especially for gpgsql, we do not need the full postgresql-server-dev package.

Support serving APL records pointing to fe80:: on macOS

rec: Reply with NOTIMP instead of terminating the TCP connection

For qdcount == 0 and UPDATE/NOTIFY opcodes.

Fix error checking on one getaddrinfo call

Merge pull request #9569 from pieterlexis/rec-no-cfg-parse-on-version

recursor: Don't parse any config with `--version`

Merge pull request #9562 from omoerbeek/rec-flush-typed-api

rec: Expose typed cache flush via Web API

Be explicit we're talking about the recursor.

rec: Terminate TCP connections instead of 'ignoring' errors

We used to ignore questions that we consider invalid (unexpected
opcode, qdcount != 1, QR=1, parse error, ...) but also those
received from source addresses blocked by ipfilter, still waiting
for a new question to come up on the socket.
That might be fine for clients that will keep sending queries, even
though they will still end up wondering what happened to the ignored
queries, but some clients like dnsdist will wait until a response is
sent, or a time out occurs.
Closing the TCP connection instead allows dnsdist to keep going,
possibly retrying over a new connection but finally giving up,
instead of keeping the connection alive.

Add script to check sort order for SOURCES

Merge pull request #9563 from omoerbeek/rec-docs-endpoint-servers-config

rec: fix double "Configuration endpoint" in docs

Sort all _SOURCES = lines

Merge pull request #9558 from Habbie/auth-4.4.0-alpha1-docs

auth: changelog for 4.4.0-alpha1

recursor: Don't parse any config with `--version`

This ensures we don't log anything _apart_ from the version info.
Spotted in https://github.com/PowerDNS/pdns_recursor-ansible/issues/66

generate-repo-files: add auth-44

Merge pull request #9564 from Habbie/centos6-inttypes

el6: set __STDC_FORMAT_MACROS during package builds

document 4.3.1 sqlite index changes

4.4.0-alpha1 changelog, secpoll, notes

auth: Speedup presigned signature lookups.
There was an unnecessary soa lookup which resulted in a large number of extra backend queries.

Merge pull request #9551 from rgacogne/ddist151-changelog-secpoll

dnsdist: ChangeLog and secpoll update for 1.5.1

dnsdist: 1.5.1 is a bit late, fixing the secpoll and ChangeLog :)

dnsdist: Add 9555 to 1.5.x's changelog

dnsdist: Prepare secpoll update for 1.5.1

dnsdist: Add the ChangeLog entries for 1.5.1

Switch off IPv4 for this test to make sure there's only one IP per NS.
This avoids the non-deterministic behaviour.

Merge pull request #9559 from omoerbeek/rec-test-glueless-referral-loop

rec: add unit test for potential NS loop.

Count number of queries as suggested by @rgacogne. Do not assert on it yet,
we are seeing some non-deterministic behaviour.

el6: set __STDC_FORMAT_MACROS during package builds

Merge pull request #9554 from pieterlexis/rec-rm-qla6

Recursor: Remove query-local-address6

Due to include structure we have a double "Configuration endpoint" header

Expose typed cache flush via Web API

Merge pull request #9553 from rgacogne/ddist-edns-negative-additional-soa

dnsdist: Only add EDNS on negative answers if the query had EDNS

Merge pull request #9556 from rgacogne/ddist-debug-trailing

dnsdist: Debug trailing data failures

auth: remove local-ipv6, query-local-address6

Merge pull request #9515 from omoerbeek/rec-wipe-cache-typed-followup

rec: Actually fix wipe-cache-typed

Recursor: Remove query-local-address6

dnsdist: Update regression tests for dnspython 2.0.0

dnsdist: Fix Cookie and ECS implementations for dnspython 2.0.0

dnsdist: Only add EDNS on negative answers if the query had EDNS

Merge pull request #9550 from zeha/apl-testcase

dnsrecords: fix APL test on macOS

Merge pull request #9549 from zeha/auth-stat-prefix

pdns_control: add show <prefix>

Merge pull request #9519 from rgacogne/check-sscanf-return-value

Always check sscanf's return value

dnsrecords: fix APL test on macOS

macOS does not life fe80:: without an %interface specifier, so inet_pton
fails, and we end with an error like this:

check rec->getZoneRepresentation() == zoneval has failed [2:invalid Non-recoverable failure in name resolution/128 != 2:fe80:1234:5678:9910:8bc:3359:b2e8:720e/128]

Fix test by switching to a "normal" internet class prefix.

pdns_control: add show <prefix>

Merge pull request #9544 from pieterlexis/svcb-coverity

svb-records: Initialize d_port

svb-records: Initialize d_port

Should fix 6 coverity warnings.

dnsdist: Debug trailing data failures

FIXME400

Merge pull request #9531 from RobinGeuze/fixCoverityLMDBIssues

Fix a small coverity issue in lmdb backend and fix a compiler warning

dnsdist: Fix documentation issues reported by Josh Soref (thanks!)

dnsdist: Rename topRule() and friends

Merge pull request #9530 from Habbie/additional-processing-db-state

auth: avoid bad DB state in impossible situation; fix a typo

Catch the none existent shard case and just skip that

Fix a small coverity issue in lmdb backend and fix a compiler warning

avoid bad DB state in impossible situation; fix a typo

Fix wrong parameter

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

Merge pull request #9369 from pieterlexis/draft-ietf-dnsop-svcb-https-01

SVCB and HTTPS support

Add API test for SVCB records

Switch SvcParam to std::move semantics

SVCB: return const-refs for heavy kvs

Add SVCB and HTTPS to LDAP schema

Add some regression tests for SVCB

Add SVCB/HTTPS checks to pdnsutil

Add additional processing for SVCB and HTTPS

Add SVCB records, parsing and writing

Add SvcParam class plus tests

iputils: Add function to print comma-separated CA containers

Merge pull request #9518 from mind04/pdns-api-metadata

auth: make sure the metadata cache is not filled from within an api transaction

Merge pull request #9524 from qvr/fix/rediscover-master-alsonotify-changes

auth: bindbackend: 'rediscover' changes to master and also-notifies

auth: bindbackend: 'rediscover' changes to master and also-notifies

builder-support: Disable luajit usage on aarch64 Debian/Ubuntu builds

The luajit package is currently not compatible with the aarch64 architecture,
so when Debian or Ubuntu packages are built for that architecture, use lua5.3
instead.

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

builder-support: Improve consistency of Debian Stretch/Buster configurations

(Also applies to all post-Xenial Ubuntu builds since they use these configurations too)

* Use debhelper version 10.x on both distributions

* Use consistent logic for enabling systemd support

* Remove unnecessary --parallel flags (default mode in debhelper 10.x)

* Always use architecture.mk and pkg-info.mk to set DEB_* variables

* Enable warnings and hardening on Stretch builds

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

builder-support: Extend 'all' module support to docs/sdist/deb targets

The build.sh script can build 'all' modules in a single execution, but
not all of the Dockerfile templates supported this mode. Since this is useful
for manual testing of the build process, this patch adds 'all' support
to the 'docs' and 'sdist' templates, and the templates which produce
Debian-style packages.

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

Always check sscanf's return value

These remaining cases are not in any way a security issue since the
variables are initialized before the call. This mostly improves the
reporting, and make sure we fail earlier.

auth: canDoAXFR: avoid confusing B name

auth: dnsbackend: replace a bare NULL

slaveRefresh: avoid extra SOA lookups

For incoming NOTIFYs, we already call getDomainInfo, so reuse the serial
from there. For unfresh domains, we already have a serial from the
backend.

However, a serial of 0 is now always considered stale.

auth: fix typo in backend-writers-guide.rst

auth: remove UeberBackend::getSOA

slaveRefresh: remove B->getSOA

Merge pull request #9496 from mind04/pdns-dnskey

auth: ignore cryptokeys in presigned zones

Merge pull request #9039 from Habbie/pg-escape-connect-string

auth: quote/escape PG connection parameters

Merge pull request #9389 from RobinGeuze/lmdbNoDupSort

Removed the need for DUPSORT lmdb databases

auth: replace full zone rectifies with rectifyZone() in the rfc2136 code

auth: avoid metadata cache polution in the api code

Removed the need for DUPSORT lmdb databases. Includes automatic database conversion

Merge pull request #8942 from Habbie/lua-backtrace

lua: add backtraces to errors

clarify quoting/escaping upgrade note, thanks @jsoref

quote/escape PG connection parameters

Merge pull request #9427 from mind04/pdns-2136-metadata

auth: improve metadata caching

Merge pull request #9517 from jsoref/spelling

Spelling

Merge pull request #9495 from rgacogne/rec-rearm-after-ooor-timeout

rec: Watch the descriptor again after an out-of-order read timeout

Merge pull request #9504 from rgacogne/rec-time-constness

rec: Better const-ness when dealing with timestamps

Merge pull request #9505 from rgacogne/rec-unused-sign-tags

rec: Remove unused vector of tags in validateDNSKeysAgainstDS()

Merge pull request #9513 from rgacogne/ddist-fix-parse-edns-options

dnsdist: Fix getEDNSOptions() for {AN,NS}COUNT != 0 and ARCOUNT = 0

A few more g_negache un-threading cases.

Since record cache and negcache are global now no need to broadcast to all
threads.