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 many algorithm supports #37

Merged
merged 1 commit into from May 24, 2012
Merged

add many algorithm supports #37

merged 1 commit into from May 24, 2012

Conversation

nappa
Copy link
Contributor

@nappa nappa commented Mar 26, 2012

Hi, I wrote a patch that implements many algorighms described below.

  • Key Exchange: diffie-hellman-group14-sha1, ecdh-sha2-nistp{256,384,521}
  • Host Key: ecdsa-sha2-nistp{256,384,521}
  • Authentication: ecdsa-sha2-nistp{256,384,521}
  • HMAC: hmac-ripemd160
  • Cipher: aes{128,192,256}-ctr, camellia{128,192,256}-ctr, blowfish-ctr, cast128-ctr, 3des-ctr, camellia{128,192,256}-cbc and arcfour (note that arcfour has problems with weak keys, and should be used with caution)

I've tested with these Rubies on Mac OS X:

  • Ruby-1.8.6-p420
  • Ruby-1.8.7-p358
  • Ruby-1.9.2-p318
  • Ruby-1.9.2-p318 (Linked with OpenSSL-1.0.1 that has Camellia and IDEA cipher algorithms)
  • Ruby-1.9.3-p125
  • JRuby-1.6.7 with JRE 1.6.0u29

(note that Ruby-1.8.6 and JRuby doesn't have OpenSSL::PKey::EC implementation, so "ecdh-" and "ecdsa-" are unavailable under them.)

And I confirmed compatibility with:

Hope it helps.

 * Key Exchange
   * diffie-hellman-group14-sha1
   * ecdh-sha2-nistp{256,384,521}
 * Host Key
   * ecdsa-sha2-nistp{256,384,521}
 * Authentication
   * ecdsa-sha2-nistp{256,384,521}
 * HMAC
   * hmac-ripemd160
 * Cipher:
   * aes{128,192,256}-ctr
   * camellia{128,192,256}-ctr
   * blowfish-ctr
   * cast128-ctr
   * 3des-ctr
   * arcfour (has problems with weak keys, and should be used with caution)
   * camellia{128,192,256}-cbc
@jmikola
Copy link

jmikola commented Mar 26, 2012

On a side note, this would make http://net-ssh.lighthouseapp.com/projects/36253/tickets/39-ecdsa-support redundant.

@nappa
Copy link
Contributor Author

nappa commented Mar 27, 2012

Thank you for the mention. I put a comment on the ticket ( http://net-ssh.lighthouseapp.com/projects/36253/tickets/39-ecdsa-support )

@nappa
Copy link
Contributor Author

nappa commented Apr 9, 2012

ping?

I've been waiting for comments...

@delano
Copy link
Collaborator

delano commented May 23, 2012

This looks great.

I still need to go through it but I'll get it in as soon as I can. Sorry for the delay.

@delano delano merged commit 3a74662 into net-ssh:master May 24, 2012
@delano
Copy link
Collaborator

delano commented May 24, 2012

Awesome work. I pulled it into the repo and pushed a 2.5.0 release to Rubygems.

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.

None yet

3 participants