PR feedback

Author: sebastienros

src/Middleware/OutputCaching/src/Policies/NoLookupPolicy.cs

@@ -6,7 +6,7 @@ namespace Microsoft.AspNetCore.OutputCaching;
 /// <summary>
 /// A policy that prevents the response from being served from cached.
 /// </summary>
-internal class NoLookupPolicy : IOutputCachePolicy
+internal sealed class NoLookupPolicy : IOutputCachePolicy
 {
     public static NoLookupPolicy Instance = new();
 

src/Middleware/OutputCaching/src/Policies/NoStorePolicy.cs

@@ -6,7 +6,7 @@ namespace Microsoft.AspNetCore.OutputCaching;
 /// <summary>
 /// A policy that prevents the response from being cached.
 /// </summary>
-internal class NoStorePolicy : IOutputCachePolicy
+internal sealed class NoStorePolicy : IOutputCachePolicy
 {
     public static NoStorePolicy Instance = new();
 

src/Middleware/OutputCaching/src/Policies/VaryByHeaderPolicy.cs

@@ -10,7 +10,7 @@ namespace Microsoft.AspNetCore.OutputCaching;
 /// </summary>
 internal sealed class VaryByHeaderPolicy : IOutputCachePolicy
 {
-    private StringValues _headers { get; set; }
+    private readonly StringValues _headers;
 
     /// <summary>
     /// Creates a policy that doesn't vary the cached content based on headers.

src/Middleware/OutputCaching/src/Serialization/FormatterEntry.cs

@@ -2,7 +2,7 @@
 // The .NET Foundation licenses this file to you under the MIT license.
 
 namespace Microsoft.AspNetCore.OutputCaching.Serialization;
-internal class FormatterEntry
+internal sealed class FormatterEntry
 {
     public DateTimeOffset Created { get; set; }
     public int StatusCode { get; set; }