Updated SqlClient to 5.0.1

Breaking change: "Encrypt=False" must now be explicitly set in connection string if no encryption should be used. Otherwise an SqlException will occurr.

Details: https://github.com/dotnet/SqlClient/blob/main/release-notes/4.0/4.0.0.md#breaking-changes

Author: ckadluba

README.md

@@ -19,6 +19,7 @@ A Serilog sink that writes events to Microsoft SQL Server. This sink will write
 * [External Configuration Syntax](#external-configuration-syntax)
 * [Troubleshooting](#troubleshooting)
 * [Querying Property Data](#querying-property-data)
+* [Breaking Changes](#breaking-changes)
 * [Deprecated Features](#deprecated-features)
 
 ## Quick Start
@@ -790,6 +791,12 @@ WHERE
   [Properties].value('(//property[@key="Controller"]/node())[1]', 'nvarchar(max)') = 'UserController'
 ```
 
+## Breaking Changes
+
+### Release 6.0.0
+
+Micosoft.Data.SqlClient was upgraded to >4.0.0 which introduces a breaking change regarding the connection strings. If your SQL Server does not use encryption you have to explicitly specify this in the connection string by adding `Encrypt=False`. Otherwise the connection will fail with an SqlException. Refer to the [SqlClient documentation](https://github.com/dotnet/SqlClient/blob/main/release-notes/4.0/4.0.0.md#breaking-changes) for details.
+
 ## Deprecated Features
 
 Feature | Notes

sample/AppConfigDemo/Program.cs

@@ -8,7 +8,7 @@ namespace AppConfigDemo
 {
     public static class Program
     {
-        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
         private const string _schemaName = "dbo";
         private const string _tableName = "LogEvents";
 

sample/CombinedConfigDemo/appsettings.json

@@ -7,7 +7,7 @@
     }
   },
   "ConnectionStrings": {
-    "LogDatabase": "Server=localhost;Database=LogTest;Integrated Security=SSPI;"
+    "LogDatabase": "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;"
   },
   "Serilog": {
     "SinkOptions": {

sample/CustomLogEventFormatterDemo/Program.cs

@@ -7,7 +7,7 @@ namespace CustomLogEventFormatterDemo
 {
     public static class Program
     {
-        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
         private const string _schemaName = "dbo";
         private const string _tableName = "LogEvents";
 

sample/NetStandardDemo/NetStandardDemoLib/Initializer.cs

@@ -8,7 +8,7 @@ namespace NetStandardDemoLib
 {
     public static class Initializer
     {
-        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
         private const string _tableName = "LogEvents";
 
         public static LoggerConfiguration CreateLoggerConfiguration()

sample/WorkerServiceDemo/appsettings.json

@@ -13,7 +13,7 @@
       {
         "Name": "MSSqlServer",
         "Args": {
-          "connectionString": "Server=localhost;Database=LogTest;Integrated Security=SSPI;",
+          "connectionString": "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;",
           "sinkOptionsSection": {
             "tableName": "LogEvents",
             "autoCreateSqlTable": true
@@ -42,7 +42,7 @@
       {
         "Name": "MSSqlServer",
         "Args": {
-          "connectionString": "Server=localhost;Database=LogTest;Integrated Security=SSPI;",
+          "connectionString": "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;",
           "restrictedToMinimumLevel": "Information",
           "logEventFormatter": "WorkerServiceDemo.CustomLogEventFormatter::Formatter, WorkerServiceDemo",
           "sinkOptionsSection": {

src/Serilog.Sinks.MSSqlServer/Serilog.Sinks.MSSqlServer.csproj

@@ -32,7 +32,7 @@
 
   <ItemGroup>
     <PackageReference Include="Microsoft.SourceLink.GitHub" Version="1.1.1" PrivateAssets="All" />
-    <PackageReference Include="Microsoft.Data.SqlClient" Version="3.1.1" />
+    <PackageReference Include="Microsoft.Data.SqlClient" Version="5.0.1" />
     <PackageReference Include="Microsoft.Extensions.Configuration" Version="6.0.1" />
     <PackageReference Include="Microsoft.Extensions.Options.ConfigurationExtensions" Version="6.0.0" />
     <PackageReference Include="Serilog" Version="2.12.0" />

test/Serilog.Sinks.MSSqlServer.Tests/Configuration/Implementations/Microsoft.Extensions.Configuration/MicrosoftExtensionsConnectionStringProviderTests.cs

@@ -13,7 +13,7 @@ public class MicrosoftExtensionsConnectionStringProviderTests
         public void GetConnectionStringCalledWithConnectionStringReturnsSameValue()
         {
             // Arrange
-            const string connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+            const string connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
             var configurationMock = new Mock<IConfiguration>();
             var sut = new MicrosoftExtensionsConnectionStringProvider();
 
@@ -29,7 +29,7 @@ public void GetConnectionStringCalledWithNameItGetsConnectionStringFromConfig()
         {
             // Arrange
             const string connectionStringName = "LogDatabase";
-            const string connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+            const string connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
             var configurationMock = new Mock<IConfiguration>();
             var configSectionMock = new Mock<IConfigurationSection>();
             configurationMock.Setup(c => c.GetSection(It.IsAny<string>())).Returns(configSectionMock.Object);

test/Serilog.Sinks.MSSqlServer.Tests/Sinks/MSSqlServer/Dependencies/SinkDependenciesFactoryTests.cs

@@ -8,7 +8,7 @@ namespace Serilog.Sinks.MSSqlServer.Tests.Dependencies
     [Trait(TestCategory.TraitName, TestCategory.Unit)]
     public class SinkDependenciesFactoryTests
     {
-        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;";
+        private const string _connectionString = "Server=localhost;Database=LogTest;Integrated Security=SSPI;Encrypt=False;";
         private readonly MSSqlServerSinkOptions _sinkOptions;
         private readonly Serilog.Sinks.MSSqlServer.ColumnOptions _columnOptions;