fix(headers): enhance error handling for invalid Host header

Author: spaceone

httoop/header/messaging.py

@@ -325,7 +325,10 @@ def ip4address(self) -> str | None:
 
     def sanitize(self) -> None:
         self.value = self.value.lower()
-        self.host, self.port = self.HOSTPORT.match(self.value).groups()
+        match = self.HOSTPORT.match(self.value)
+        if not match:
+            raise InvalidHeader(_('Invalid Host header: %s'), self.value)
+        self.host, self.port = match.groups()
         if self.host.endswith(']') and self.host.startswith('['):
             self.host = self.host[1:-1]
         if self.port:

tests/headers/test_host_header.py

@@ -57,6 +57,7 @@ def _test_iter(header, host, port, headers):
 
 @pytest.mark.parametrize('invalid', list(
     list((set('\x7F()<>@,;:/\\[\\]={} \t\\\\^"\'') | set(map(chr, range(0x1F)))) - set(';\n\x00')) + [
+        ', ',
         pytest.param(';', marks=pytest.mark.xfail),  # FIXME
         pytest.param('\n', marks=pytest.mark.xfail),  # FIXME
         pytest.param('\x00', marks=pytest.mark.xfail),  # FIXME