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

Add ECDSA support. This fixes #1 #2

Merged
merged 3 commits into from
Oct 22, 2014
Merged

Add ECDSA support. This fixes #1 #2

merged 3 commits into from
Oct 22, 2014

Conversation

northox
Copy link
Contributor

@northox northox commented Mar 7, 2013

I changed my mind, here's the patch.

I'm not sure about this part:

  • if not quiet:
  • print >> sys.stderr, "Could only find key type %s for %s" % (keytype, hostname)

@xelerance
Copy link
Collaborator

I merged it into the devel branch , for testing.

@northox
Copy link
Contributor Author

northox commented Mar 11, 2013

This last patch would effectively complete draft-os-ietf-sshfp-ecdsa-sha2-00 support.

@northox
Copy link
Contributor Author

northox commented Mar 22, 2013

Any news?

@northox
Copy link
Contributor Author

northox commented Apr 8, 2013

Anything you want me to fix?

xelerance pushed a commit that referenced this pull request Oct 22, 2014
Add ECDSA support. This fixes #1
@xelerance xelerance merged commit 33d2783 into xelerance:master Oct 22, 2014
@nirgal
Copy link

nirgal commented Jan 23, 2015

This is great ^^

When is 1.2.3 or 1.3 scheduled?

@northox
Copy link
Contributor Author

northox commented Jan 23, 2015

By the mean time you can use this: https://github.com/northox/2sshfp

@nirgal
Copy link

nirgal commented Jan 23, 2015

Danny Fullerton:

By the mean time you can use this: https://github.com/northox/2sshfp

Nice :)

By the way, IANA assigned (temporarily) RR type 4 to ED25519:
https://www.iana.org/assignments/dns-sshfp-rr-parameters/dns-sshfp-rr-parameters.xml

Here, with unreleased Debian 8, I have an extra ssh-keyscan record with
algo = "ssh-ed25519".

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.

3 participants