Addressing feedback given by asp.net team

Author: uabarahona

src/Antiforgery/test/DefaultAntiforgeryTokenStoreTest.cs

@@ -491,7 +491,7 @@ public void Append(string key, string value)
                 throw new NotImplementedException();
             }
 
-            public void Append(IDictionary<string, string> keyValuePairs, CookieOptions options)
+            public void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options)
             {
                 throw new NotImplementedException();
             }

src/Http/Http.Features/src/IResponseCookies.cs

@@ -30,7 +30,7 @@ public interface IResponseCookies
         /// </summary>
         /// <param name="keyValuePairs">Key value pair collections whose elements will be added as cookies.</param>
         /// <param name="options"><see cref="CookieOptions"/> included in new cookie settings.</param>
-        void Append(IDictionary<string, string> keyValuePairs, CookieOptions options);
+        void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options);
 
         /// <summary>
         /// Sets an expired cookie.

src/Http/Http.Features/src/PublicAPI.Unshipped.txt

@@ -15,4 +15,4 @@ Microsoft.AspNetCore.Http.Features.IFeatureCollection.Set<TFeature>(TFeature? in
 Microsoft.AspNetCore.Http.Features.IServerVariablesFeature.this[string! variableName].get -> string?
 Microsoft.AspNetCore.Http.ISession.TryGetValue(string! key, out byte[]? value) -> bool
 Microsoft.AspNetCore.Http.Features.FeatureCollection.FeatureCollection(int initialCapacity) -> void
-Microsoft.AspNetCore.Http.IResponseCookies.Append(System.Collections.Generic.IDictionary<string!, string!>! keyValuePairs, Microsoft.AspNetCore.Http.CookieOptions! options) -> void
+Microsoft.AspNetCore.Http.IResponseCookies.Append(System.Collections.Generic.IEnumerable<System.Collections.Generic.KeyValuePair<string!, string!>>! keyValuePairs, Microsoft.AspNetCore.Http.CookieOptions! options) -> void

src/Http/Http/src/Internal/ResponseCookies.cs

@@ -3,6 +3,7 @@
 
 using System;
 using System.Collections.Generic;
+using System.Linq;
 using Microsoft.AspNetCore.Http.Features;
 using Microsoft.Extensions.DependencyInjection;
 using Microsoft.Extensions.Logging;
@@ -89,7 +90,7 @@ public void Append(string key, string value, CookieOptions options)
         }
 
         /// <inheritdoc />
-        public void Append(IDictionary<string, string> keyValuePairs, CookieOptions options)
+        public void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options)
         {
             if (options == null)
             {
@@ -126,17 +127,17 @@ public void Append(IDictionary<string, string> keyValuePairs, CookieOptions opti
             };
 
             var cookierHeaderValue = setCookieHeaderValue.ToString()[1..];
-            var cookies = new string[keyValuePairs.Count];
+            var cookies = new string[keyValuePairs.Count()];
             var position = 0;
 
             foreach (var keyValuePair in keyValuePairs)
             {
-                cookies[position] = $"{keyValuePair.Key}={keyValuePair.Value}{cookierHeaderValue}";
+
+                cookies[position] = string.Concat(_enableCookieNameEncoding ? Uri.EscapeDataString(keyValuePair.Key) : keyValuePair.Key, "=", Uri.EscapeDataString(keyValuePair.Value), cookierHeaderValue);
                 position++;
             }
 
             Headers.Append(HeaderNames.SetCookie, cookies);
-            
         }
 
         /// <inheritdoc />

src/Mvc/Mvc.ViewFeatures/test/CookieTempDataProviderTest.cs

@@ -417,7 +417,7 @@ IEnumerator IEnumerable.GetEnumerator()
                 return GetEnumerator();
             }
 
-            public void Append(IDictionary<string, string> keyValuePairs, CookieOptions options)
+            public void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options)
             {
                 foreach (var keyValuePair in keyValuePairs)
                 {

src/Security/CookiePolicy/src/ResponseCookiesWrapper.cs

@@ -153,7 +153,7 @@ public void Append(string key, string value, CookieOptions options)
             }
         }
 
-        public void Append(IDictionary<string, string> keyValuePairs, CookieOptions options)
+        public void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options)
         {
             if (options == null)
             {

src/Security/CookiePolicy/test/CookiePolicyTests.cs

@@ -486,7 +486,7 @@ public void Append(string key, string value, CookieOptions options)
                     throw new NotImplementedException();
                 }
 
-                public void Append(IDictionary<string, string> keyValuePairs, CookieOptions options)
+                public void Append(IEnumerable<KeyValuePair<string, string>> keyValuePairs, CookieOptions options)
                 {
                     throw new NotImplementedException();
                 }

src/Shared/ChunkingCookieManager/ChunkingCookieManager.cs

@@ -208,20 +208,19 @@ public void AppendResponseCookie(HttpContext context, string key, string? value,
                 var dataSizePerCookie = ChunkSize.Value - templateLength - 3; // Budget 3 chars for the chunkid.
                 var cookieChunkCount = (int)Math.Ceiling(value.Length * 1.0 / dataSizePerCookie);
 
-                IDictionary<string, string> keyValuePairs = new Dictionary<string, string>(cookieChunkCount)
+                List<KeyValuePair<string, string>> keyValuePairs = new(cookieChunkCount)
                 {
-                    [key] = ChunkCountPrefix + cookieChunkCount.ToString(CultureInfo.InvariantCulture)
+                    KeyValuePair.Create(key, ChunkCountPrefix + cookieChunkCount.ToString(CultureInfo.InvariantCulture))
                 };
 
-
                 var offset = 0;
                 for (var chunkId = 1; chunkId <= cookieChunkCount; chunkId++)
                 {
                     var remainingLength = value.Length - offset;
                     var length = Math.Min(dataSizePerCookie, remainingLength);
                     var segment = value.Substring(offset, length);
                     offset += length;
-                    keyValuePairs.Add(key + ChunkKeySuffix + chunkId.ToString(CultureInfo.InvariantCulture), segment);
+                    keyValuePairs.Add(KeyValuePair.Create(string.Concat(key, ChunkKeySuffix, chunkId.ToString(CultureInfo.InvariantCulture)), segment));
                 }
 
                 responseCookies.Append(keyValuePairs, options);    
@@ -307,15 +306,14 @@ public void DeleteCookie(HttpContext context, string key, CookieOptions options)
 
             var responseCookies = context.Response.Cookies;
 
-
-            IDictionary<string, string> keyValuePairs = new Dictionary<string, string>(chunks)
+            List<KeyValuePair<string, string>> keyValuePairs = new(chunks)
             {
-                [key] = string.Empty
+                KeyValuePair.Create(key, string.Empty)
             };
 
             for (var i = 1; i <= chunks; i++)
             {
-                keyValuePairs.Add(key + "C" + i.ToString(CultureInfo.InvariantCulture), string.Empty);
+                keyValuePairs.Add(KeyValuePair.Create(string.Concat(key, "C", i.ToString(CultureInfo.InvariantCulture)), string.Empty));
             }
 
             responseCookies.Append(keyValuePairs, new CookieOptions()