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

Allow ssh-agent keys that are unsupported #68

Merged
merged 3 commits into from
Sep 7, 2021
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
8 changes: 8 additions & 0 deletions Readme.md
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,7 @@ As I explained in a [blog post](http://www.matez.de/index.php/2020/06/22/the-fut
* This library is a Multi-Release-jar, which means that you can only use certain features when a more recent Java version is used.
* In order to use ssh-ed25519 & ssh-ed448, you must use at least Java 15.
* In order to use curve25519-sha256, curve448-sha512 & chacha20-poly1305@<!-- -->openssh.com, you must use at least Java 11.
* As of the [0.1.66](https://github.com/mwiede/jsch/releases/tag/jsch-0.1.66) release, these algorithms can now be used with older Java releases if [Bouncy Castle](https://www.bouncycastle.org/) (bcprov-jdk15on) is added to the classpath.

## Changes since fork:
* [0.1.66](https://github.com/mwiede/jsch/releases/tag/jsch-0.1.66)
Expand Down Expand Up @@ -99,6 +100,13 @@ As I explained in a [blog post](http://www.matez.de/index.php/2020/06/22/the-fut
* See `examples/JSchWithAgentProxy.java` for simple example
* ssh-agent support requires either [Java 16's JEP 380](https://openjdk.java.net/jeps/380) or the addition of [junixsocket](https://github.com/kohlschutter/junixsocket) to classpath
* Pageant support is untested & requires the addition of [JNA](https://github.com/java-native-access/jna) to classpath
* Added support for the following algorithms with older Java releases by using [Bouncy Castle](https://www.bouncycastle.org/):
* ssh-ed25519
* ssh-ed448
* curve25519-sha256
* curve25519-sha256@<!-- -->libssh.org
* curve448-sha512
* chacha20-poly1305@<!-- -->openssh.com
* [0.1.65](https://github.com/mwiede/jsch/releases/tag/jsch-0.1.65)
* Added system properties to allow manipulation of various crypto algorithms used by default
* Integrated JZlib, allowing use of zlib@<!-- -->openssh.com & zlib compressions without the need to provide the JZlib jar-file
Expand Down
7 changes: 7 additions & 0 deletions pom.xml
Original file line number Diff line number Diff line change
Expand Up @@ -50,6 +50,12 @@
<jna.version>5.9.0</jna.version>
</properties>
<dependencies>
<dependency>
<groupId>org.bouncycastle</groupId>
<artifactId>bcprov-jdk15on</artifactId>
<version>1.69</version>
<optional>true</optional>
</dependency>
<dependency>
<groupId>com.kohlschutter.junixsocket</groupId>
<artifactId>junixsocket-common</artifactId>
Expand Down Expand Up @@ -335,6 +341,7 @@
<version>0.8.7</version>
<configuration>
<excludes>
<exclude>com/jcraft/jsch/JavaVersion.class</exclude>
<exclude>com/jcraft/jsch/UnixDomainSocketFactory.class</exclude>
</excludes>
</configuration>
Expand Down
2 changes: 1 addition & 1 deletion src/main/java/com/jcraft/jsch/DHXEC.java
Original file line number Diff line number Diff line change
Expand Up @@ -89,7 +89,7 @@ public void init(Session session,
Q_C = xdh.getQ();
buf.putString(Q_C);
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
throw new JSchException(e.toString(), e);
}

Expand Down
26 changes: 20 additions & 6 deletions src/main/java/com/jcraft/jsch/JSch.java
Original file line number Diff line number Diff line change
Expand Up @@ -106,13 +106,9 @@ public class JSch{

config.put("ecdh-sha2-nistp", "com.jcraft.jsch.jce.ECDHN");

config.put("ssh-ed25519", "com.jcraft.jsch.jce.SignatureEd25519");
config.put("ssh-ed448", "com.jcraft.jsch.jce.SignatureEd448");

config.put("curve25519-sha256", "com.jcraft.jsch.DH25519");
config.put("[email protected]", "com.jcraft.jsch.DH25519");
config.put("curve448-sha512", "com.jcraft.jsch.DH448");
config.put("xdh", "com.jcraft.jsch.jce.XDH");

config.put("dh", "com.jcraft.jsch.jce.DH");
config.put("3des-cbc", "com.jcraft.jsch.jce.TripleDESCBC");
Expand Down Expand Up @@ -156,12 +152,10 @@ public class JSch{
config.put("keypairgen.dsa", "com.jcraft.jsch.jce.KeyPairGenDSA");
config.put("keypairgen.rsa", "com.jcraft.jsch.jce.KeyPairGenRSA");
config.put("keypairgen.ecdsa", "com.jcraft.jsch.jce.KeyPairGenECDSA");
config.put("keypairgen.eddsa", "com.jcraft.jsch.jce.KeyPairGenEdDSA");
config.put("random", "com.jcraft.jsch.jce.Random");

config.put("none", "com.jcraft.jsch.CipherNone");

config.put("[email protected]", "com.jcraft.jsch.jce.ChaCha20Poly1305");
config.put("[email protected]", "com.jcraft.jsch.jce.AES128GCM");
config.put("[email protected]", "com.jcraft.jsch.jce.AES256GCM");

Expand Down Expand Up @@ -189,6 +183,26 @@ public class JSch{

config.put("pbkdf", "com.jcraft.jsch.jce.PBKDF");

if(JavaVersion.getVersion()>=11){
config.put("[email protected]", "com.jcraft.jsch.jce.ChaCha20Poly1305");
config.put("xdh", "com.jcraft.jsch.jce.XDH");
}
else{
config.put("[email protected]", "com.jcraft.jsch.bc.ChaCha20Poly1305");
config.put("xdh", "com.jcraft.jsch.bc.XDH");
}

if(JavaVersion.getVersion()>=15){
config.put("keypairgen.eddsa", "com.jcraft.jsch.jce.KeyPairGenEdDSA");
config.put("ssh-ed25519", "com.jcraft.jsch.jce.SignatureEd25519");
config.put("ssh-ed448", "com.jcraft.jsch.jce.SignatureEd448");
}
else{
config.put("keypairgen.eddsa", "com.jcraft.jsch.bc.KeyPairGenEdDSA");
config.put("ssh-ed25519", "com.jcraft.jsch.bc.SignatureEd25519");
config.put("ssh-ed448", "com.jcraft.jsch.bc.SignatureEd448");
}

config.put("StrictHostKeyChecking", "ask");
config.put("HashKnownHosts", "no");

Expand Down
8 changes: 8 additions & 0 deletions src/main/java/com/jcraft/jsch/JavaVersion.java
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
package com.jcraft.jsch;

final class JavaVersion {

static int getVersion() {
return 8;
}
}
31 changes: 19 additions & 12 deletions src/main/java/com/jcraft/jsch/KeyExchange.java
Original file line number Diff line number Diff line change
Expand Up @@ -143,18 +143,25 @@ else if(guess[i]==null){
}
}

Class<?> _s2cclazz=Class.forName(session.getConfig(guess[PROPOSAL_ENC_ALGS_STOC]));
Cipher _s2ccipher=(Cipher)(_s2cclazz.getDeclaredConstructor().newInstance());
boolean _s2cAEAD=_s2ccipher.isAEAD();
if(_s2cAEAD){
guess[PROPOSAL_MAC_ALGS_STOC]=null;
}
boolean _s2cAEAD=false;
boolean _c2sAEAD=false;
try{
Class<?> _s2cclazz=Class.forName(session.getConfig(guess[PROPOSAL_ENC_ALGS_STOC]));
Cipher _s2ccipher=(Cipher)(_s2cclazz.getDeclaredConstructor().newInstance());
_s2cAEAD=_s2ccipher.isAEAD();
if(_s2cAEAD){
guess[PROPOSAL_MAC_ALGS_STOC]=null;
}

Class<?> _c2sclazz=Class.forName(session.getConfig(guess[PROPOSAL_ENC_ALGS_CTOS]));
Cipher _c2scipher=(Cipher)(_c2sclazz.getDeclaredConstructor().newInstance());
boolean _c2sAEAD=_c2scipher.isAEAD();
if(_c2sAEAD){
guess[PROPOSAL_MAC_ALGS_CTOS]=null;
Class<?> _c2sclazz=Class.forName(session.getConfig(guess[PROPOSAL_ENC_ALGS_CTOS]));
Cipher _c2scipher=(Cipher)(_c2sclazz.getDeclaredConstructor().newInstance());
_c2sAEAD=_c2scipher.isAEAD();
if(_c2sAEAD){
guess[PROPOSAL_MAC_ALGS_CTOS]=null;
}
}
catch(Exception | NoClassDefFoundError e){
throw new JSchException(e.toString(), e);
}

if(JSch.getLogger().isEnabled(Logger.INFO)){
Expand Down Expand Up @@ -361,7 +368,7 @@ else if(alg.equals("ssh-ed25519") ||
sig=(SignatureEdDSA)(c.getDeclaredConstructor().newInstance());
sig.init();
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
System.err.println(e);
}

Expand Down
6 changes: 3 additions & 3 deletions src/main/java/com/jcraft/jsch/KeyPairEdDSA.java
Original file line number Diff line number Diff line change
Expand Up @@ -57,7 +57,7 @@ void generate(int key_size) throws JSchException{

keypairgen=null;
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
//System.err.println("KeyPairEdDSA: "+e);
throw new JSchException(e.toString(), e);
}
Expand Down Expand Up @@ -134,7 +134,7 @@ public byte[] getSignature(byte[] data, String alg){
tmp[1] = sig;
return Buffer.fromBytes(tmp).buffer;
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
}
return null;
}
Expand All @@ -160,7 +160,7 @@ public Signature getVerifier(String alg){
eddsa.setPubKey(pub_array);
return eddsa;
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
}
return null;
}
Expand Down
15 changes: 6 additions & 9 deletions src/main/java/com/jcraft/jsch/Session.java
Original file line number Diff line number Diff line change
Expand Up @@ -610,7 +610,7 @@ private KeyExchange receive_kexinit(Buffer buf) throws Exception {
Class<?> c=Class.forName(getConfig(guess[KeyExchange.PROPOSAL_KEX_ALGS]));
kex=(KeyExchange)(c.getDeclaredConstructor().newInstance());
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
throw new JSchException(e.toString(), e);
}

Expand Down Expand Up @@ -1528,7 +1528,7 @@ private void updateKeys(KeyExchange kex) throws Exception{
method=guess[KeyExchange.PROPOSAL_COMP_ALGS_STOC];
initInflater(method);
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
if(e instanceof JSchException)
throw e;
throw new JSchException(e.toString(), e);
Expand Down Expand Up @@ -2574,9 +2574,6 @@ private void initDeflater(String method) throws JSchException{
catch(Exception ee){ }
deflater.init(Compression.DEFLATER, level);
}
catch(NoClassDefFoundError ee){
throw new JSchException(ee.toString(), ee);
}
catch(Exception ee){
throw new JSchException(ee.toString(), ee);
//System.err.println(foo+" isn't accessible.");
Expand Down Expand Up @@ -2855,7 +2852,7 @@ static boolean checkCipher(String cipher){
new byte[_c.getIVSize()]);
return true;
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
return false;
}
}
Expand Down Expand Up @@ -2904,7 +2901,7 @@ static boolean checkMac(String mac){
_c.init(new byte[_c.getBlockSize()]);
return true;
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
return false;
}
}
Expand Down Expand Up @@ -2947,7 +2944,7 @@ static boolean checkKex(Session s, String kex){
_c.init(s ,null, null, null, null);
return true;
}
catch(Exception e){ return false; }
catch(Exception | NoClassDefFoundError e){ return false; }
}

private String[] checkSignatures(String sigs){
Expand All @@ -2967,7 +2964,7 @@ private String[] checkSignatures(String sigs){
final Signature sig=(Signature)(c.getDeclaredConstructor().newInstance());
sig.init();
}
catch(Exception e){
catch(Exception | NoClassDefFoundError e){
result.addElement(_sigs[i]);
}
}
Expand Down
Loading