Merge pull request #2652 from 01100010011001010110010101110000/v5-handle-null-insecure-tcp

Handle clusters with only secure tcp endpoints during reconnect

Author: hayley-jean

src/EventStore.ClientAPI/Messages/ClientMessagesExtensions.cs

@@ -8,7 +8,11 @@ internal static partial class ClientMessage {
 		public partial class NotHandled {
 			public partial class MasterInfo {
 				public IPEndPoint ExternalTcpEndPoint {
-					get { return new IPEndPoint(IPAddress.Parse(ExternalTcpAddress), ExternalTcpPort); }
+					get {
+						return ExternalTcpAddress == null
+							? null
+							: new IPEndPoint(IPAddress.Parse(ExternalTcpAddress), ExternalTcpPort);
+					}
 				}
 
 				public IPEndPoint ExternalSecureTcpEndPoint {

src/EventStore.ClientAPI/SystemData/InspectionResult.cs

@@ -11,8 +11,11 @@ internal class InspectionResult {
 
 		public InspectionResult(InspectionDecision decision, string description, IPEndPoint tcpEndPoint = null,
 			IPEndPoint secureTcpEndPoint = null) {
-			if (decision == InspectionDecision.Reconnect)
-				Ensure.NotNull(tcpEndPoint, "tcpEndPoint");
+			if (decision == InspectionDecision.Reconnect) {
+				if (tcpEndPoint == null && secureTcpEndPoint == null) {
+					throw new ArgumentException("tcpEndPoint and secureTcpEndPoint cannot both be null");
+				}
+			}
 			else {
 				if (tcpEndPoint != null)
 					throw new ArgumentException(string.Format("tcpEndPoint is not null for decision {0}.", decision));