daemon/http: fix crash on large buffers and document logic issues

daemon/session: fix an issue with timers

The practical problem was also mitigated by libuv >= 1.32.0 (2ee2d46)

daemon/http: add more info to debug messagess

daemon/http: add helper assert to detect potential memleak

daemon/worker: ensure HTTP headers are freed on ignored queries

Ignored queries never call create_request() where ownership of headers
is taken care of. They need to be explicitly cleared instead, because
we're the owners of the pointer here.

daemon/http: refactor submitting to wirebuffer

daemon/http: ensure missing variable in URI is treated as error

daemon/http: ensure headers are freed

daemon/http: refactor freeing headers into function

fixup! log: lua logging

fixup! treewide: replace VERBOSE_STATUS, WITH_VERBOSE, etc.

I forgot that in some parts of code `qry` used for logging
could really be NULL; I suppose this should be cheap enough.

lib/utils: drop a pair of unused macros

treewide: replace VERBOSE_STATUS, WITH_VERBOSE, etc.

The main problem there was ignoring the log groups,
and VERBOSE_STATUS also ignores trace-logging.
Typically the new kr_log_is_debug*() are the best choice.

In a couple places with heavy WITH_VERBOSE usage it's
instead locally redefined to consider the right group.

validate: add kr_rrset_validation_ctx_t::log_qry

... so we can use it for more precise logging.  Some calls get simpler.
In particular, without the associated request, we can't produce anything
into trace-logs, which could be confusing.  Normal logs will benefit, too.
(more precise replacement of WITH_VERBOSE will come in a subsequent commit)

lib/log: add kr_log_is_debug*()

... and optimize a suspect hot path (through VERBOSE_MSG)

fixup! rework logging from control sockets (+simplify)

lib/log: make trace-logging not prevent normal logging

I think it's an old issue, but why not fix it now.
I believe it's better when these two are "independent".

Removed comment: once upon a time it belonged to auto_free
and similar macros, but somehow it survived various moves.

rework logging from control sockets (+simplify)

- unify interactive mode to stdout
- use its own logging group
- elevated log level when the command throws an exception
- don't try detecting that the logs go back into the same console
  (yes, in that case you can see some lines twice)
- don't make the binary mode turn off logging

QRVERBOSE: fix ignoring the log groups

It would be yet another condition to check; overall I think the
necessary checks would get too complex to be done in a macro
and therefore copied all over the place.

Note: it's interesting that such a change reduces binary size
by about 3% (on master), and I suspect that speed won't suffer at all
(after that TODO is done).

lib/log: typing improvements

- const for names of log groups
- enum kr_log_group for a single log group
- the kr_log_groups bitmap doesn't need to be exposed or even exported
- return bool instead of int

fixup! logging: use LOG_GRP_HTTP for http lua module

fixup! logging: use dedicated DOTAUTH log for experimental auth DoT

logging: fix GNUTLS logging

- ensure gnutls logging can be turned on/off by using GNUTLS log group
- keep TLS / TLSCLIENT as log groups for our TLS logs and use GNUTLS
  instead

logging: use LOG_GRP_HTTP for http lua module

It's better to use separate log group, to separate between logs that
come from the lua module vs native C implementation. It is also more
descriptive, since http modules is used for other stuff besides its
deprecated DoH.

logging: nitpicks

logging: use dedicated DOTAUTH log for experimental auth DoT

logging: rename *_log_groups() function to plural

Plural makes more sense to me here, since it can take a table with
multiple entries as input.

logging: add test for log groups

Besides testing the API, running this test also ensures all log groups
have been properly defined (as long as kr_assertions are turned on).

logging: add missing LOG_GRP_PLAN

logging: fix get_log_group()

To ensure all log groups have been properly defined, iterate over the
entire enum. If a value is missing, assert will be triggered.

Also fixes groups with number >=32 (32bit value was used to test
whether group is set before).

logging: add LOG_GRP_DEVEL

This serves two purposes:

1. As a utility logger during development.
2. As the last entry in enum - to make iteration over the values
   possible. Changing the value of LOG_GRP_DEVEL shouldn't be an issue,
   since it shouldn't be used in production code.

logging: print group name padded to 6 characters

Add space padding when printing the group name to allow usage of
regular group names in lua code (e.g. "io" instead of "io    ").

logging: add get_log_group() with better API

The function returns a table where key is the group name and the value
is either true (when logging selected group), or nil (when group isn't
selected).

This replaces the list_log_group() which didn't really work.

logging: ensure get/set_log_level behaves consistently

Ensure both these functions return string that represents the log level.

Also avoid using printf - it'd probably break API when using map().

lib: make NOTICE the default log level

I think notice is a good default - it should be used for scarce messages
that the user should see, while info level provides more low level info.

It could make sense to increase the log level to info if running with
journalctl (but it might be overkill).

daemon/main: log interactive message properly

fixup! log: fix CI tests

log: fix kr_log_fmt SIGSEGV crash

log: use kr_log_debug instead kr_log_verbose

log: fix CI tests

log: lua logging

log: fix CI tests

log: shortening LOG_GRP_<SOMETHING> to <SOMETHING>

log: remove hard-coded prefix from log messages

log: add level prefix to each message

log: remove NOVERBOSELOG option

log: lua: add list_log_group

fixup! log: add syslog target, systemd

fixup! log: groups lua interface

fixup! log: split to groups

fixup! log: add syslog target, systemd

fixup! log: groups lua interface

fixup! log: lua interface

fixup! log: remove kr_verbose_set

fixup! log: split to groups

fixup! log: lua interface

fixup! log: remove kr_verbose_set

log: groups lua interface

log: fix journal metadata

log: split to groups

log: lua interface

log: add syslog target, systemd

log: remove kr_verbose_status

log: remove kr_verbose_set

log: add log levels

Merge branch 'selection-txn' into 'master'

lib/selection: be more careful with RO transactions

See merge request knot/knot-resolver!1184

lib/selection: be more careful with RO transactions

They probably couldn't hang open for long, as each client request
should cause some cache-searching and thus close it, and even with
queries stopping I haven't managed to find a case where it would be
left open but... it's nicer to clean up and it should be very cheap.

Merge branch 'task-refcount-minimal' into 'master'

daemon/worker: fix a memory leak

See merge request knot/knot-resolver!1182

daemon/worker: fix a memory leak

Discovered case: TCP write towards upstream fails due to ECONNRESET,
and on this place of code we "forget" the whole qr_task and thus its
corresponding kr_request, so it remains unanswered and using memory.

Merge branch 'ci-centos8' into 'master'

ci: allow failure of CentOS 8 packaging tests

See merge request knot/knot-resolver!1185

ci: allow failure of CentOS 8 packaging tests

Once again there's an issue with libuv-devel. Since the distro is EOL in
6 months, I'm not going to waste effort with investigating or fixing it.

Merge branch 'disable-asserts' into 'master'

meson: disable assertions

See merge request knot/knot-resolver!1183

meson: disable assertions

Make sure assertions are turned off - we use our own mechanism
(implemented in !1146), so this mostly affects foreign code, like Knot
DNS headers.

https://gitlab.nic.cz/knot/knot-resolver/-/merge_requests/1146#note_203044

Merge branch 'apkg' into 'master'

distro: add apkg support

See merge request knot/knot-resolver!1178

ci: reorder pipeline

ci: reuse pkg artifacts

scripts: remove scripts obsoleted by apkg

distro/pkg: support custom release numbers

scripts: update OBS source creation script

ci: switch to Debian 11

scripts/make-archive: allow creation of tagged archive

ci: split pkgtest into separate child pipeline

ci: build and install packages with apkg

distro: add apkg support

move distro files into apkg template dir

Merge branch 'kind-doh-legacy' into 'master'

rename kind="doh" to kind="doh_legacy"

See merge request knot/knot-resolver!1180

rename kind="doh" to kind="doh_legacy"

The purpose of this change is to make it harder to accidentally use the
legacy DoH implementation and free up the "doh" kind which may be used
as an alias to a modern implementation in the future.

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%.