gh actions - build-and-test-all: use a ubuntu-22 runner for job collect

gh actions - replace yq snap in collect job build-and-test-all

Merge pull request #13626 from Habbie/backport-13552-to-auth-4.8.x

auth-4.8: extend the systemd startup timeout during lmdb schema migrations

Merge pull request #13625 from Habbie/backport-13619-to-auth-4.8.x

auth-4.8: Add supervisor to Auth container image

Merge pull request #13624 from Habbie/backport-13093-to-auth-4.8.x

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

auth, extend the systemd startup timeout during lmdb schema migrations

(cherry picked from commit bd61829b3fbaf654129be0b77183d0b2ab966fe7)

one sentence per line

(cherry picked from commit 670426f796a849ed3a733aa750fac285ad0b81ad)

Minor docs change regarding supervisord in Kubernetes

(cherry picked from commit 9759a3983789f64fe3ac6a56ed671cdda467644d)

Supervisor in Auth container image

(cherry picked from commit 638d656cb6d9fea17d8065ea8ac173a99518ada2)

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

(cherry picked from commit b16aa8e4d785babed1b4b45e8165fed9f473a86b)

Merge pull request #13534 from romeroalx/auth-4.8.x-workflow-call

GH Actions - rel/auth-4.8.x: make build-and-test-all and builder workflows reusable from other branches

make builder workflow reusable

make build-and-test-all reusable

test ubuntu jammy build target

builder: drop ubuntu kinetic, it is EOL

Switch from deprecated ::set-output

https://github.blog/changelog/2022-10-11-github-actions-deprecating-save-state-and-set-output-commands/

Merge pull request #13316 from Habbie/backport-13136-to-auth-4.8.x

auth-4.8: ixfrdist: set AA=1 on SOA responses

ixfrdist: set AA=1 on SOA responses

(cherry picked from commit 63f3e97b0def39e0199a3c1c28d611ec7f74a686)

eqdnsmessage: adjust to type changes in dnspython

(cherry picked from commit 255ec835c0c0475f86ec81ed838dd3014d365043)

Merge pull request #13271 from Habbie/backport-13245-to-auth-4.8.x

auth-4.8.x: smysql: stop explicity setting MYSQL_OPT_RECONNECT to 0

smysql: stop explicity setting MYSQL_OPT_RECONNECT to 0

Setting this option, even to 0, causes spurious warnings to the console
with recent libmysqlclient versions. The upstream bug
( https://bugs.mysql.com/bug.php?id=112089 )
has now been open for a month, so we're implementing a workaround.

0 was the default since at least MySQL 5.7, perhaps longer.

closes #13242

(cherry picked from commit f1cbc20b72ab95531f0a84bab40ac6ea1610241f)

Merge pull request #13240 from Habbie/backport-13238-to-auth-4.8.x

auth-4.8: add default-catalog-zone setting

auth: add default-catalog-zone setting

(cherry picked from commit c8734ecd8cca0089a8a9c032bfab1069f60381e6)

Merge pull request #13186 from Habbie/backport-13105-to-auth-4.8.x

auth-4.8.x: (I)XFR: handle partial read of len prefix

Merge pull request #13187 from Habbie/backport-13071-to-auth-4.8.x

auth-4.8.x: fix code producing json

clang-tidy: Disable cppcoreguidelines-special-member-functions

We will need to re-enable it once https://github.com/PowerDNS/pdns/issues/12809
has been fixed.

(cherry picked from commit 4c6dd7970397f2220c015ab2403cc9f1977c1d19)

Merge pull request #13188 from Habbie/backport-13018-to-auth-4.8.x

auth-4.8.x: calidns: fix setting an ECS source of 0

Merge pull request #13189 from Habbie/backport-13005-to-auth-4.8.x

auth-4.8.x: Fix incorrect optsize

Fix incorrect optsize

(cherry picked from commit ca250bdad16b9ac6a2789541be88bf17a653262b)

calidns: Fix setting an ECS source of 0

(cherry picked from commit ba770e4f65aefdadc489a169ae66fd20c897162c)

Skip smileys for now, they take 4 bytes to encode and out current mysql
schema has 'utf8', which only handles 3 bytes max, should be changed to
utf8mb4 one day.

(cherry picked from commit 93ad866b4e2f4afb017e8b3b08041598a2378ea5)

Test a few non-ASCII chars in comments

(cherry picked from commit 56726eb113ab135ec890e79e94c0393986e9edad)

Add a few testcases for "incomplete" URLs

(cherry picked from commit 7ab40a80547d112914b71919d8f4aa14cc24b047)

Check all chars in the URL are valid URL chars.

Should probably (also) be done in YaHTTP::URL, though currently the
return value of YaHTTP::URL::parse() is completely ignored, so
there is no easy way to do.

(cherry picked from commit 35eb2fcffa40e7f70b716e99158efe72a0e864d9)

Implement recomendationm from #13050: step 1

Revert #12660

(cherry picked from commit 26f5d6058d8b0cf4ad2f8da729cb906796c297a0)

remove redundant assignment

(cherry picked from commit 8d3ab63b412fb4b9fd8732af47a5d1c18ba7e786)

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

(cherry picked from commit 8faf5a90992b2613cf5999c8dd5e26b0025050b7)

typo fix

(cherry picked from commit 8fb5bba04f7a211ac2eb815f5c340e69070dc3e0)

Merge pull request #13194 from Habbie/auth-4.8-docker-compose-space

auth-4.8: unbreak docker compose

unverbose apt

switch from `docker-compose` to `docker compose` and
stop installing docker-compose because that uninstalls runc

Merge pull request #13099 from Habbie/backport-13096-to-auth-4.8.x

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

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

(cherry picked from commit e0e59b3868b4caf7a2e4f93e6d8a802b8d83eccd)

Merge pull request #12996 from Habbie/backport-12990-to-auth-4.8.x

auth 4.8.x lmdb: in Lightning Stream mode, during deleteDomain, use RW transaction to get ID list

Merge pull request #12997 from Habbie/backport-12985-to-auth-4.8.x

auth 4.8 lmdb: add backend commands for checking & refreshing indexes

Merge pull request #12993 from Habbie/backport-12935-to-auth-4.8.x

auth-4.8.x: Stop using the now deprecated ERR_load_CRYPTO_strings() to detect OpenSSL

Merge pull request #12992 from Habbie/backport-12892-to-auth-4.8.x

auth-4.8: YaHTTP: Prevent integer overflow on very large chunks

Merge pull request #12991 from Habbie/backport-12961-to-auth-4.8.x

auth-4.8.x: Work around Red Hat 8 pooping the bed in OpenSSL's headers

auth lmdb: add index refresh-all backend command

(cherry picked from commit a451e81e890c91833d8f0c66488f488647ed51b6)

auth lmdb: add 'index check domains' and 'index refresh domains <ID>' backend-cmds

(cherry picked from commit d250e94f22407e2c8a08853229c8030382c75581)

auth lmdb: in Lightning Stream mode, during deleteDomain, use RW transaction to get ID list

(cherry picked from commit 3a99c00de710001bf28b940578ddc8d92272b28a)

Stop using the now deprecated ERR_load_CRYPTO_strings() to detect OpenSSL

And move to BN_new() instead, which has been present since at least
0.9.6 and is still in 3.1.

(cherry picked from commit 9fcef4932c9323b085984f8a087045fef70103f5)

YaHTTP: Prevent integer overflow on very large chunks

If the chunk_size is very close to the maximum value of an integer,
we trigger an integer overflow when checking if we have a trailing
newline after the payload.
Reported by OSS-Fuzz as:
https://oss-fuzz.com/testcase-detail/6439610474692608
https://bugs.chromium.org/p/oss-fuzz/issues/detail?id=56804

(cherry picked from commit b602982fc5b4fb9139dec591541e0c070ceb47f5)

Work around Red Hat 8 pooping the bed in OpenSSL's headers

The openssl/kdf.h header on EL8 is invalid because someone backported
a work-in-progress feature to an older OpenSSL branch and did not
bother to backport the fixes that were added later.

Red Hat declined to fix their mess and helpfully suggested we do the
work instead in https://bugzilla.redhat.com/show_bug.cgi?id=2215856

(cherry picked from commit 3dabf2d4a1a478fb00a232259e8043f075eb4d03)

Merge pull request #12878 from Habbie/auth-4.8.x-partial-12792

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

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

Merge pull request #12869 from Habbie/backport-12860-to-auth-4.8.x

auth-4.8.x: do not answer with broken TYPE0 data when expanding an ENT wildcard

Merge pull request #12872 from Habbie/backport-12866-to-auth-4.8.x

auth-4.8.x: lmdb: delete duplicate domain entries in deleteDomain

Merge pull request #12868 from Habbie/backport-12859-to-auth-4.8.x

auth-4.8.x: pdnsutil: if user pushes unknown key in response to "problem with zone" prompt, do not throw away their changes

document shards=1 enforcement

(cherry picked from commit 80ad487a185430a2f2878734018fc77803ec3d96)

format

(cherry picked from commit 28b19cd03961e52ea5e7aa091cee5bc8af594c2f)

simplify

(cherry picked from commit 9bd88b4b317aa347e95d723042e4b4098d1b619c)

use d_transactiondomain[id]

(cherry picked from commit 2f2cafa93d85e54e8a60b5a118e0c57aec80822f)

Update modules/lmdbbackend/lmdbbackend.cc

Co-authored-by: Otto Moerbeek <otto.moerbeek@open-xchange.com>
(cherry picked from commit 76eac3db390d121786d6f3da43e174ff2ebc31f3)

only throw when idvec is empty

(cherry picked from commit 53fdfae2c0e0aff70bb6ab5a834cffff084fbf7c)

auth lmdb: enforce shards=1 when running in lightning-stream mode

(cherry picked from commit 38eb2496bacf10df57569d937f6de85b59592c4a)

auth lmdb: delete duplicate domain entries in deleteDomain

(cherry picked from commit b2d1d86ad632f48fd4ae7dffa26f3eac05a8dce2)

pdnsutil: if user pushes unknown key in response to "problem with zone" prompt, do not throw away their changes

(cherry picked from commit 1ad0d42532e4934290bcf251f9e9865cc4519f01)

auth: do not answer with broken TYPE0 data when expanding an ENT wildcard

(cherry picked from commit e6d4f36b935567256ddcb01a9539bc784a3b814e)

Merge pull request #12871 from Habbie/backport-12818-to-auth-4.8.x

auth-4.8.x: clang-tidy: Increase the cognitive complexity threshold to 75

clang-tidy: Increase the cognitive complexity threshold to 75

The current level of 50 is too low for a function like
'handleQueuedHealthChecks' which is quite simple.

(cherry picked from commit 1463e0417a11bb97017662d7e4adf2abac846d19)

Merge pull request #12828 from Habbie/backport-12780-to-auth-4.8.x

auth-4.8: add setting workaround-11804-max-chunk-records #12780

Merge pull request #12827 from Habbie/backport-12683-to-auth-4.8.x

auth-4.8.x: remove ubuntu bionic target

Merge pull request #12826 from Habbie/auth-4.8-ci-cleanup

auth-4.8.x: clean up CI jobs for this branch

Apply changes for doc nits

(cherry picked from commit f5d4cf70dd112cc26c01f097d10984098375d69a)

auth: add setting workaround-11804

Workaround for https://github.com/PowerDNS/pdns/issues/11804. Defaults to no,
implying the previously hard-coded value of 100.

(cherry picked from commit 3af419da4c52fe7f9ba19f78b0349cc5eda9e1f2)

switch builder docs target to ubuntu jammy

(cherry picked from commit 37285a568a91542909394013d7619d4b9f17b512)

remove ubuntu bionic target, as 18.04 will EOL soon

(cherry picked from commit 17b4faeecdb345b104c641f56461ccc24364041a)

auth-4.8.x: clean up CI jobs for this branch

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

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)