diff --git a/src/JsonApiDotNetCore/QueryStrings/QueryStringReader.cs b/src/JsonApiDotNetCore/QueryStrings/QueryStringReader.cs index 8e58d1a1ff..89337708e7 100644 --- a/src/JsonApiDotNetCore/QueryStrings/QueryStringReader.cs +++ b/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.")] diff --git a/test/TestBuildingBlocks/XUnitLogHttpMessageHandler.cs b/test/TestBuildingBlocks/XUnitLogHttpMessageHandler.cs index 514ba19c51..4b981f81d2 100644 --- a/test/TestBuildingBlocks/XUnitLogHttpMessageHandler.cs +++ b/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; ///