fix: use hostname for serverUri

MirageNet · Oct 15, 2020 · 4b38fbe · 4b38fbe
1 parent 78e2197
commit 4b38fbe
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 3 deletions.
diff --git a/Assets/Mirror/Runtime/Transport/Kcp/KcpTransport.cs b/Assets/Mirror/Runtime/Transport/Kcp/KcpTransport.cs
@@ -14,8 +14,6 @@ public class KcpTransport : Transport
         [Header("Transport Configuration")]
         public ushort Port = 7777;
 
-        [SerializeField] private string _bindAddress = "localhost";
-
         internal readonly Dictionary<IPEndPoint, KcpServerConnection> connectedClients = new Dictionary<IPEndPoint, KcpServerConnection>(new IPEndpointComparer());
         readonly Channel<KcpServerConnection> acceptedConnections = Channel.CreateSingleConsumerUnbounded<KcpServerConnection>();
 
@@ -97,7 +95,7 @@ public override IEnumerable<Uri> ServerUri()
             var builder = new UriBuilder
             {
                 Scheme = "kcp",
-                Host = _bindAddress,
+                Host = Dns.GetHostName(),
                 Port = Port
             };
             return new[] { builder.Uri };

diff --git a/Assets/Tests/Runtime/Transport/TransportTests.cs b/Assets/Tests/Runtime/Transport/TransportTests.cs
@@ -10,10 +10,12 @@
 using Object = UnityEngine.Object;
 using System.Linq;
 using Cysharp.Threading.Tasks;
+using Mirror.KCP;
 
 namespace Mirror.Tests
 {
     [TestFixture(typeof(TcpTransport), new[] { "tcp4" }, "tcp4://localhost", 7777)]
+    [TestFixture(typeof(KcpTransport), new[] { "kcp" }, "kcp://localhost", 7777)]
     public class AsyncTransportTests<T> where T : Transport
     {
         #region SetUp