From 34b97b2d33a1196d3b22bd0ade1e758be613f7f3 Mon Sep 17 00:00:00 2001
From: Jean-Noel Rouvignac <jean-noel.rouvignac@forgerock.com>
Date: Fri, 06 Jun 2014 14:37:42 +0000
Subject: [PATCH] Ensured no accidental protocol break can happen by renaming all ByteArrayBuilder.append*() methods and getting rid of all method overloading.
---
opendj3-server-dev/src/server/org/opends/server/replication/protocol/ReplServerStartDSMsg.java | 21 ++++++++++-----------
1 files changed, 10 insertions(+), 11 deletions(-)
diff --git a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/ReplServerStartDSMsg.java b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/ReplServerStartDSMsg.java
index c201f19..3e13c4d 100644
--- a/opendj3-server-dev/src/server/org/opends/server/replication/protocol/ReplServerStartDSMsg.java
+++ b/opendj3-server-dev/src/server/org/opends/server/replication/protocol/ReplServerStartDSMsg.java
@@ -131,7 +131,7 @@
degradedStatusThreshold =scanner.nextIntUTF8();
weight = scanner.nextIntUTF8();
connectedDSNumber = scanner.nextIntUTF8();
- serverState = scanner.nextServerState();
+ serverState = scanner.nextServerStateMustComeLast();
}
/**
@@ -182,16 +182,15 @@
*/
final ByteArrayBuilder builder = new ByteArrayBuilder();
encodeHeader(MSG_TYPE_REPL_SERVER_START_DS, builder, protocolVersion);
- builder.append(baseDN);
- builder.appendUTF8(serverId);
- builder.append(serverURL);
- builder.appendUTF8(windowSize);
- builder.append(Boolean.toString(sslEncryption));
- builder.appendUTF8(degradedStatusThreshold);
- builder.appendUTF8(weight);
- builder.appendUTF8(connectedDSNumber);
- // Caution: ServerState MUST be the last field.
- builder.append(serverState);
+ builder.appendDN(baseDN);
+ builder.appendIntUTF8(serverId);
+ builder.appendString(serverURL);
+ builder.appendIntUTF8(windowSize);
+ builder.appendString(Boolean.toString(sslEncryption));
+ builder.appendIntUTF8(degradedStatusThreshold);
+ builder.appendIntUTF8(weight);
+ builder.appendIntUTF8(connectedDSNumber);
+ builder.appendServerStateMustComeLast(serverState);
return builder.toByteArray();
}
--
Gitblit v1.10.0