include attempted bind address in DotNetty binding failure messages (#…

…5132) * close #5130 - include attempted bind address in binding failure messages * forgot to add semicolon * added inner exception back * added note about InnerException containing descriptive socket error
akkadotnet · Jul 6, 2021 · 0a21bc2 · 0a21bc2
1 parent 951be4a
commit 0a21bc2
Showing 1 changed file with 11 additions and 4 deletions.
diff --git a/src/core/Akka.Remote/Transport/DotNetty/DotNettyTransport.cs b/src/core/Akka.Remote/Transport/DotNetty/DotNettyTransport.cs
@@ -159,12 +159,19 @@ protected async Task<IChannel> NewServer(EndPoint listenAddress)
             if (InternalTransport != TransportMode.Tcp)
                 throw new NotImplementedException("Haven't implemented UDP transport at this time");
 
-            if (listenAddress is DnsEndPoint dns)
+            try
             {
-                listenAddress = await DnsToIPEndpoint(dns).ConfigureAwait(false);
-            }
+                if (listenAddress is DnsEndPoint dns)
+                {
+                    listenAddress = await DnsToIPEndpoint(dns).ConfigureAwait(false);
+                }
 
-            return await ServerFactory().BindAsync(listenAddress).ConfigureAwait(false);
+                return await ServerFactory().BindAsync(listenAddress).ConfigureAwait(false);
+            }
+            catch (Exception ex)
+            {
+                throw new RemoteTransportException($"Failed to bind to [{listenAddress}]. See InnerException for details.", ex);
+            }
         }
 
         public override async Task<(Address, TaskCompletionSource<IAssociationEventListener>)> Listen()