moar nits

Author: mgravell

src/Middleware/OutputCaching/src/OutputCacheEntryFormatter.cs

@@ -396,7 +396,7 @@ private static string ReadCommonHeader(ref FormatterBinaryReader reader)
 
     static readonly string[] CommonHeaders = new string[]
     {
-        // to remove values, use ""; DO NOT just remove the line, and do not re-order
+        // DO NOT remove values, and do not re-order/insert - append only
         // NOTE: arbitrary common strings are fine - it doesn't all have to be headers
         HeaderNames.Accept,
         HeaderNames.AcceptCharset,

src/Middleware/OutputCaching/src/OutputCacheMiddleware.cs

@@ -415,7 +415,8 @@ internal async ValueTask FinalizeCacheBodyAsync(OutputCacheContext context)
                 || (cachedResponseBody.Length == 0
                     && HttpMethods.IsHead(context.HttpContext.Request.Method)))
             {
-                context.CachedResponse.SetBody(cachedResponseBody, recycleBuffers: true); // transfer lifetime from the buffer to the cached response
+                // transfer lifetime from the buffer to the cached response
+                context.CachedResponse.SetBody(cachedResponseBody, recycleBuffers: true);
 
                 if (string.IsNullOrEmpty(context.CacheKey))
                 {