Readability, and don't -I embedded lmdb if not using it.

Symbols not marked by KR_EXPORT shouldn't be visible outside
the same output binary (e.g. sbin/kresd or lib/kdns_modules/hints.so)
Also mark `engine_hint_root_file`.

Discovered by clang scan.

fixup! memcached: rename module kmemcached -> memcached

See merge request !421

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

See merge request !420

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

Monotonic nitpicks

See merge request !418

detect_time_jump: remove forgotten debug print

See merge request !417

fixup! module: detect discontinuous jumps in the system time

use monotonic time

Closes #247

See merge request !392

Vitezslav Kriz authored 7 years ago and Petr Špaček committed 7 years ago

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

Vitezslav Kriz authored 7 years ago and Petr Špaček committed 7 years ago

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 !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 !414

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

Vitezslav Kriz authored 7 years ago and Petr Špaček committed 7 years ago

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 !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 !408

Fix confusing error messages regarding root hints

Closes #278

See merge request !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

Previous cleanup attempt broke installation on systems without libedit.
client-install has to be defined at least as .PHONY target.
(I want to avoid if spagethi outside of client.mk.)

Pre-2.0 cleanup

See merge request !409