GitLab

The `QRVERBOSE` macro uses the `query` pointer to find out whether the
request has trace log enabled. If it does, it uses trace log to log verbose messages using that callback (regardless of whether verbose mode is set or not).

This required changing of structure printing functions to formatting functions returning textual representation of the objects (dname, rrset, pkt).
This is potentially slower as creates heap objects, but it doesn't happen
in the hotpath so it doesn't really matter for verbose logs.

This doesn't affect any of the objects lifetime, just provides a
convenience for logging as some subsystems take only pointer of queries
not the request object.

This is useful in many troubleshooting scenarios when you want debug logs
just for a single request. It's going to expand on TRACE flag functionality
in the next PRs, so that special requests can be invoked with various tracers attached.

Currently this is only available in the C modules that can set the callback,
it's not called anywhere in the library yet.

this allows for changing request settings or performing other actions
just after the request is created.

CI: test transports

See merge request knot/knot-resolver!423

fixup! memcached: rename module kmemcached -> memcached

See merge request knot/knot-resolver!421

http: error out if GeoIP is requested but mmdblua is not available

See merge request knot/knot-resolver!420

struct timeval wasn't available, probably depending on libknot version,
breaking e.g. the Docker build.

Monotonic nitpicks

See merge request knot/knot-resolver!418

detect_time_jump: remove forgotten debug print

See merge request knot/knot-resolver!417

fixup! module: detect discontinuous jumps in the system time

use monotonic time

Closes #247

See merge request knot/knot-resolver!392

Check differences between real and monotonic time.
It clears cache when this diffrence changes since
last cache clear.

Monotonic time from libuv function uv_now (wrapped in kr_now) is
used for query timeout, stats and RTT in reputation cache.

Cache, DNSSEC Validation and Cookies use real time.

daemon: add missing RR type definitions

See merge request knot/knot-resolver!416

Some RR type definitions present in IANA DNS parameters registry were
missing in kresd. This commit synchronizes the tables with IANA registry
as of 2017-12-08 13:20 UTC.

At the same time, this commit is
fixup! converted constant tables, support kres.type.TYPE1234
The mentioned commit accidentally removed NULL definition which broke TA
signaling module.

time_skew: Detect time skew during kresd start.

See merge request knot/knot-resolver!414

Without this check, the module would print warning for every single run
without root trust anchor.

This module is enabled by default, but disabled in Deckard tests.

Originally division around sizeof(uint32_t) caused silent truncation
for struct kr_qflags with sizes not multiple of 4 bytes.

Attempts to optimize using uint32_t blocks could lead to read/write
beyond end of uint32_t so I'm not willing to risk it.

Also, the code was refactored to avoid duplication between _set and _clear.
Quick look into assembly produced by gcc 7.2.1 with -O2 on x86_64 confirms that
all auxiliary functions got inlined so there are not extra function calls.

Unit tests are attached. These fail on the previous version of _set() and
_clear() and work now.

CI fixups

Closes #282

See merge request knot/knot-resolver!415

Related: !405

Hopefully this will make build more reliable because breakage in Knot
DNS master will not affect us. On the other hand, we must not forget to
update Knot DNS in the image!

Closes: #282

code-style: replace `-->` in for loops with `i > 0` and `i--`

See merge request knot/knot-resolver!408

Fix confusing error messages regarding root hints

Closes #278

See merge request knot/knot-resolver!413

If user provides custom hints file with config option hints.root_file('file')
resovler fail with error. Before it just silently pass and may fail
when loading defaults file, which leads to confusing error message.

fixup! client: move sources into separate directory

See merge request !410