fix(WSConnector): Removed http case for changing URLs, added pass through for unhandled URLs, Added warning to use US South instance or .NET 4.6

Author: mediumTaj

Scripts/Connection/WSConnector.cs

@@ -22,11 +22,8 @@
 using IBM.Watson.DeveloperCloud.Utilities;
 using System.Collections;
 using System.Collections.Generic;
-using System.Threading;
-#if UNITY_2018_2_OR_NEWER
 using System.Security.Authentication;
-#endif
-
+using System.Threading;
 #if !NETFX_CORE
 using UnitySDK.WebSocketSharp;
 #else
@@ -205,99 +202,74 @@ public static string FixupURL(string URL)
 #if UNITY_2018_2_OR_NEWER
 #if NET_4_6
             //  Use standard endpoints since 2018.2 supports TLS 1.2
-            if (URL.StartsWith("http://stream."))
-            {
-                URL = URL.Replace("http://stream.", "ws://stream.");
-            }
-            else if (URL.StartsWith("https://stream."))
+            if (URL.StartsWith("https://stream."))
             {
                 URL = URL.Replace("https://stream.", "wss://stream.");
             }
 
             //  TLS 1.0 endpoint - Do not change this to TLS 1.2 endpoint since
             //  users may need to use the TLS 1.0 endpoint because of different
             //  platforms.
-            else if (URL.StartsWith("http://stream-tls10."))
-            {
-                URL = URL.Replace("http://stream-tls10.", "ws://stream-tls10.");
-            }
             else if (URL.StartsWith("https://stream-tls10."))
             {
                 URL = URL.Replace("https://stream-tls10.", "wss://stream-tls10.");
             }
-
             //  Germany
-            else if (URL.StartsWith("http://gateway-fra."))
-            {
-                URL = URL.Replace("http://gateway-fra.", "ws://stream-fra.");
-            }
             else if (URL.StartsWith("https://gateway-fra."))
             {
                 URL = URL.Replace("https://gateway-fra.", "wss://stream-fra.");
             }
-
             //  US East
-            else if (URL.StartsWith("http://gateway-wdc."))
-            {
-                URL = URL.Replace("http://gateway-wdc.", "ws://gateway-wdc.");
-            }
             else if (URL.StartsWith("https://gateway-wdc."))
             {
                 URL = URL.Replace("https://gateway-wdc.", "wss://gateway-wdc.");
             }
-
-
             //  Sydney
-            else if (URL.StartsWith("http://gateway-syd."))
-            {
-                URL = URL.Replace("http://gateway-syd.", "ws://gateway-syd.");
-            }
             else if (URL.StartsWith("https://gateway-syd."))
             {
                 URL = URL.Replace("https://gateway-syd.", "wss://gateway-syd.");
             }
-
             else
             {
-                Log.Warning("WSConnector", "No case for URL for wss://. Leaving URL unchanged.");
+                URL = URL.Replace("https://", "wss://");
+                Log.Warning("WSConnector", "No case for URL for wss://. Replacing https:// with wss://.");
             }
 #else
             //  Use TLS 1.0 endpoint if user is on .NET 3.5. US South is the 
             //  only region that supports this endpoint.
-            if (URL.StartsWith("http://stream."))
-            {
-                URL = URL.Replace("http://stream.", "ws://stream-tls10.");
-            }
-            else if (URL.StartsWith("https://stream."))
+            if (URL.StartsWith("https://stream."))
             {
                 URL = URL.Replace("https://stream.", "wss://stream-tls10.");
             }
-            else if (URL.StartsWith("http://stream-tls10."))
-            {
-                URL = URL.Replace("http://stream-tls10.", "ws://stream-tls10.");
-            }
             else if (URL.StartsWith("https://stream-tls10."))
             {
                 URL = URL.Replace("https://stream-tls10.", "wss://stream-tls10.");
             }
+            else
+            {
+                URL = URL.Replace("https://", "wss://");
+                Log.Warning("WSConnector", "No case for URL for wss://. Replacing https:// with wss://.");
+                Log.Warning("WSConnector", "Streaming with TLS 1.0 is only available in US South. Please create your Speech to Text instance in US South. Alternatviely, use Unity 2018.2 with .NET 4.x Scripting Runtime Version enabled (File > Build Settings > Player Settings > Other Settings > Scripting Runtime Version).");
+            }
 #endif
 #else
-            //  Redirect to TLS 1.0 endpoints. 
-            //  Note frankfurt endpoint does not support TLS 1.0.
-            if (URL.StartsWith("http://stream."))
-                URL = URL.Replace("http://stream.", "ws://stream-tls10.");
-            else if (URL.StartsWith("https://stream."))
+            //  Use TLS 1.0 endpoint if user is on .NET 3.5 or 4.6 if using Unity 2018.1 or older.
+            //  US South is the only region that supports this endpoint.
+            if (URL.StartsWith("https://stream."))
+            {
                 URL = URL.Replace("https://stream.", "wss://stream-tls10.");
-            else if (URL.StartsWith("http://stream-tls10."))
-                URL = URL.Replace("http://stream-tls10.", "ws://stream-tls10.");
+            }
             else if (URL.StartsWith("https://stream-tls10."))
+            {
                 URL = URL.Replace("https://stream-tls10.", "wss://stream-tls10.");
-            else if (URL.StartsWith("http://stream-fra."))
-                URL = URL.Replace("http://stream-fra.", "ws://stream-fra.");
-            else if (URL.StartsWith("https://stream-fra."))
-                URL = URL.Replace("https://stream-fra.", "wss://stream-fra.");
+            }
+            else
+            {
+                URL = URL.Replace("https://", "wss://");
+                Log.Warning("WSConnector", "No case for URL for wss://. Replacing https:// with wss://.");
+                Log.Warning("WSConnector", "Streaming with TLS 1.0 is only available in US South. Please create your Speech to Text instance in US South. Alternatviely, use Unity 2018.2 with .NET 4.x Scripting Runtime Version enabled (File > Build Settings > Player Settings > Other Settings > Scripting Runtime Version).");
+            }
 #endif
-
             return URL;
         }
 
@@ -330,7 +302,7 @@ public static WSConnector CreateConnector(Credentials credentials, string functi
             return connector;
         }
 
-        #region Public Functions
+#region Public Functions
         /// <summary>
         /// This function sends the given message object.
         /// </summary>
@@ -382,9 +354,9 @@ public void Close()
             // setting the state to closed will make the SendThread automatically exit.
             _connectionState = ConnectionState.CLOSED;
         }
-        #endregion
+#endregion
 
-        #region Private Functions
+#region Private Functions
         private IEnumerator ProcessReceiveQueue()
         {
             while (_connectionState == ConnectionState.CONNECTED
@@ -415,9 +387,9 @@ private IEnumerator ProcessReceiveQueue()
             if (OnClose != null)
                 OnClose(this);
         }
-        #endregion
+#endregion
 
-        #region Threaded Functions
+#region Threaded Functions
         // NOTE: All functions in this region are operating in a background thread, do NOT call any Unity functions!
 #if !NETFX_CORE
         private void SendMessages()
@@ -616,6 +588,6 @@ private void WebSocket_MessageReceived(MessageWebSocket sender, MessageWebSocket
             }
         }
 #endif
-        #endregion
+#endregion
     }
 }