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

pkg-bind: dnssec-keygen to use ECDSA instead of default RSA algo/keys #216

Closed
wants to merge 7 commits into from

Conversation

hb9cwp
Copy link
Contributor

@hb9cwp hb9cwp commented Nov 21, 2016

dnssec-keygen without parameters for algorithm and key sizes uses per default RSA SHA1 (alg=5) with 2048 bit KSK resp. 1024 bit ZSK.

Add parameter -a ECDSAP256SHA256 to use ECDSA P245 SHA256 (alg=13) instead of defaults, following the trend and recommendations presented in http://schd.ws/hosted_files/icann562016/f5/Wander-ICANN56-DNSSEC-Adoption-v2.pdf
and
https://www.cloudflare.com/dns/dnssec/ecdsa-and-dnssec/

The default behaviour of BIND is that Slaves will send NOTIFY to other Secondaries. In pfSense, ```notify no``` in all Slave zones prevents this default behaviour, which is unexpected and looks like a bug until inspecting named.conf.

There is no way to overcome this ```notify no```, even when putting ```notify explicit {}``` into Custom Options of surrounding views for example. Even worse, BIND will throw an error when putting another ```notify yes|explicit {}``` into Custom Options of zones, and named will subsequently fail to start.

Admins who do want to prevent their Slaves from sending NOTIFY should put ```notify no``` explicitly into the Custom Options.
In my use case however, I need my Slaves to send NOTIFY to I/AXFR servers of the global anycast Secondaries at DNSMadeEasy using in the Custom Options of the view ```notify explicit;
also-notify {
 208.94.147.135;
 208.94.150.198;
 63.219.151.12;
}; ```

This only works with the proposed PR.
@rbgarga
Copy link
Member

rbgarga commented Nov 30, 2016

Looks like changes for #215 and #216 got mixed. Could you please re-submit? You could submit both changes, with 2 different commits, on the same Pull Request

@rbgarga
Copy link
Member

rbgarga commented Dec 1, 2016

Manually merged, thanks!

@rbgarga rbgarga closed this Dec 1, 2016
netgate-git-updates pushed a commit that referenced this pull request Apr 10, 2020
ChangeLog: https://github.com/canop/broot/releases/tag/v0.13.6

* Ignore diacritics in searches - Fix #216

PR:	245484
Submitted by:	[email protected] (maintainer)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants