dnsdist: Add an optional Lua callback for altering a Protobuf message #4305

merged 1 commit into from Aug 31, 2016


None yet

4 participants


For anonymization purpose, for example.
Closes #4112.

Habbie commented Aug 11, 2016 edited

I think this needs two extra lines after #4291? sorry :)


ready for review!

@rgacogne rgacogne added this to the dnsdist-1.1.0 milestone Aug 19, 2016
@pieterlexis pieterlexis commented on an outdated diff Aug 26, 2016
@@ -1274,8 +1274,8 @@ instantiate a server with additional parameters
* `QPSPoolAction(maxqps, poolname)`: set the packet into the specified pool only if it **does not** exceed the specified QPS limits, letting the subsequent rules apply otherwise
* `QPSAction(rule, maxqps)`: drop these packets if the QPS limits are exceeded
* `RCodeAction(rcode)`: reply immediatly by turning the query into a response with the specified rcode
- * `RemoteLogAction(RemoteLogger)`: send the content of this query to a remote logger via Protocol Buffer
- * `RemoteLogResponseAction(RemoteLogger)`: send the content of this response to a remote logger via Protocol Buffer
+ * `RemoteLogAction(RemoteLogger [, alterFunction])`: send the content of this query to a remote logger via Protocol Buffer. `alterFunction` is a callback, receiving a DNSQuestion and a DNSDistProtoBufMessage, that can be used to modify the Protocol Buffer content, for example for anonymization purpose
pieterlexis Aug 26, 2016 Member

s/purpose/purposes/ ?

@ahupowerdns ahupowerdns merged commit a6feb90 into PowerDNS:master Aug 31, 2016

1 check passed

continuous-integration/travis-ci/pr The Travis CI build passed
@rgacogne rgacogne deleted the rgacogne:dnsdist-lua-anon branch Aug 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment