Merge branch 'improve-assertions' into 'master'

lib/utils: define kr_require(), kr_assume() macros

Closes #495

See merge request knot/knot-resolver!1146

treewide: rename assumptions to kr_assert() / kr_fails_assert()

To (hopefully) improve readability, rename the typical macro usage of:

    if (!kr_assume(x)) y; // to
    if (kr_fails_assert(x)) y;

As a convenience, replace the assert without a return value to a more
simple version:

    (void)!kr_assume(x);  // becomes
    kr_assert(x);

lib/dnssec/ta.c: replace asserts

lib/utils kr_ranked_rrarray_add(): clarify RRset sizes

lib/cache assertions: saving to cache is not critical

lib/cache: simplify rdataset_dematerialize()

Trying to handle the NULL here was pointless, so let's simplify.
As it was, it couldn't trigger anyway (would crash earlier).

lib/cache entry2answer(): trigger failures before jumps

It's a little more verbose, but it should be more valuable in case
we encounter them.

treewide: minor C11 cleanup

lib/utils kr_fail(): don't mangle errno

lib/cookies/nonce.c: remove assert.h include

treewide: eradicate "assert" in comments to fix CI

ci: check for "assert" in most C files

lib/utils kr_fail(): simple rate limit for forking

Default 5 minutes (but off).
Randomize the delays +-25%.

doc: document debugging options

doc: mention DEBUG_* policies for debugging requests

daemon: allow kr_assume() configuration

- add lua options debugging.assumption_abort and
  debugging.assumption_fork to make the behaviour configurable
- select default values based on meson buildtype:
  - debug/debugoptimized (default, e.g. used in CI) - abort and don't
    fork
  - plain/release - don't abort, but fork if abort is user-configured

utils/client/kresc.c: replace asserts

utils/cache_gc/main.c: replace asserts

utils/cache_gc/kr_cache_gc.c: replace asserts

utils/cache_gc/db.c: replace asserts

modules/stats/stats.c: replace asserts

modules/hints/hints.c: replace asserts

modules/dnstap/dnstap.c: replace asserts

modules/cookies/cookies.c: replace asserts

modules/cookies/cookiemonster.c: replace asserts

modules/cookies/cookiectl.c: replace asserts

lib/zonecut.c: replace asserts

lib/utils.h: replace asserts

lib/utils.c: replace asserts

lib/selection_iter.c: replace asserts

lib/selection_forward.c: replace asserts

lib/selection.c: replace asserts

lib/rplan.c: replace asserts

lib/resolve.h: replace asserts

lib/resolve.c: replace asserts

lib/module.c: replace asserts

lib/layer/validate.c: replace asserts

lib/layer/iterate.c: replace asserts

lib/generic/trie.c: replace asserts

lib/generic/queue.h: replace asserts

lib/generic/queue.c: replace asserts

lib/generic/pack.h: replace asserts

lib/generic/lru.h: replace asserts

lib/generic/lru.c: replace asserts

lib/dnssec/signature.c: replace asserts

lib/dnssec/nsec3.c: replace asserts

lib/dnssec/nsec.h: replace asserts

lib/dnssec/nsec.c: replace asserts

lib/dnssec.c: replace asserts

lib/cookies/helper.c: replace asserts

lib/cookies/alg_sha.c: replace asserts

lib/cookies/alg_containers.c: replace asserts

lib/cache/peek.c: replace asserts

lib/cache/nsec3.c: replace asserts

lib/cache/nsec1.c: replace asserts

lib/cache/knot_pkt.c: replace asserts

lib/cache/impl.h: replace asserts

lib/cache/entry_rr.c: replace asserts

lib/cache/entry_pkt.c: replace asserts

lib/cache/entry_list.c: replace asserts

lib/cache/cdb_lmdb.c: replace asserts

lib/cache/api.c: replace asserts

daemon/zimport.c: replace asserts

daemon/worker.c: replace asserts

daemon/udp_queue.c: replace asserts

daemon/tls_session_ticket-srv.c: replace asserts

daemon/tls.c: replace asserts

daemon/session.c: replace asserts

daemon/network.c: replace asserts

daemon/main.c: replace asserts

daemon/io.c: replace asserts

daemon/http.c: replace asserts

daemon/ffimodule.c: replace asserts

daemon/engine.c: replace asserts

daemon/bindings/net.c: replace asserts

daemon/bindings/impl.h: replace asserts

daemon/bindings/cache.c: replace asserts

meson: disable assetions

This primarily affects dependency headers and contrib code, as kresd now
uses kr_require() or kr_assume() instead of assert().

lib/utils: define kr_require(), kr_assume() macros

These macros should replace the use of assert() in our entire codebase.
assert() have the following issues:
  - can be turned off at compilation time
  - they don't have consistent meaning in our code

kr_require() behaves similarly to assert - it checks a condition and
aborts if it fails. Unlike asserts, these aren't turned off by using
-DNDEBUG. kr_require() should be used for non-recoverable errors.

kr_assume() is a way to check for non-fatal errors which supports error
reporing, debugging and recovery. The function returns a boolean value
which the caller must use for error handling. An error log message is
produced when the condition fails. Optionally, when kr_debug_assumption
is set to true, the process will use fork() and the child will abort().
This generates a coredump for debugging purposes, while allowing the
parent process to keep running and recover from the non-fatal error.
This can be useful for debugging hard to reproduce errors in production
environments.

Merge branch 'expose-http-headers' into 'master'

daemon/http: expose HTTP headers to kr_request

Closes #616

See merge request knot/knot-resolver!1165

doc: add DoH configuration net.doh_headers

daemon/http: add expose headers config test

nitpick: fix lint:tidy warnings

bindings/net: rename to doh_headers()

lib/resolve: make qsource.headers a struct instead of pointer

daemon/http: expose only chosen headers and limit max value length

daemon/worker: add doh_headers_in list

doc/net.tls(): update command example output

Technically, we return an actual lua table instead of this
pretty-printed representation, but I chose it to be consistent with the
rest of the docs.

daemon/http: expose HTTP headers to kr_request

Add HTTP headers to an optional kr_request.qsource.headers structure.
Headers are stored as name, value string pairs.

The following snippet can be used to access the headers in lua modules:

```
if (req.qsource.headers ~= nil) then
for i = 1, tonumber(req.qsource.headers.len) do
local name = ffi.string(req.qsource.headers.at[i - 1].name)
local value = ffi.string(req.qsource.headers.at[i - 1].value)
print(name, value)
end
end
```

Fixes #616

daemon/http: use struct for http stream

Merge branch 'trust-anchors' into 'master'

trust anchor improvements

Closes #673

See merge request knot/knot-resolver!1177

lib/dnssec/ta: remove trivial kr_ta_covers_qry()

lib/resolve: minor improvements around kr_ta_* calls

This orphaned kr_ta_get_longest_name() so it got removed as well,
even though it's a potentially useful abstraction.

lib/dnssec/ta: remove unused kr_ta_covers() function

It was word-for-word identical with kr_ta_get_longest_name() anyway,
except for additional pointer->bool conversion.

lib/dnssec/ta kr_ta_covers_qry(): generalize and improve

The closer (N)TA will win now.  That wasn't the case
if we had a positive TA underneath a negative TA.
(Well, I can't recall ever seeing anything close to a use case.)

validator: improve NTA precision

Merge branch 'ci-tags' into 'master'

ci: skip pipeline tests on tags

See merge request knot/knot-resolver!1176

ci: skip pipeline tests on tags

When releasing, checks are run before merging and tagging. Having these
checks run again only slows down the release process, especially if some
tests are flaky.

Merge branch 'release-5-3-2' into 'master'

release 5.3.2

See merge request knot/knot-resolver!1175

release 5.3.2