Feedback

Author: JunTaoLuo

src/Microsoft.AspNetCore.ResponseCaching/Internal/LoggerExtensions.cs

@@ -50,91 +50,91 @@ static LoggerExtensions()
             _logRequestWithAuthorizationNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 2,
-                formatString: $"The request cannot be served from cache because it contains an {HeaderNames.Authorization} header.");
+                formatString: $"The request cannot be served from cache because it contains an '{HeaderNames.Authorization}' header.");
             _logRequestWithNoCacheNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 3,
-                formatString: "The request cannot be served from cache because it contains a no-cache cache directive.");
+                formatString: "The request cannot be served from cache because it contains a 'no-cache' cache directive.");
             _logRequestWithPragmaNoCacheNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 4,
-                formatString: "The request cannot be served from cache because it contains a no-cache pragma directive.");
+                formatString: "The request cannot be served from cache because it contains a 'no-cache' pragma directive.");
             _logExpirationMinFreshAdded = LoggerMessage.Define<TimeSpan>(
                 logLevel: LogLevel.Debug,
                 eventId: 5,
                 formatString: "Adding a minimum freshness requirement of {Duration}.");
             _logExpirationSharedMaxAgeExceeded = LoggerMessage.Define<TimeSpan, TimeSpan>(
                 logLevel: LogLevel.Debug,
                 eventId: 6,
-                formatString: "The age of the entry is {Age} and has exceeded the maximum age for shared caches of {SharedMaxAge} specified by s-maxage.");
+                formatString: "The age of the entry is {Age} and has exceeded the maximum age for shared caches of {SharedMaxAge} specified by the 's-maxage' cache directive.");
             _logExpirationMustRevalidate = LoggerMessage.Define<TimeSpan, TimeSpan>(
                 logLevel: LogLevel.Debug,
                 eventId: 7,
-                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by max-age. It must be revalidated becauses must-revalidate is specified.");
+                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by the 'max-age' cache directive. It must be revalidated because the 'must-revalidate' cache directive is specified.");
             _logExpirationMaxStaleSatisfied = LoggerMessage.Define<TimeSpan, TimeSpan, TimeSpan>(
                 logLevel: LogLevel.Debug,
                 eventId: 8,
-                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by max-age. However, it satisfied the maximum stale allowance of {MaxStale} specified by max-stale.");
+                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by the 'max-age' cache directive. However, it satisfied the maximum stale allowance of {MaxStale} specified by the 'max-stale' cache directive.");
             _logExpirationMaxAgeExceeded = LoggerMessage.Define<TimeSpan, TimeSpan>(
                 logLevel: LogLevel.Debug,
                 eventId: 9,
-                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by max-age.");
+                formatString: "The age of the entry is {Age} and has exceeded the maximum age of {MaxAge} specified by the 'max-age' cache directive.");
             _logExpirationExpiresExceeded = LoggerMessage.Define<DateTimeOffset, DateTimeOffset>(
                 logLevel: LogLevel.Debug,
                 eventId: 10,
-                formatString: $"The response time of the entry is {{ResponseTime}} and has exceeded the expiry date of {{Expired}} specified by the {HeaderNames.Expires} header.");
+                formatString: $"The response time of the entry is {{ResponseTime}} and has exceeded the expiry date of {{Expired}} specified by the '{HeaderNames.Expires}' header.");
             _logResponseWithoutPublicNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 11,
-                formatString: "Response is not cacheable because it does not contain the public cache directive.");
+                formatString: "Response is not cacheable because it does not contain the 'public' cache directive.");
             _logResponseWithNoStoreNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 12,
-                formatString: "Response is not cacheable because it or its corresponding request contains a no-store cache directive.");
+                formatString: "Response is not cacheable because it or its corresponding request contains a 'no-store' cache directive.");
             _logResponseWithNoCacheNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 13,
-                formatString: "Response is not cacheable because it contains a no-cache cache directive.");
+                formatString: "Response is not cacheable because it contains a 'no-cache' cache directive.");
             _logResponseWithSetCookieNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 14,
-                formatString: $"Response is not cacheable because it contains a {HeaderNames.SetCookie} header.");
+                formatString: $"Response is not cacheable because it contains a '{HeaderNames.SetCookie}' header.");
             _logResponseWithVaryStarNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 15,
-                formatString: $"Response is not cacheable because it contains a {HeaderNames.Vary} header with a value of *.");
+                formatString: $"Response is not cacheable because it contains a '{HeaderNames.Vary}' header with a value of *.");
             _logResponseWithPrivateNotCacheable = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 16,
-                formatString: "Response is not cacheable because it contains the private cache directive.");
+                formatString: "Response is not cacheable because it contains the 'private' cache directive.");
             _logResponseWithUnsuccessfulStatusCodeNotCacheable = LoggerMessage.Define<int>(
                 logLevel: LogLevel.Debug,
                 eventId: 17,
                 formatString: "Response is not cacheable because its status code {StatusCode} does not indicate success.");
             _logNotModifiedIfNoneMatchStar = LoggerMessage.Define(
                 logLevel: LogLevel.Debug,
                 eventId: 18,
-                formatString: $"The content requested has not been modified. The {HeaderNames.IfNoneMatch} header of the request contains a value of *.");
+                formatString: $"The content requested has not been modified. The '{HeaderNames.IfNoneMatch}' header of the request contains a value of *.");
             _logNotModifiedIfNoneMatchMatched = LoggerMessage.Define<EntityTagHeaderValue>(
                 logLevel: LogLevel.Debug,
                 eventId: 19,
-                formatString: $"The content requested has not been modified. The ETag {{ETag}} in the {HeaderNames.IfNoneMatch} header matched the ETag of a cached entry.");
+                formatString: $"The content requested has not been modified. The ETag {{ETag}} in the '{HeaderNames.IfNoneMatch}' header matched the ETag of a cached entry.");
             _logNotModifiedIfUnmodifiedSinceSatisfied = LoggerMessage.Define<DateTimeOffset, DateTimeOffset>(
                 logLevel: LogLevel.Debug,
                 eventId: 20,
-                formatString: $"The content requested has not been modified. The last modified date of {{LastModified}} is before the date {{IfUnmodifiedSince}} specified in the {HeaderNames.IfUnmodifiedSince} header.");
+                formatString: $"The content requested has not been modified. The last modified date of {{LastModified}} is before the date {{IfUnmodifiedSince}} specified in the '{HeaderNames.IfUnmodifiedSince}' header.");
             _logNotModifiedServed = LoggerMessage.Define(
                 logLevel: LogLevel.Information,
                 eventId: 21,
                 formatString: "The content requested has not been modified.");
             _logCachedResponseServed = LoggerMessage.Define(
                 logLevel: LogLevel.Information,
                 eventId: 22,
-                formatString: "Serving response from cache");
+                formatString: "Serving response from cache.");
             _logGatewayTimeoutServed = LoggerMessage.Define(
                 logLevel: LogLevel.Information,
                 eventId: 23,
-                formatString: "No cached response available for this request and the only-if-cached cache directive was specified.");
+                formatString: "No cached response available for this request and the 'only-if-cached' cache directive was specified.");
             _logNoResponseServed = LoggerMessage.Define(
                 logLevel: LogLevel.Information,
                 eventId: 24,
@@ -154,7 +154,7 @@ static LoggerExtensions()
             _logResponseContentLengthMismatchNotCached = LoggerMessage.Define(
                 logLevel: LogLevel.Warning,
                 eventId: 28,
-                formatString: $"The response could not be cached for this request because the {HeaderNames.ContentLength} did not match the body length.");
+                formatString: $"The response could not be cached for this request because the '{HeaderNames.ContentLength}' did not match the body length.");
         }
 
         internal static void LogRequestMethodNotCacheable(this ILogger logger, string method)