-
Notifications
You must be signed in to change notification settings - Fork 70
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
Add support for defining custom dnssec-policies #206
Conversation
bd94f89
to
b4fc612
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Some very minor suggestions, overall it looks good to me.
b4fc612
to
3e58e83
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm still not entirely sure.
51ad705
to
653b75b
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I have no strong opinion (and not too versed in dnssec), but it doesn't look crazy to me? ;) |
653b75b
to
b84ec7f
Compare
Thanks! I squashed the commits. |
This is a complement to #205 to allow to setup custom policies.
This is currently a draft until I manage to deploy this code in a real-world workload.