Add custom exception type for insecure remoting

src/Hosting.Services.Remoting/RemotingListenerBuilder.cs

@@ -1,8 +1,8 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
-using System;
 using System.Fabric;
+using Microsoft.Omex.Extensions.Services.Remoting;
 using Microsoft.Omex.Extensions.Services.Remoting.Runtime;
 using Microsoft.ServiceFabric.Services.Communication.Runtime;
 using Microsoft.ServiceFabric.Services.Remoting;
@@ -33,7 +33,7 @@ protected RemotingListenerBuilder(
 		{
 			if (settings == null && !FabricTransportRemotingListenerSettings.TryLoadFrom("TransportSettings", out settings))
 			{
-				throw new InvalidOperationException("Transport security is required for Service Fabric Remoting connections. TransportSettings must be defined in settings.xml.");
+				throw new InsecureRemotingUnsupportedException();
 			}
 
 			settings.ExceptionSerializationTechnique = FabricTransportRemotingListenerSettings.ExceptionSerialization.Default;

src/Services.Remoting/Client/OmexServiceProxyFactory.cs

@@ -1,7 +1,6 @@
 // Copyright (c) Microsoft Corporation. All rights reserved.
 // Licensed under the MIT license.
 
-using System;
 using Microsoft.ServiceFabric.Services.Remoting.Client;
 using Microsoft.ServiceFabric.Services.Remoting.FabricTransport;
 using Microsoft.ServiceFabric.Services.Remoting.V2.FabricTransport.Client;
@@ -24,7 +23,7 @@ public static class OmexServiceProxyFactory
 		{
 			if (!FabricTransportRemotingSettings.TryLoadFrom("TransportSettings", out FabricTransportRemotingSettings remotingSettings))
 			{
-				throw new InvalidOperationException("Transport security is required for Service Fabric Remoting connections. TransportSettings must be defined in settings.xml.");
+				throw new InsecureRemotingUnsupportedException();
 			}
 
 			remotingSettings.ExceptionDeserializationTechnique = FabricTransportRemotingSettings.ExceptionDeserialization.Default;

src/Services.Remoting/InsecureRemotingUnsupportedException.cs

@@ -0,0 +1,22 @@
+// Copyright (c) Microsoft Corporation. All rights reserved.
+// Licensed under the MIT license.
+
+using System;
+
+namespace Microsoft.Omex.Extensions.Services.Remoting
+{
+	/// <summary>
+	/// Specific exception type used when an attempt to create an insecure listener or insecure proxy is made.
+	/// </summary>
+	[Serializable]
+	public class InsecureRemotingUnsupportedException : InvalidOperationException
+	{
+		/// <summary>
+		/// Creates an instance of <see cref="InsecureRemotingUnsupportedException"/>.
+		/// </summary>
+		public InsecureRemotingUnsupportedException()
+			: base("Transport security is required for Service Fabric Remoting connections. TransportSettings must be defined in settings.xml.")
+		{
+		}
+	}
+}

tests/Hosting.Services.Remoting.UnitTests/GenericRemotingListenerBuilderTests.cs

@@ -3,6 +3,7 @@
 
 using System;
 using Microsoft.Omex.Extensions.Hosting.Services.UnitTests;
+using Microsoft.Omex.Extensions.Services.Remoting;
 using Microsoft.ServiceFabric.Services.Remoting;
 using Microsoft.ServiceFabric.Services.Remoting.FabricTransport.Runtime;
 using Microsoft.VisualStudio.TestTools.UnitTesting;
@@ -21,7 +22,7 @@ public void GenericRemotingListenerBuilder_NoTransportSettigns_ThrowsException()
 			IServiceProvider mockProvider = new Mock<IServiceProvider>().Object;
 			OmexStatefulService omexStatefulService = MockServiceFabricServices.MockOmexStatefulService;
 
-			Assert.ThrowsException<InvalidOperationException>(() => new GenericRemotingListenerBuilder<OmexStatefulService>(name, mockProvider,
+			Assert.ThrowsException<InsecureRemotingUnsupportedException>(() => new GenericRemotingListenerBuilder<OmexStatefulService>(name, mockProvider,
 					(p, s) =>
 					{
 						Assert.AreEqual(omexStatefulService, s);