If CNAME chain occurs CNAME is placed before AAAA.

Casting is dangerous, e.g. it's easy to misconfigure policy with
  policy.add(policy.FORWARD('some address'))
which lead to segfault without showing any indication of the cause.
Now this case will show as
  .../policy.lua:98: 'struct kr_query' has no member named 'current'
and only abort the policy module instead of the whole process.

Also rename NOAUTH->NONAUTH.

... *if* we only want to ask the NSs, i.e. not to be put into answer.
This fixes iter_cname_cache test.

I see no sane way to set the flags from lua, so I made a C function.

function `add_file` is added as an alias to `config`,
but otherwise the interface is almost identical.

- update() had nothing to do in the public interface
- config() implementation moved out of the definition of the main table

Fixes #167.

The output is only better if built with libknot >= 2.4.0.
As a side-effect, add lua method knot_rrset_t::txt_dump;
it's a light wrapper around knot_rrset_txt_dump.

Re-tested rolling with a.moot-servers.net, to be sure.

Vladimír Čunát authored 8 years ago and Ondřej Surý committed 8 years ago

engine_cmd() doesn't print the error() exceptions thrown from lua;
it only leaves the message on lua stack.

(cherry picked from commit a316b9f7)

Vladimír Čunát authored 8 years ago and Ondřej Surý committed 8 years ago

- docs: fix cache.current_* since long ago d5272b4b
- don't allow "cache.foo = 'bar'" for abitrary foo
- restore cache['nic.cz'] after b31bad2ccf while not breaking completion
- #cache won't work on lua 5.1, so remove it

Vladimír Čunát authored 8 years ago and Ondřej Surý committed 8 years ago

These shouldn't make any problems:
- the verbose messages don't print any scope, and
- reputation cache doesn't consider scope.

The format of TA store is compatible both ways with old kresd.
Note: it requires the parent commit to work, i.e. new libzscanner.

Comment parsing will only be supported in future libzscanner version.
Also move out of the module, as trust_anchors.lua requires it and isn't
in a module.

engine_cmd() doesn't print the error() exceptions thrown from lua;
it only leaves the message on lua stack.

Ondřej Surý authored 8 years ago and Ondřej Surý committed 8 years ago

trust anchor fixes

Closes #151

See merge request !193

- Remove abstraction of the refresh callback, as we always use the same
  function.  This way the code is significantly easier to understand.
- Add a couple more comments.

It was strange to keep it indefinitely, and the code was already
half-prepared for this change.  Theoretically, if a resolver has been
stopped for too long to miss a key's period of Revoked status, it would
continue to trust the key forever.

- printing refresh time in milliseconds could've been confusing;
- move syntactic sugar definitions close to the rest of the table;

... via trust_anchors.{bootstrap_url,bootstrap_ca}

- docs: fix cache.current_* since long ago d5272b4b
- don't allow "cache.foo = 'bar'" for abitrary foo
- restore cache['nic.cz'] after b31bad2ccf while not breaking completion
- #cache won't work on lua 5.1, so remove it

Vladimír Čunát authored 8 years ago and Štěpán Balážik committed 8 years ago

That will allow us to discover the top-level symbols for completion.
For others we can e.g.: for n in pairs(net) do print(n); end
I must admit I fail to understand some of the related lua magic.