Use Uri.AbsolteUri instead of Uri.ToString().

kpreisser · Nov 25, 2020 · 2a2b441 · 2a2b441
1 parent 904b8dc
commit 2a2b441
Showing 1 changed file with 5 additions and 2 deletions.
diff --git a/SimpleSocketClient/UiSocketHandler.cs b/SimpleSocketClient/UiSocketHandler.cs
@@ -99,14 +99,17 @@ public void Start()
         {
             if (this.isWebsocket)
             {
+                // Use Uri.AbsoluteUri instead of Uri.ToString() as the latter unescapes
+                // characters and thus could return a string e.g. with spaces and non-ASCII
+                // characters.
                 this.OnSocketMessage(new SocketMessageEventArgs(
-                    $"Connecting to WebSocket URL '{this.webSocketUrl}'…",
+                    $"Connecting to WebSocket URL '{this.webSocketUrl!.AbsoluteUri}'…",
                     isMetaText: true));
             }
             else
             {
                 this.OnSocketMessage(new SocketMessageEventArgs(
-                    $"Connecting to '{host}:{port}'…",
+                    $"Connecting to '{this.host}:{this.port}'…",
                     isMetaText: true));
             }