Refine scoped header list parsing follow-up

Author: rodrigobnogueira

CHANGES/12249.bugfix.rst

@@ -0,0 +1,5 @@
+Normalized parsing of list-style ``Connection`` and ``Transfer-Encoding``
+headers in the pure-Python HTTP parser so repeated field lines and
+comma-joined values are handled consistently for keep-alive, close,
+upgrade, and chunked detection.
+-- by :user:`rodrigobnogueira`.

CHANGES/12253.bugfix.rst

@@ -1,5 +1,4 @@
-Normalized parsing of list-style ``Connection`` and ``Transfer-Encoding``
-headers in the pure-Python HTTP parser so repeated field lines and
-comma-joined values are handled consistently for keep-alive, close,
-upgrade, and chunked detection.
+Added ``parse_http_list_values()`` to normalize repeated and comma-joined
+list-style header values, and used it for ``Connection`` and
+``Transfer-Encoding`` handling in the pure-Python HTTP parser.
 -- by :user:`rodrigobnogueira`.

tests/test_http_parser.py

@@ -17,7 +17,7 @@
 import aiohttp
 from aiohttp import http_exceptions, streams
 from aiohttp.base_protocol import BaseProtocol
-from aiohttp.helpers import NO_EXTENSIONS
+from aiohttp.helpers import NO_EXTENSIONS, parse_http_list_values
 from aiohttp.http_parser import (
     DeflateBuffer,
     HeadersParser,
@@ -614,6 +614,27 @@ def test_request_te_duplicate_chunked(parser: HttpRequestParser) -> None:
         parser.feed_data(text)
 
 
+@pytest.mark.parametrize(
+    ("header_lines", "expected"),
+    [
+        (b"Foo: 1\r\nFoo: 2\r\n", ("1", "2")),
+        (b"Foo: 1, 2\r\n", ("1", "2")),
+        (
+            b'Foo: "http://example.com/a.html,foo", apples\r\n',
+            ('"http://example.com/a.html,foo"', "apples"),
+        ),
+    ],
+)
+def test_parse_http_list_values_normalizes_parser_header_views(
+    parser: HttpRequestParser, header_lines: bytes, expected: tuple[str, ...]
+) -> None:
+    text = b"GET /test HTTP/1.1\r\n" + header_lines + b"\r\n"
+    messages, upgrade, tail = parser.feed_data(text)
+    msg = messages[0][0]
+
+    assert parse_http_list_values(msg.headers.getall("Foo", ())) == expected
+
+
 def test_request_te_empty_list_value_errors_in_py_parser(
     loop: asyncio.AbstractEventLoop, protocol: BaseProtocol
 ) -> None: