remove ubuntu bionic target, as 18.04 will EOL soon

Don't fail when there are no warnings produced by clang-tidy

Quote the output vars

Add // NOLINT(readability-function-cognitive-complexity)
Once readability-function-cognitive-complexity.IgnoreMacros works
in CI, these annotations can be removed. It currenlty does not work
as clang-12 does not support it.

mark getenv() NOLINT

Locally (using clangd) no warning is reported...

Some general clang-tidy cleanup

Split recursorThread() to reduce complexity

Reformat

split main() to reduce complexity

Split serviceMain() to reduce cognitive complexity

serviceMain: fix all lints except complexity

Merge pull request #12788 from rgacogne/clang-tidy-increase-code-complexity-threshold

clang-tidy: Increase the code complexity threshold to 50 for now

clang-tidy: Lower the code complexity threshold to 50

Let's see how it goes.

clang-tidy: Increase the code complexity threshold to 100 for now

Merge pull request #12786 from rgacogne/ci-separate-failure-tidy

ci: Add a separate check for clang-tidy failure

Merge pull request #12787 from Habbie/auth-4.8.0-beta1-secpoll-serial

bump secpoll serial, adjust release date

bump secpoll serial, adjust release date

ci: Add a separate check for clang-tidy failure

This way the tests are still run even if clang-tidy reported a
warning, instead of stopping everything.

Merge pull request #12781 from Habbie/auth-4.8.0-beta1-docs

auth-4.8.0-beta1: secpoll&docs

dnsdist: Fix cache hit and miss metrics with DoH queries

Since we do two lookups for DoH queries forwarded over UDP (first
TCP then UDP), we need to be careful to only record a cache miss
in our last attempt.

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.

auth-4.8.0-beta1: secpoll&docs

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

Make DNSQType.TSIG available in dnsdist

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

Update Dynamic DNS Update Docs with GSS-TSIG

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

auth doc: Document 2-step process for consumer zone updates.

Closes #12714.

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

Log if the answer was marked variable by SyncRes and if it was stored into
the packet cache (if !quiet)

document bind-supermaster-destdir

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

dnsdist: Add ChangeLog and secpoll update for dnsdist 1.7.4