Merge pull request #569 from caioavidal/fix/socket-connection-error

fix: adjustments to intermittent socket error on unexpected client closed

src/NetworkingServer/NeoServer.Networking.Packets/Connection/Connection.cs

@@ -22,7 +22,7 @@ public class Connection : IConnection
     private readonly ILogger _logger;
 
     private readonly Socket _socket;
-    private readonly Stream _stream;
+    private readonly NetworkStream _stream;
     private readonly object _writeLock;
 
     public Connection(Socket socket, ILogger logger)
@@ -246,7 +246,7 @@ private bool CompleteRead(IAsyncResult ar)
 
                 while (totalBytesRead < size)
                 {
-                    if (!_stream.CanRead)
+                    if (!_stream.CanRead || !_stream.DataAvailable)
                     {
                         return false;
                     }
@@ -262,6 +262,11 @@ private bool CompleteRead(IAsyncResult ar)
 
             return true;
         }
+
+        catch (ObjectDisposedException)
+        {
+            // this exception is expected when the clientListener got disposed. In this case we don't want to spam the log.
+        }
         catch (Exception ex)
         {
             _logger.Error(ex, "Unable to complete stream read");