-
Notifications
You must be signed in to change notification settings - Fork 2.1k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Enhance configuration of dnstap plugin #4198
Labels
Comments
I would prefer the second syntax and probable s/allow/options/. Potentially also not using the abbreviations, because 'CR', 'FR' require deep knowledge of dnstap |
no one is picking this up |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
What would you like to be added:
Plugin
dnstap
should support configuration options for allowed types of DNSTAP messages.The config may look like
where all parameters after endpoint are optional
or
Other plugins should be able to check these config options. Bunch of booleans is ok.
See also #4179 (comment) for reference
Why is this needed:
PR #4179 removed the ability to disable CQ and/or CR messages with flags passes via context.
Adding the configuration options to dnstap plugin will restore ability to control generation of dnstap messages of particular types.
Note, other popular DNS servers (e.g. unbound) also have similar configuration options.
The text was updated successfully, but these errors were encountered: