Merge pull request #12748 from omoerbeek/rec-ws-total-unused

rec: warning: variable 'total' set but not used

Merge pull request #12739 from omoerbeek/rec-lwres-uninited-len

rec: Fix uninited var causing wrong `bytes` field reported in protobuf log in rare cases

Merge pull request #12782 from omoerbeek/dsdist-api-test-timeout

dnsdist: github runners can be *very* slow wrt I/O, use a more generous timeout in tests

Init len earlier, as suggested by rgacogne

Merge pull request #12737 from nbakker/patch-1

Some webserver parameters were removed with 1.8.0

github runners can be *very* slow wrt I/O, use a more generous timeout in tests

This issue seems to be more prominent when using a TSAN enabled dnsdist.

Merge pull request #12765 from fredmorcos/clang-tidy-prs

Run `clang-tidy` on auth, rec and dnsdist when opening PRs

Merge pull request #12690 from rgacogne/ddist-statnode-docs

dnsdist: Fix the documentation of the `StatNode` object

Merge pull request #12729 from Habbie/lmdb-ls-dup-domain

auth LMDB: handle duplicate domain existence consistently

Merge pull request #12768 from chbruyand/ixfrdist-max-soa-refresh

ixfrdist: add a per domain max-soa-refresh option

Merge pull request #12636 from Habbie/lmdb-comments

auth lmdb: handle lack of support for RRset comments better

Merge pull request #12738 from paddg/patch-12

Undo an accidentally change (disableZeroScope -> disableZeroScoping)

getCatalogMembers: bubble up full return false

Document the process to disable specific clang-tidy lints

Improve clang-format documentation

Ignore macros when calculating cognitive complexity of functions

move assignment outside of condition

add comments

simplify oldestts checking, optimise hot path for non-LS users

move transaction opening outside of both if branches

simplify dup check

drop unused const

Merge pull request #12766 from phonedph1/patch-37

dnsdist: Update config.rst

rec: warning: variable 'total' set but not used [-Wunused-but-set-variable]

Merge pull request #12770 from neilcook/patch-2

Make Additionals documentation more accurate

format

remove debug print

filter duplicate domain entries

move handle_dups flag to lmdbbackend

lmdb-typed: teach get() to optionally fetch just the oldest entry

add comment

lmdb: add interface to get timestamp from LS header

lmdb indexops: avoid updating timestamp when there is no change

fix warnings

add missing defaults to doc

add lmdb-lightning-stream option

refactor getCatalogMembers

refactor getUpdatedMasters

refactor getUnfreshSlaveInfos

move loop into getAllDomainsFiltered

Make Additionals documentation more accurate

ixfrdist: add a per domain max-soa-refresh option to optionaly cap the refresh interval

Update config.rst

Filter file diffs that are not in the current project

Delete old clang-tidy workflow

Move clang-tidy-dnsdist to build-and-test-all workflow

Move clang-tidy-rec to build-and-test-all workflow

Move clang-tidy-auth to build-and-test-all workflow

Github action to run clang-tidy on dnsdist PRs

Github action to run clang-tidy on rec PRs

Github action to run clang-tidy on auth PRs

Merge pull request #12764 from fredmorcos/ci-fixes-cleanups

Ci fixes and cleanups

Pin pyasn1 to 0.4.8 because 0.5.0 broke the API with 0.5.0

https://github.com/pyasn1/pyasn1/issues/28

On 20230420: https://pypi.org/project/pyasn1/#history

Merge pull request #12740 from mind04/auth-nsec-at-delegation

Pick the right signer name when a NSEC name is also a delegation point

Merge pull request #12669 from chbruyand/auth-ifurlup-ips

auth: luarecords, enhance ifportup() with lists of sets of addresses like ifurlup()

Tell the libdecaf cmake build to use clang-12

Add cmake to auth build deps (needed for libdecaf)

Cleanup ci_auth/rec_configure pyinvoke tasks

Cleanup the ci_docs_upload_master pyinvoke task

Gitignore verify-dnssec-zone/allow-missing

Merge pull request #12758 from zeha/tcpreceiver-unused-var

auth: remove unused var in tcpreceiver.cc

Merge pull request #12759 from zeha/lmdb-unused-var

lmdb-safe: remove leftover debug variable

Merge pull request #12757 from zeha/apitestpipelimit

api tests: avoid default pipe size limit

lmdb-safe: remove leftover debug variable

In file included from ../../ext/lmdb-safe/lmdb-typed.cc:1:
../../ext/lmdb-safe/lmdb-typed.hh:656:11: warning: variable 'scanned' set but not used [-Wunused-but-set-variable]
      int scanned = 0;
          ^

auth: remove unused var in tcpreceiver.cc

tcpreceiver.cc:1005:7: warning: variable 'records' set but not used [-Wunused-but-set-variable]
  int records=0;

api tests: avoid default pipe size limit

Causes tests to freeze at a more or less random point.

Merge pull request #12747 from Habbie/auth-4.7.4-secpoll-docs

auth 4.7.4: secpoll and docs

Merge pull request #12755 from cmouse/lmdb-rst

docs: lmdb - Add blank line after explicit markup

auth 4.7.4: secpoll and docs

docs: lmdb - Add blank line after explicit markup

Merge pull request #12686 from aerique/feature/report-tested-repo-versions

Report Version when Testing Repositories

Merge pull request #12721 from mind04/pdns-loop

auth: calm down the communicator loop

Merge pull request #12749 from ossobv/wjd-document-bind-supermaster-destdir

document bind-supermaster-destdir

document bind-supermaster-destdir

Merge pull request #12693 from rgacogne/ddist174-changelog-secpoll

dnsdist: Add ChangeLog and secpoll update for dnsdist 1.7.4

dnsdist: Add ChangeLog and secpoll update for dnsdist 1.7.4

auth: calm down the communicator loop

make getCommonLabels() root aware

auth: add nsec at delegation point test

auth: fix nsec at delegation point

Print reported versions when testing repos.

Remove Ubuntu Bionic for auth-48 and dnsdist-18.
Remove rec-45 from repo test script since it is end-of-life.
Add rec-49 to repo test script.

rec: Fix uninited var causing wrong `bytes` field reported in protobuf log in rare cases

Reported by gcc build:

wres.cc: In function 'asyncresolve':
lwres.cc:631:26: warning: 'len' may be used uninitialized in this function [-Wmaybe-uninitialized]
  631 |       logIncomingResponse(outgoingLoggers, context ? context->d_initialRequestId : boost::none, uuid, ip, domain, type, qid, doTCP, dnsOverTLS, srcmask, len, lwr->d_rcode, lwr->d_records, queryTime, exportTypes);
      |                          ^
lwres.cc:390:10: note: 'len' was declared here
  390 |   size_t len;
      |          ^

While there, replace some mandatory pointers by refs

Merge pull request #12734 from omoerbeek/prep-4.9.0-alpha1

rec: Prep for rec-4.9.0-alpha1

Undo an accidentally change

'disableZeroScope' was  accidentally changed to 'disableZeroScoping' in commit 597a91a3cec373ebd4172d167a0f9c7500781027

Some webserver parameters were removed with 1.8.0

Merge pull request #12735 from zeha/rec-ubuntu-compression

recursor: restore explicit gzip compression for Ubuntu

Prep for rec-4.9.0-alpha1

recursor: restore explicit gzip compression for Ubuntu

reprepro for builder.powerdns.com cannot handle xz compression yet, but
newer Debian and Ubuntu dh_builddeb turn it on by default.

Was accidentally removed in #10072.

Merge pull request #12733 from omoerbeek/dnsdist-ebpf-docs

dnsdist: fix formatting and `external` description

dnsdist: fix formatting and `external` description

Merge pull request #12730 from dwfreed/fix-pcks

Fix spelling of PKCS

Merge pull request #12723 from omoerbeek/rec-bulk-no-powerdnssec.org

rec: Do not generate large not really useful test file containing names in powerdnssec.org in bulk test

rec: Do not generate large not really useful test files containing names in powerdnssec.org in bulk test by default

Fix spelling of PKCS

Merge pull request #12722 from omoerbeek/rec-bulk-pdq

rec: Also set pdns-distributes-queries for bulk tests.

rec: Also set pdns-distributes-quqeries for bulk tests.

Large imbalance seen without.

Merge pull request #12571 from fredmorcos/rec-unused-args-cleanup

Recursor: Unused parameters cleanup

Remove unused `now` argument from addRecordToRRSet

Fix cases mentioned in review, plus a few spotted on macOS

rec: Remove -Wno-unused-parameter from default C(XX)FLAGS