Fix possibly null references

nager · Nov 7, 2024 · 6ec9a5a · 6ec9a5a
1 parent ecfc69b
commit 6ec9a5a
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 4 deletions.
diff --git a/src/Nager.TcpClient.UnitTest/SendAfterConnectedEventTest.cs b/src/Nager.TcpClient.UnitTest/SendAfterConnectedEventTest.cs
@@ -31,7 +31,7 @@ public void SendAfterConnect_Successful()
                 x => x.Log(
                     LogLevel.Debug,
                     It.IsAny<EventId>(),
-                    It.Is<It.IsAnyType>((o, t) => o.ToString().Contains("SendAsync")),
+                    It.Is<It.IsAnyType>((o, t) => (o.ToString() ?? string.Empty).Contains("SendAsync")),
                     It.IsAny<Exception>(),
                     It.IsAny<Func<It.IsAnyType, Exception?, string>>()),
                 Times.Once);
@@ -52,7 +52,7 @@ public async Task SendAfterConnectAsync_Successful()
                 x => x.Log(
                     LogLevel.Debug,
                     It.IsAny<EventId>(),
-                    It.Is<It.IsAnyType>((o, t) => o.ToString().Contains("SendAsync")),
+                    It.Is<It.IsAnyType>((o, t) => (o.ToString() ?? string.Empty).Contains("SendAsync")),
                     It.IsAny<Exception>(),
                     It.IsAny<Func<It.IsAnyType, Exception?, string>>()),
                 Times.Once);

diff --git a/src/Nager.TcpClient/Nager.TcpClient.csproj b/src/Nager.TcpClient/Nager.TcpClient.csproj
@@ -22,7 +22,7 @@
 
     <TargetFrameworks>netstandard2.0;netstandard2.1;net6;net8</TargetFrameworks>
 
-    <Version>1.3.0</Version>
+    <Version>1.3.1</Version>
   </PropertyGroup>
 
   <ItemGroup>

diff --git a/src/Nager.TcpClient/TcpClient.cs b/src/Nager.TcpClient/TcpClient.cs
@@ -580,8 +580,13 @@ await Task
             this._logger.LogInformation($"{nameof(DataReceiverAsync)} - Stopped");
         }
 
-        private bool IsKnownException(Exception exception)
+        private bool IsKnownException(Exception? exception)
         {
+            if (exception == null)
+            {
+                return false;
+            }
+
             if (exception is AggregateException aggregateException)
             {
                 if (aggregateException.InnerException is IOException ioException)