From 5b2a1411644980f10b6d826155752f85e40bdd78 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Wed, 25 Sep 2013 08:39:41 +0000
Subject: [PATCH] OPENDJ-1134 (CR-2355) Introduce a class in replication for encapsulating host+port combinations

---
 opends/src/server/org/opends/server/replication/service/ReplicationBroker.java |   24 +++++++++++-------------
 1 files changed, 11 insertions(+), 13 deletions(-)

diff --git a/opends/src/server/org/opends/server/replication/service/ReplicationBroker.java b/opends/src/server/org/opends/server/replication/service/ReplicationBroker.java
index f9e81c0..dba65da 100644
--- a/opends/src/server/org/opends/server/replication/service/ReplicationBroker.java
+++ b/opends/src/server/org/opends/server/replication/service/ReplicationBroker.java
@@ -403,35 +403,37 @@
     private long generationId;
     private byte groupId = -1;
     private int serverId;
-    // Received server URL
+    /** Received server URL. */
     private String serverURL;
     private String baseDn = null;
     private int windowSize;
     private ServerState serverState = null;
     private boolean sslEncryption;
     private int degradedStatusThreshold = -1;
-    // Keeps the 1 value if created with a ReplServerStartMsg
+    /** Keeps the 1 value if created with a ReplServerStartMsg. */
     private int weight = 1;
-    // Keeps the 0 value if created with a ReplServerStartMsg
+    /** Keeps the 0 value if created with a ReplServerStartMsg. */
     private int connectedDSNumber = 0;
     private List<Integer> connectedDSs = null;
-    // Is this RS locally configured ? (the RS is recognized as a usable server)
+    /**
+     * Is this RS locally configured? (the RS is recognized as a usable server).
+     */
     private boolean locallyConfigured = true;
 
     /**
      * Create a new instance of ReplicationServerInfo wrapping the passed
      * message.
      * @param msg Message to wrap.
-     * @param server Override serverURL.
+     * @param newServerURL Override serverURL.
      * @return The new instance wrapping the passed message.
      * @throws IllegalArgumentException If the passed message has an unexpected
      *                                  type.
      */
     public static ReplicationServerInfo newInstance(
-      ReplicationMsg msg, String server) throws IllegalArgumentException
+      ReplicationMsg msg, String newServerURL) throws IllegalArgumentException
     {
       ReplicationServerInfo rsInfo = newInstance(msg);
-      rsInfo.serverURL = server;
+      rsInfo.serverURL = newServerURL;
       return rsInfo;
     }
 
@@ -1152,17 +1154,13 @@
     try
     {
       // Open a socket connection to the next candidate.
-      final HostPort hp = HostPort.valueOf(server);
-      InetSocketAddress serverAddr = new InetSocketAddress(
-          InetAddress.getByName(hp.getHost()), hp.getPort());
       socket = new Socket();
       socket.setReceiveBufferSize(1000000);
       socket.setTcpNoDelay(true);
       int timeoutMS = MultimasterReplication.getConnectionTimeoutMS();
-      socket.connect(serverAddr, timeoutMS);
+      socket.connect(HostPort.valueOf(server).toInetSocketAddress(), timeoutMS);
       localSession = replSessionSecurity.createClientSession(socket, timeoutMS);
-      boolean isSslEncryption = replSessionSecurity
-          .isSslEncryption(server);
+      boolean isSslEncryption = replSessionSecurity.isSslEncryption();
 
       // Send our ServerStartMsg.
       String url = socket.getLocalAddress().getHostName() + ":"

--
Gitblit v1.10.0