You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@camel.apache.org by gg...@apache.org on 2022/03/21 13:35:03 UTC

[camel] 01/01: Fixed CS

This is an automated email from the ASF dual-hosted git repository.

ggrzybek pushed a commit to branch CAMEL-17835
in repository https://gitbox.apache.org/repos/asf/camel.git

commit be182f8c0132c30d11b669f20bb362ebdf56d056
Author: Grzegorz Grzybek <gr...@gmail.com>
AuthorDate: Mon Mar 21 14:34:13 2022 +0100

    Fixed CS
---
 .../camel/component/scp/ScpAlgorithmsTest.java     | 34 +++++++++++++++-------
 .../camel/component/scp/ScpServerTestSupport.java  |  4 +--
 2 files changed, 26 insertions(+), 12 deletions(-)

diff --git a/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpAlgorithmsTest.java b/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpAlgorithmsTest.java
index e8462f6..830d038 100644
--- a/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpAlgorithmsTest.java
+++ b/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpAlgorithmsTest.java
@@ -59,13 +59,25 @@ public class ScpAlgorithmsTest extends ScpServerTestSupport {
     @Test
     public void testScpSimpleProduceWithDifferentAlgorithms() throws Exception {
         Config[] configs = new Config[] {
-//                new Config("hostkey-dsa.pem", BuiltinSignatures.dsa, BuiltinDHFactories.dhg14_256, BuiltinCiphers.aes128ctr),
-                new Config("hostkey-rsa.pem", BuiltinSignatures.rsaSHA256, BuiltinDHFactories.dhg16_512, BuiltinCiphers.aes192ctr),
-                new Config("hostkey-rsa.pem", BuiltinSignatures.rsaSHA512, BuiltinDHFactories.dhg16_512, BuiltinCiphers.aes192ctr),
-                new Config("hostkey-ecdsa256.pem", BuiltinSignatures.nistp256, BuiltinDHFactories.ecdhp256, BuiltinCiphers.aes128ctr),
-                new Config("hostkey-ecdsa384.pem", BuiltinSignatures.nistp384, BuiltinDHFactories.ecdhp384, BuiltinCiphers.aes192ctr),
-                new Config("hostkey-ecdsa521.pem", BuiltinSignatures.nistp521, BuiltinDHFactories.ecdhp521, BuiltinCiphers.aes256ctr),
-                new Config("hostkey-ed25519.pem", BuiltinSignatures.ed25519, BuiltinDHFactories.curve25519, BuiltinCiphers.aes256gcm)
+                //                new Config("hostkey-dsa.pem", BuiltinSignatures.dsa, BuiltinDHFactories.dhg14_256, BuiltinCiphers.aes128ctr),
+                new Config(
+                        "hostkey-rsa.pem", BuiltinSignatures.rsaSHA256, BuiltinDHFactories.dhg16_512,
+                        BuiltinCiphers.aes192ctr),
+                new Config(
+                        "hostkey-rsa.pem", BuiltinSignatures.rsaSHA512, BuiltinDHFactories.dhg16_512,
+                        BuiltinCiphers.aes192ctr),
+                new Config(
+                        "hostkey-ecdsa256.pem", BuiltinSignatures.nistp256, BuiltinDHFactories.ecdhp256,
+                        BuiltinCiphers.aes128ctr),
+                new Config(
+                        "hostkey-ecdsa384.pem", BuiltinSignatures.nistp384, BuiltinDHFactories.ecdhp384,
+                        BuiltinCiphers.aes192ctr),
+                new Config(
+                        "hostkey-ecdsa521.pem", BuiltinSignatures.nistp521, BuiltinDHFactories.ecdhp521,
+                        BuiltinCiphers.aes256ctr),
+                new Config(
+                        "hostkey-ed25519.pem", BuiltinSignatures.ed25519, BuiltinDHFactories.curve25519,
+                        BuiltinCiphers.aes256gcm)
         };
 
         for (final Config config : configs) {
@@ -78,7 +90,8 @@ public class ScpAlgorithmsTest extends ScpServerTestSupport {
                 sshd.setSignatureFactories(signatureFactories);
                 List<KeyExchangeFactory> keyExchangeFactories = sshd.getKeyExchangeFactories();
                 keyExchangeFactories.clear();
-                keyExchangeFactories.add(ServerBuilder.DH2KEX.apply(BuiltinDHFactories.resolveFactory(config.kexAlgorithm.getName())));
+                keyExchangeFactories
+                        .add(ServerBuilder.DH2KEX.apply(BuiltinDHFactories.resolveFactory(config.kexAlgorithm.getName())));
                 sshd.setKeyExchangeFactories(keyExchangeFactories);
                 List<NamedFactory<Cipher>> cipherFactories = sshd.getCipherFactories();
                 cipherFactories.clear();
@@ -99,13 +112,14 @@ public class ScpAlgorithmsTest extends ScpServerTestSupport {
         }
     }
 
-    private static class Config {
+    private static final class Config {
         final String privateKeyLocation;
         final BuiltinSignatures signatureAlgorithm;
         final BuiltinDHFactories kexAlgorithm;
         final BuiltinCiphers cipher;
 
-        private Config(String privateKeyLocation, BuiltinSignatures signatureAlgorithm, BuiltinDHFactories kexAlgorithm, BuiltinCiphers cipher) {
+        private Config(String privateKeyLocation, BuiltinSignatures signatureAlgorithm, BuiltinDHFactories kexAlgorithm,
+                       BuiltinCiphers cipher) {
             this.privateKeyLocation = "src/test/resources/keys/" + privateKeyLocation;
             this.signatureAlgorithm = signatureAlgorithm;
             this.kexAlgorithm = kexAlgorithm;
diff --git a/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpServerTestSupport.java b/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpServerTestSupport.java
index 64838a5..7a0f1a4 100644
--- a/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpServerTestSupport.java
+++ b/components/camel-jsch/src/test/java/org/apache/camel/component/scp/ScpServerTestSupport.java
@@ -59,14 +59,14 @@ public abstract class ScpServerTestSupport extends CamelTestSupport {
     protected static final String KNOWN_HOSTS = "known_hosts";
     protected static int port;
 
+    protected Consumer<SshServer> serverConfigurer;
+
     private boolean acceptLocalhostConnections = true;
     private String knownHostsFile;
 
     private boolean setupComplete;
     private SshServer sshd;
 
-    protected Consumer<SshServer> serverConfigurer = null;
-
     protected ScpServerTestSupport() {
         this(true);
     }