Merge pull request #13168 from rgacogne/default-net-snmp-to-no

Change the default for building with net-snmp from `auto` to `no`

Change the default for building with net-snmp from `auto` to `no`

We have had a fair amount of issues with `net-snmp` adding unexpected
and unwanted values to both `CXXFLAGS` and `LDFLAGS`, and it was just
reported to also define `HAVE_LIBSSL` is in public header, which messes
up our own feature detection. Therefore it is likely better for everyone
to prevent net-snmp from being enabled without the user intending it.
For our own packages we explicitly enable net-snmp when supported,
and this commit also enables it in our CI for dnsdist (it was already
done for the recursor) so it should not have any impact.

Merge pull request #13166 from omoerbeek/rec-buildbot-nosnmp

rec/buildbot: Avoid running SNMP tests on the current buildbot setup

Avoid running SNMP tests on the current buildbot setup

Since the change from nose to pytest, the SNMP exclusion via the environment
var NOSE_EXLUDE does no longer work.

Merge pull request #13095 from rgacogne/coding-guidelines

Add a coding guidelines document to the public repository

Merge pull request #13071 from omoerbeek/json-encoding

Fix code producing json

Coding Guidelines: Apply suggestions from reviews

Merge pull request #13155 from rgacogne/channel-make-blocking-params-explicit

channel: Make the blocking parameters of the object queue explicit

Merge pull request #13102 from omoerbeek/rec-nodata-condition

rec: Do not assume the records are in a particular order when determining if an answer is NODATA

Merge pull request #13106 from omoerbeek/rec-aggr-cache-replace

rec: replace data in the aggressive cache if new data becomes available

Merge pull request #13144 from fredmorcos/iputils-cleanup

Consistency: Use #ifdef instead of #if in `iputils.cc`

Merge pull request #13152 from fredmorcos/cleanup-yaml-to-json

Cleanup `convert-yaml-to-json.py`

Consistency: Use #ifdef instead of #if in iputils.cc

channel: Make the blocking parameters of the object queue explicit

Merge pull request #13151 from aj-gh/fixtypos

Fix a few typos in log messages

Merge pull request #13147 from chbruyand/dnsdist-channel-fix

dnsdist: DOH - make query channel receiving part blocking

Cleanup convert-yaml-to-json.py

Fix a few typos.

Merge pull request #13149 from rgacogne/ddist-fix-frontend-metrics-recvmmsg

dnsdist: Properly record self-answered UDP responses with recvmmsg

Merge pull request #13146 from chbruyand/dnsdist-cosmetic-fixes

dnsdist: cosmetic compilation warning fixes

dnsdist: Properly record self-answered UDP responses with recvmmsg

Responses sent directly from dnsdist, without reaching out to a backend
(self-generated and cache hits answers, mostly) where not properly
accounted for in frontend metrics, ring buffer entries and latency
computation when recvmmsg/sendmmsg support was enabled via
`setUDPMultipleMessagesVectorSize()`.

dnsdsit: reset IncomingTCPConnectionState's buffer with clear() instead of resize()

dnsdist: remove unused variables warnings

dnsdist: make query channel receiving part blocking to avoid unnecessary CPU consumption

Merge pull request #13141 from rgacogne/auth-coverity-4

tools: Address more Coverity warnings

Merge pull request #13005 from jellekaufmann/master

optsize off by -4 when including DNS cookie

dnsbulktest: Fix warnings from clang-tidy

calidns: Fix warnings from clang-tidy

dnsgram: Fix Coverity CID 1401647

ixfrdist: Fix Coverity CID 1504137: Uncaught exception

dnsbulktest: Fix Coverity CID 1401681 and 1401676

calidns: Fix Coverity CID 1401679: Uncaught exception

Coding Guidelines: Mention reserve()/resize() explicitly, with links

Merge pull request #13135 from rgacogne/ddist-gettc

dnsdist: Add a DNSHeader:getTC() Lua binding

Merge pull request #13138 from Habbie/cid398388

coverity CID 398388: useless <0 check on unsigned value

dnsdist: Remove a second useless check, useless static casts

Merge pull request #13139 from fredmorcos/minor-cleanups

Minor cleanup

gettime: fix minor compilation warnings

Makefile whitespace

Merge pull request #13136 from Habbie/ixfrdist-soa-aa

ixfrdist: set AA=1 on SOA responses

Merge pull request #13134 from Habbie/ixfrdist-pytest

ixfrdist: switch testing to pytest

coverity CID 398388: useless <0 check on unsigned value

dnsdist: Fix the documented return value of DNSHeader:getTC()

ixfrdist: set AA=1 on SOA responses

eqdnsmessage: adjust to type changes in dnspython

Merge pull request #13094 from rgacogne/ixfrdist-main-complexity

ixfrdist: Reduce the complexity of the main function

dnsdist: Add a DNSHeader:getTC() Lua binding

ixfrdist: switch testing to pytest

ixfrdist: after --help or --version, exit (includes var rename)

Merge pull request #13114 from rgacogne/auth-coverity-3

auth: Fix uninitialized scalar field warnings from Coverity

Coding Guidelines: More suggestions from Chris Hofstaedtler (thanks!)

Merge pull request #13093 from rgacogne/ixfrdist-fix-max-soa-refresh

ixfrdist: Fix the validation of 'max-soa-refresh'

Merge pull request #13113 from rgacogne/ddist-discovery-short-read

dnsdist: Properly handle short reads on backend upgrade discovery

Coding Guidelines: Apply suggestions from review

ixfrdist: Call `getpwuid()` before going multi-threaded

auth: Fix Coverity CID 1504120: Uninitialized scalar field

auth: Fix coverity CID 1030024: Uninitialized scalar field

dnsdist: Uglify the code to make the formatter happy

Merge pull request #13105 from Habbie/rec-ixfr-debugging

(I)XFR: handle partial read of len prefix

dnsdist: Add a 'partial read' case to the backend upgrade tests

sstuff: Document that Socket::readWithTimeout() does not handle partial reads

dnsdist: Properly handle short reads on backend upgrade discovery

Merge pull request #13111 from franklouwers/master

Document default for `webserver-loglevel`

Coding Guidelines: Formatting of the C++ casts bullet points

Coding Guidelines: Some formatting of C++ keywords

Coding Guidelines: More descriptive section titles

Coding Guidelines: Capitalize list items

Coding Guidelines: Use decltype in code example

Coding Guidelines: BadFileDescriptorWrapper -> GoodFileDescriptorWrapper

Coding Guidelines: Some code formatting and minor fixes

Coding Guidelines: Rewording and typos

Coding Guidelines: Properly capitalize section names

Document default for `webserver-loglevel`

Merge pull request #12565 from jsoref/harmonize-xfr-out-messages

auth: harmonize *xfr log messages

add a few links; collection of small nits

formatting: one sentence per line. added ## to two things that looked like headings

Merge pull request #12949 from rgacogne/auth-channel

auth: Refactor the MultiThreadDistributor using pdns::channel

Make clang-tidy happy

remove redundant assignment

Merge pull request #13108 from fredmorcos/auth-fix-bind-backend-warnings

Auth: Fix warnings in bind backend

Merge pull request #13098 from omoerbeek/rec-docs-upgrade-stack

rec: Expand the note on max-recursion-depth in the upgrade notes

Auth: Fix warnings in bind backend

Merge pull request #13107 from paddg/max-packetcache-entries-doc-patch-1

Update max-packetcache-entries setting doc

IXFR client: handle partial reads of the TCP chunk length header, plus:
* add primarySOACount to exception text
* add indicator of current state to exception text
* a test

Update max-packetcache-entries setting doc

Since 4.9.0, the packet cache is sharded and shared by all threads.

rec: replace data in the aggressive cache if it becomes available

Currently, new data does not get recorded into the aggressive cache
if there's an existing entry that matches. Together with the fact
that in some cases pruning can be unfair (it scans the zones
always in the same order and stops clearing when it has reached the
goal) and/or not very active (when the recursor is lighlty loaded)
this has the consequence that old expired records can remain in
the cache that prevent new data to be recorded and used.

typo fix

rec: Do not assume the records are in a particular order when determining if an answer is
NODATA.

Merge pull request #13096 from Habbie/lmdb-do-reset-index-timestamp

auth lmdb: when broadcasting indexes, -do- rewrite them even if they are unchanged

Merge pull request #13089 from Habbie/auth-docs-metadata-dnssec

auth docs: remove domainmetadata/DNSSEC warning, has not been true for years

Expand the note on max-recursion-depth

auth lmdb: when broadcasting indexes, -do- rewrite them even if they are unchanged

Merge pull request #13087 from Habbie/rm-init.d

remove unused sysv init files

remove unused sysv init files - we don't ship them, and non-systemd downstream packages all have their own

ixfrdist: Delint ixfrdist.cc a bit more

Add a coding guidelines document to the public repository

We have had an internal version of this document for a while now,
but there is no reason not to make it available to external
contributors as well.

Merge pull request #13092 from rgacogne/rec-coverity

rec: Fixes a few performance issues reported by Coverity

Merge pull request #13091 from rgacogne/auth-coverity-2

auth: Fixes a few issues reported by Coverity

ixfrdist: Reduce the complexity of the main function

ixfrdist: Fix the validation of 'max-soa-refresh'