[release/2.1] Do not escape cookie names in lookups

- **Do Not Merge**
- undo an important part of #24264
- this is an attempt to demonstrate #29023
  - should show test indirect references let most tests keep working

Author: dougbu

eng/PatchConfig.props

@@ -72,7 +72,7 @@ Later on, this will be checked using this condition:
       Microsoft.AspNetCore.ResponseCompression;
     </PackagesInPatch>
   </PropertyGroup>
-  <PropertyGroup Condition=" '$(VersionPrefix)' == '2.1.22' ">
+  <PropertyGroup Condition=" '$(VersionPrefix)' == '2.1.24' ">
     <PackagesInPatch>
       Microsoft.AspNetCore.Http;
     </PackagesInPatch>

src/Http/Http/src/Internal/RequestCookieCollection.cs

@@ -88,6 +88,7 @@ internal static RequestCookieCollection ParseInternal(IList<string> values, bool
                     {
                         name = Uri.UnescapeDataString(name);
                     }
+
                     var value = Uri.UnescapeDataString(cookie.Value.Value);
                     store[name] = value;
                 }
@@ -127,8 +128,8 @@ public bool ContainsKey(string key)
             {
                 return false;
             }
-            return Store.ContainsKey(key)
-                || !_enableCookieNameDecoding && Store.ContainsKey(Uri.EscapeDataString(key));
+
+            return Store.ContainsKey(key);
         }
 
         public bool TryGetValue(string key, out string value)
@@ -139,8 +140,7 @@ public bool TryGetValue(string key, out string value)
                 return false;
             }
 
-            return Store.TryGetValue(key, out value)
-                || !_enableCookieNameDecoding && Store.TryGetValue(Uri.EscapeDataString(key), out value);
+            return Store.TryGetValue(key, out value);
         }
 
         /// <summary>