Partially revert changes for CA1873 (Evaluation of this argument may be expensive and unnecessary if logging is disabled), now that it's not so rigid anymore since .NET 10 SDK v10.0.102

Author: bkoelman

src/JsonApiDotNetCore/QueryStrings/QueryStringReader.cs

@@ -47,11 +47,7 @@ public void ReadAll(DisableQueryStringAttribute? disableQueryStringAttribute)
 
             if (reader != null)
             {
-                if (_logger.IsEnabled(LogLevel.Debug))
-                {
-                    string readerType = reader.GetType().Name;
-                    LogParameterAccepted(parameterName, parameterValue, readerType);
-                }
+                LogParameterAccepted(parameterName, parameterValue, reader.GetType().Name);
 
                 if (!reader.AllowEmptyValue && string.IsNullOrEmpty(parameterValue))
                 {
@@ -78,8 +74,7 @@ public void ReadAll(DisableQueryStringAttribute? disableQueryStringAttribute)
         }
     }
 
-    [LoggerMessage(Level = LogLevel.Debug, SkipEnabledCheck = true,
-        Message = "Query string parameter '{ParameterName}' with value '{ParameterValue}' was accepted by {ReaderType}.")]
+    [LoggerMessage(Level = LogLevel.Debug, Message = "Query string parameter '{ParameterName}' with value '{ParameterValue}' was accepted by {ReaderType}.")]
     private partial void LogParameterAccepted(string parameterName, StringValues parameterValue, string readerType);
 
     [LoggerMessage(Level = LogLevel.Debug, Message = "Query string parameter '{ParameterName}' was successfully read.")]

test/TestBuildingBlocks/XUnitLogHttpMessageHandler.cs

@@ -1,8 +1,6 @@
 using Microsoft.Extensions.Logging;
 using Xunit.Abstractions;
 
-#pragma warning disable CA1873 // Avoid potentially expensive logging
-
 namespace TestBuildingBlocks;
 
 /// <summary>