Results 287 comments of Remi Gacogne

If I'm reading https://github.com/PowerDNS/pdns/blob/master/pdns/lua-base4.cc#L247 right, the authoritative server and the recursor use the all caps version, so perhaps it would make sense to do that as well?

Looks ready for a merge apart from the whitespace spotted by Otto. Shall I fix it?

> Looks ready for a merge apart from the whitespace spotted by Otto. Shall I fix it? done

> I found no transaction in PacketHandler::performUpdate or GSQLBackend::replaceRRSet In case anyone else is wondering, `PacketHandler::performUpdate()` is called from `PacketHandler::processUpdate()` after a transaction has been started and before it's committed...

The `patchZone()` function also starts a transaction before doing anything else, weird..

Hi! I understand the issue, but I'm unsure what the solution is. > 1. Have separate counters for OPCODE drops. This seems too specific to me, if we go this...

Technically we can, but the amount of information there is already huge enough that it often doesn't fit in a terminal line, and my completely unscientific feeling is that it's...

It's `DNSOpcode.Query` see https://dnsdist.org/reference/constants.html#dnsopcode So ```lua addAction(NotRule(OpcodeRule(DNSOpcode.Query)), DropAction()) ```

I really want to implement this! I did not manage to squeeze it for 1.9.0 but it will be in 1.10 for sure, and perhaps in 1.9.1 if the change...

Thinking about this a bit more, also in the context of https://github.com/PowerDNS/pdns/issues/13750, I'm pondering a slightly different solution: adding a new action for dynamic block which would set a tag,...