-
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
plugin/dnssec, plugin/sign: ed25519 support #3380
Conversation
Thank you for your contribution. I've just checked the OWNERS files to find a suitable reviewer. This search was successful and I've asked dilyevsky (via If you have questions or suggestions for this bot, please file an issue against the miekg/dreck repository. The bot understands the commands that are listed here. |
Signed-off-by: Sean Liao <seankhliao@gmail.com>
Signed-off-by: Sean Liao <seankhliao@gmail.com>
This might need a tweak in plugin/sign as well. Could you check?
Current PR lgtm
…On Tue, 15 Oct 2019, 17:44 Sean Liao, ***@***.***> wrote:
1. Why is this pull request needed and what does it do?
adds support for ed25519 keys to plugin/dnssec, and corrects plugin/sign
ed25519 type assertion
2. Which issues (if any) are related?
#3379 <#3379>
3. Which documentation changes (if any) need to be made?
none
4. Does this introduce a backward incompatible change or deprecation?
no
------------------------------
You can view, comment on, or merge this pull request online at:
#3380
Commit Summary
- add ed25519 dnskey support
- fix ed25519 type assertion
File Changes
- *M* plugin/dnssec/dnskey.go
<https://github.com/coredns/coredns/pull/3380/files#diff-0> (5)
- *M* plugin/sign/keys.go
<https://github.com/coredns/coredns/pull/3380/files#diff-1> (2)
Patch Links:
- https://github.com/coredns/coredns/pull/3380.patch
- https://github.com/coredns/coredns/pull/3380.diff
—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
<#3380?email_source=notifications&email_token=AACWIW6LKBHHZJTRE7TNFULQOXXPVA5CNFSM4JA7HVCKYY3PNVWWK3TUL52HS4DFUVEXG43VMWVGG33NNVSW45C7NFSM4HR5XNQQ>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AACWIWZVSIQDS6FCJCCJTCDQOXXPVANCNFSM4JA7HVCA>
.
|
i just checked on my server, the second commit |
I believe both README.md now also need to be updated? |
Signed-off-by: Sean Liao <seankhliao@gmail.com>
I'm not sure which part of the READMEs to update. People should probably still use ecdsa keys if they don't want things to break. With this change coredns supports almost everything |
Codecov Report
@@ Coverage Diff @@
## master #3380 +/- ##
==========================================
+ Coverage 56.27% 56.29% +0.01%
==========================================
Files 218 218
Lines 10818 10820 +2
==========================================
+ Hits 6088 6091 +3
+ Misses 4253 4252 -1
Partials 477 477
Continue to review full report at Codecov.
|
ok, there also is https://github.com/coredns/coredns-utils/tree/master/coredns-keygen which we may want to give a single switch to allow ed25519 keys |
* add ed25519 dnskey support Signed-off-by: Sean Liao <seankhliao@gmail.com> * fix ed25519 type assertion Signed-off-by: Sean Liao <seankhliao@gmail.com> * clean up whitespace Signed-off-by: Sean Liao <seankhliao@gmail.com>
1. Why is this pull request needed and what does it do?
adds support for ed25519 keys to plugin/dnssec, and corrects plugin/sign ed25519 type assertion
2. Which issues (if any) are related?
#3379
3. Which documentation changes (if any) need to be made?
none
4. Does this introduce a backward incompatible change or deprecation?
no