IsClientConnected fix from CavemanTcp

jchristn · Jul 4, 2023 · 94677a6 · 94677a6
1 parent 378a554
commit 94677a6
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/src/SuperSimpleTcp/SimpleTcpClient.cs b/src/SuperSimpleTcp/SimpleTcpClient.cs
@@ -1131,11 +1131,13 @@ private void EnableKeepalives()
 #if NETCOREAPP3_1_OR_GREATER || NET6_0_OR_GREATER
 
                 // NETCOREAPP3_1_OR_GREATER catches .NET 5.0
+
                 _client.Client.SetSocketOption(SocketOptionLevel.Socket, SocketOptionName.KeepAlive, true);
                 _client.Client.SetSocketOption(SocketOptionLevel.Tcp, SocketOptionName.TcpKeepAliveTime, _keepalive.TcpKeepAliveTime);
                 _client.Client.SetSocketOption(SocketOptionLevel.Tcp, SocketOptionName.TcpKeepAliveInterval, _keepalive.TcpKeepAliveInterval);
 
                 // Windows 10 version 1703 or later
+
                 if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows)
                     && Environment.OSVersion.Version >= new Version(10, 0, 15063))
                 {

diff --git a/src/SuperSimpleTcp/SimpleTcpServer.cs b/src/SuperSimpleTcp/SimpleTcpServer.cs
@@ -1192,13 +1192,16 @@ private void EnableKeepalives(TcpClient client)
         {
             try
             {
-#if NETCOREAPP || NET5_0_OR_GREATER
+#if NETCOREAPP3_1_OR_GREATER || NET6_0_OR_GREATER
+
+                // NETCOREAPP3_1_OR_GREATER catches .NET 5.0
 
                 client.Client.SetSocketOption(SocketOptionLevel.Socket, SocketOptionName.KeepAlive, true);
                 client.Client.SetSocketOption(SocketOptionLevel.Tcp, SocketOptionName.TcpKeepAliveTime, _keepalive.TcpKeepAliveTime);
                 client.Client.SetSocketOption(SocketOptionLevel.Tcp, SocketOptionName.TcpKeepAliveInterval, _keepalive.TcpKeepAliveInterval);
 
                 // Windows 10 version 1703 or later
+
                 if (RuntimeInformation.IsOSPlatform(OSPlatform.Windows)
                     && Environment.OSVersion.Version >= new Version(10, 0, 15063))
                 {