Skip to content
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

Support for controls clause, more options available out of 'extra_options' #69

Open
wants to merge 17 commits into
base: master
Choose a base branch
from

Conversation

LAripping
Copy link

Mostly additions, PR does not break anything. In detail:

1.Support for controls clause. Conf. template providing options regarding the rndc
channel the user might use. (Also requested in #63)
2.Many common options can be specified without the use of extra-options. Namely:
* allow-new-zones (yes | no)
* allow-notify (IP adr. list)
* notify (yes | no | explicit)
* auth-nxdomain (yes | no)
* request-ixfr (yes | no)
3. DNSsec options are now not mandatory, so their values are not printed in
the conf file if they are not specified.
4. Location of the key-file is now parametric. Again, template adds it
in the configuration only if the path is explicitly specified.

Leonidas Tsaousis and others added 17 commits November 4, 2016 14:48
The template now includes clauses for these values, instead of
forcing the edit of their values.
Currently it's a hardcoded option, not overwriteable used only to
help us with our implementation.

In a future commit this must be
more dynamic enabling the addition of multiple (or none) inets,
ability to skip completely the `controls `
clause and so on
...before pushing local (work PC) mods enabling custom `controls` in the
template and more dynamic `dnssec`-related options.
Currently, the changes to be proposed for merging upstream in the base
project (thias's module), relate to the configuration applied and they are:

1. DNSsec options are not mandatory, so their values are not printed in
   the conf file (via the template) if not modified
2. Location of the key-file is now parametric. Again, template adds it
   in the conf only if specific value is given.
3. Support for `controls` clause. Providing options regarding the rndc
   channel the user might use.
1. auth-nxdomain {yes | no}
2. notify {yes | no | explicit}
3. allow-notify { ;-separated list of IP addrs }
4. request-ixfr {yes | no}
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant