Kludex/uvicorn
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Restore has_close_header tracking

Browse Source 
keep_alive can be False from HTTP/1.0 initialization, not only from
a response Connection: close header, so it cannot replace the flag.
This commit is contained in:
Marcelo Trylesinski 2026-03-15 18:25:23 +01:00
parent cd2f5a59ea
commit cfc5be302f
1 changed files with 3 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
uvicorn/protocols/http/httptools_impl.py
View File

@ -488,6 +488,7 @@ class RequestResponseCycle:
# Write response status line and headers
content = [STATUS_LINE[status_code]]
has_close_header = False
for name, value in self.default_headers:
content.extend([name, b": ", value, b"\r\n"])
@ -507,9 +508,10 @@ class RequestResponseCycle:
self.chunked_encoding = True
elif name == b"connection" and value.lower() == b"close":
self.keep_alive = False
has_close_header = True
content.extend([name, b": ", value, b"\r\n"])
if self.keep_alive and CLOSE_HEADER in self.scope["headers"]:
if not has_close_header and CLOSE_HEADER in self.scope["headers"]:
content.extend([b"connection", b": ", b"close", b"\r\n"])
self.keep_alive = False