Fix some sonar/checkstyle issues

Change-Id: Id38d1e320f68ca12625dedfc19ea81cd81ea3005
Signed-off-by: David Suarez <david.suarez.fuentes@gmail.com>

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/AuthorizedKeysDecoder.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/AuthorizedKeysDecoder.java
index 2f73805778256ecc488c5ad43760190ab6b8ea0a..ac476c6c9115879d5785a6f0df64c7d178db37fe 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/AuthorizedKeysDecoder.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/AuthorizedKeysDecoder.java
@@ -43,7 +43,7 @@ public class AuthorizedKeysDecoder {
     private static final String KEY_FACTORY_TYPE_DSA = "DSA";
     private static final String KEY_FACTORY_TYPE_ECDSA = "EC";
 
-    private static Map<String, String> ECDSA_CURVES = new HashMap<>();
+    private static final Map<String, String> ECDSA_CURVES = new HashMap<>();
 
     static {
         ECDSA_CURVES.put("nistp256", "secp256r1");
@@ -51,8 +51,8 @@ public class AuthorizedKeysDecoder {
         ECDSA_CURVES.put("nistp512", "secp512r1");
     }
 
-    private static String ECDSA_SUPPORTED_CURVE_NAME = "nistp256";
-    private static String ECDSA_SUPPORTED_CURVE_NAME_SPEC = ECDSA_CURVES.get(ECDSA_SUPPORTED_CURVE_NAME);
+    private static final String ECDSA_SUPPORTED_CURVE_NAME = "nistp256";
+    private static final String ECDSA_SUPPORTED_CURVE_NAME_SPEC = ECDSA_CURVES.get(ECDSA_SUPPORTED_CURVE_NAME);
 
     private static final String KEY_TYPE_RSA = "ssh-rsa";
     private static final String KEY_TYPE_DSA = "ssh-dss";

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeAuthorization.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeAuthorization.java
index 03edbdac7cf609b0498eb91a00f33ab113c41a6f..1896b2b0054a81fc2d4862cf8ddab656da965857 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeAuthorization.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeAuthorization.java
@@ -100,8 +100,8 @@ public abstract class CallHomeAuthorization {
 
         private final String nodeId;
         private final String username;
-        private Set<String> passwords = new HashSet<>();
-        private Set<KeyPair> clientKeys = new HashSet<>();
+        private final Set<String> passwords = new HashSet<>();
+        private final Set<KeyPair> clientKeys = new HashSet<>();
 
         private Builder(String nodeId, String username) {
             this.nodeId = Preconditions.checkNotNull(nodeId);

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeSessionContext.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeSessionContext.java
index 4f37f987d52f2d4d3ea3f1c4e83993f5bfce9e86..05c9991677f58d54e93766b7c8f829fdac4b1968 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeSessionContext.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/CallHomeSessionContext.java
@@ -48,8 +48,8 @@ class CallHomeSessionContext implements CallHomeProtocolSessionContext {
     private volatile MinaSshNettyChannel nettyChannel = null;
     private volatile boolean activated;
 
-    private InetSocketAddress remoteAddress;
-    private PublicKey serverKey;
+    private final InetSocketAddress remoteAddress;
+    private final PublicKey serverKey;
 
     CallHomeSessionContext(ClientSession sshSession, CallHomeAuthorization authorization,
                            SocketAddress remoteAddress, Factory factory) {

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServer.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServer.java
index 34658aae5588a6bc7e5aa2426852914937c87fbf..723273a763577bd1b7aee25840a106e72ca21586 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServer.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServer.java
@@ -40,7 +40,7 @@ public class NetconfCallHomeServer implements AutoCloseable, ServerKeyVerifier {
     private final CallHomeAuthorizationProvider authProvider;
     private final CallHomeSessionContext.Factory sessionFactory;
     private final InetSocketAddress bindAddress;
-    private StatusRecorder recorder;
+    private final StatusRecorder recorder;
 
     NetconfCallHomeServer(SshClient sshClient, CallHomeAuthorizationProvider authProvider, Factory factory,
                           InetSocketAddress socketAddress, StatusRecorder recorder) {

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServerBuilder.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServerBuilder.java
index 7814842a1d32667dbe9ea313aea3ff28daa43851..de8eb106d0237fb778e4af0f8b91eda5f751e3bf 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServerBuilder.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/NetconfCallHomeServerBuilder.java
@@ -15,7 +15,6 @@ import io.netty.util.HashedWheelTimer;
 import java.net.InetSocketAddress;
 import java.util.concurrent.TimeUnit;
 import org.apache.sshd.SshClient;
-import org.opendaylight.netconf.api.messages.NetconfHelloMessageAdditionalHeader;
 import org.opendaylight.netconf.callhome.protocol.CallHomeSessionContext.Factory;
 import org.opendaylight.netconf.client.NetconfClientSessionNegotiatorFactory;
 import org.opendaylight.yangtools.concepts.Builder;
@@ -114,7 +113,7 @@ public class NetconfCallHomeServerBuilder implements Builder<NetconfCallHomeServ
 
     private NetconfClientSessionNegotiatorFactory defaultNegotiationFactory() {
         return new NetconfClientSessionNegotiatorFactory(new HashedWheelTimer(),
-                Optional.<NetconfHelloMessageAdditionalHeader>absent(), DEFAULT_SESSION_TIMEOUT_MILLIS);
+                                                         Optional.absent(), DEFAULT_SESSION_TIMEOUT_MILLIS);
     }
 
     private EventLoopGroup defaultNettyGroup() {

diff --git a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/ReverseSshChannelInitializer.java b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/ReverseSshChannelInitializer.java
index a4fb71a17ddab6ded4e8d97a45ab007fcace98fd..09b88a944aeac94facb049bea235d0ed1c78ac87 100644 (file)
--- a/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/ReverseSshChannelInitializer.java
+++ b/netconf/callhome-protocol/src/main/java/org/opendaylight/netconf/callhome/protocol/ReverseSshChannelInitializer.java
@@ -16,7 +16,7 @@ import org.opendaylight.netconf.client.NetconfClientSessionNegotiatorFactory;
 import org.opendaylight.netconf.nettyutil.AbstractChannelInitializer;
 import org.opendaylight.protocol.framework.SessionListenerFactory;
 
-class ReverseSshChannelInitializer extends AbstractChannelInitializer<NetconfClientSession>
+final class ReverseSshChannelInitializer extends AbstractChannelInitializer<NetconfClientSession>
         implements SessionListenerFactory<NetconfClientSessionListener> {
 
     private final NetconfClientSessionNegotiatorFactory negotiatorFactory;
@@ -24,7 +24,6 @@ class ReverseSshChannelInitializer extends AbstractChannelInitializer<NetconfCli
 
     private ReverseSshChannelInitializer(NetconfClientSessionNegotiatorFactory negotiatorFactory,
                                          NetconfClientSessionListener sessionListener) {
-        super();
         this.negotiatorFactory = negotiatorFactory;
         this.sessionListener = sessionListener;
     }