Remi Gacogne
Remi Gacogne
> Actually, would it be possible to see the answers, and not just their lengths? Unfortunately dnsdist doesn't know how to parse most of the DNS records, because it doesn't...
I believe #14658 implemented everything we needed so I'm closing this issue :)
I don't think we should crash on this kind of configuration error, and I can indeed reproduce the issue: ``` (gdb) bt #0 lua_rawgeti (L=0x7ffff7f9e380, idx=, n=) at /usr/src/debug/luajit/LuaJIT-a4f56a459a588ae768801074b46ba0adcfb49eb1/src/lj_api.c:844 #1...
We crash in LuaWrapper indeed, while trying to access the traceback: ``` // stack top: {error, traceback} lua_rawgeti(state, -1, 1); // stack top: {error, traceback}, er\ ror ```
Fixed by https://github.com/PowerDNS/pdns/pull/16230
Applied, thanks!
Rebased on the current master, fixed the existing comments, and added YAML configuration support.
I fixed a lot of unused parameter warnings, likely a few of them will surface in slightly different feature sets.
> I like the unnamed parameter for always unused parameters: That works for me, but doesn't clang-tidy warn about that? Or Coverity? Perhaps it used to be a thing and...
It would so much easier if these tools could just stop reporting unused parameters for `override` methods. What is happening in this case is obvious, and clearly doesn't warrant a...