Feedback

Author: JunTaoLuo

src/Microsoft.AspNetCore.Http.Abstractions/Internal/ParsingHelpers.cs

@@ -26,9 +26,13 @@ private static IEnumerable<string> GetHeaderSplitImplementation(StringValues val
         {
             foreach (var segment in new HeaderSegmentCollection(values))
             {
-                if (segment.Data.HasValue && !StringSegment.IsNullOrEmpty(segment.Data))
+                if (!StringSegment.IsNullOrEmpty(segment.Data))
                 {
-                    yield return DeQuote(segment.Data.Value);
+                    var value = DeQuote(segment.Data.Value);
+                    if (!string.IsNullOrEmpty(value))
+                    {
+                        yield return value;
+                    }
                 }
             }
         }

test/Microsoft.AspNetCore.Http.Tests/HeaderDictionaryTests.cs

@@ -40,7 +40,7 @@ public void PropertiesAreAccessible()
 
         [Theory]
         [MemberData(nameof(HeaderSegmentData))]
-        public void EmptyHeaderSegmentsAreParsable(IEnumerable<string> segments)
+        public void EmptyHeaderSegmentsAreIgnored(IEnumerable<string> segments)
         {
             var header = string.Join(",", segments);
 
@@ -51,9 +51,22 @@ public void EmptyHeaderSegmentsAreParsable(IEnumerable<string> segments)
                });
 
             var result = headers.GetCommaSeparatedValues("Header1");
-            var expectedResult = segments.Where(s => !string.IsNullOrWhiteSpace(s));
+            var expectedResult = segments.Where(s => !string.IsNullOrEmpty(s));
 
             Assert.Equal(expectedResult, result);
         }
+
+        [Fact]
+        public void EmtpyQuotedHeaderSegmentsAreIgnored()
+        {
+            var headers = new HeaderDictionary(
+               new Dictionary<string, StringValues>(StringComparer.OrdinalIgnoreCase)
+               {
+                    { "Header1",  "Value1,\"\",,Value2" },
+               });
+
+            var result = headers.GetCommaSeparatedValues("Header1");
+            Assert.Equal(new[] { "Value1", "Value2" }, result);
+        }
     }
 }