Results 287 comments of Remi Gacogne

> CI executables used should as be close to production as possible. I fully agree but note that we already have very different binaries since we do CI with sanitizers...

Note that, for what it's worth, Arch already ships both the authoritative and recursor with LTO enabled.

I just pushed unit and regression tests, plus a new `DNSQuestion:setEDNSOption()` binding which is usable from a Lua function.

The current approach does not link, I will have to do things differently: - move the code dealing with the EDNS option to a different function in a different file...

Finally found the time to do that, this PR should now be ready for review.

Thanks! I'll have a look once they release it, although my understanding is that their code is written is Golang which is likely not going to play well with C++.

The DoQ code in the dnslibs seems to be using ngtcp2, for what it's worth.

Oh, Peter reported that a while ago..

That seems like a very reasonable feature request to me. It should not be hard to implement and I would happily merge a PR implementing these changes if anyone is...

> Since you have already added a defect-label: Are you aware of the issue and the reason for it? Is there a simple fix we could implement ourselves till 1.8.0...