Vladimír Čunát authored 5 years ago and Tomas Krizek committed 5 years ago

- written relatively defensively - act OK even if the API
  isn't used in an ideal way
- CI lint:scan-build: bump the error count;
  It's only another instance of the mis-detected array_push().
- the removed stale note in modules/meson.build isn't really related

The new way of transitioning to layer callbacks - done because of
portability (mainly to aarch64) - is a bit expensive.  This is a simple
way of recovering that cost.  Merge 603a24fc regressed speed a bit.

Somehow I didn't notice this field when adding ::add_selected.
We probably never put anything into answer's ADDITIONAL,
so noone's noticed a problem until now.

Also the log now uses the same format query UID format as elsewhere.

... without light userdata and yet it should be efficient.
Also with checkout layer, but I have no nice way of testing that one.

...when making calls from lua to C modules; use luaJIT FFI instead.
This eliminates some cases of lua_pushlightuserdata().

This will allow modules like view etc. work in the same way with packets
received over HTTP.

Petr Špaček authored 6 years ago and Vladimír Čunát committed 6 years ago

There is no reason to have two distinct functions to do the same task,
especially if the Lua version did not print all the information.
(Found and amended a bit by vcunat.)

All cache operations are now counted. These are internal cache
operations and do not map directly to number of queries or even number
of RRs.

Closes: !515

Function originally called kr_cache_sync() was in fact responsible
for calling mdb_txn_commit() and not mdb_env_sync() which was confusing.

and deduplicate the parsing logic.

It seems simpler, now that we have kr_rnd_buffered().

Vladimír Čunát authored 6 years ago and Petr Špaček committed 6 years ago

We use it as relatively high-level function, often on user-input names,
so it seems suitable that it does convert the case.
This fixes cache.clear('Example.Org.'), and probably also negative trust
anchors and policy uses.

When at it, switch to generating the libzscanner bindings.

The _t isn't uint8_t* anymore but a structure with .len and .data.

To work on RRSIG TTLs, libknot >= 2.7.1 is needed.

Petr Špaček authored 6 years ago and Vladimír Čunát committed 6 years ago

Closes: #320

Petr Špaček authored 6 years ago and Vladimír Čunát committed 6 years ago

Remove stray kr_zonecut_find_nsname() which somehow appeared in
45e38b3d.

Marek Vavruša authored 7 years ago and Grigorii Demidov committed 6 years ago

Previously the module was created on configuration time, so it wasn't
possible to inject custom endpoints to the default interface.

We don't need the end, but I prefer to get rid of the exception,
as we now require libknot > 2.6 anyway and it only implies adding
a few more binding lines.