Software / code / prosody
Changeset
2837:a17e73ab5f4c
net.httpserver: More robust handling of headers split across multiple packets
| author | Matthew Wild <mwild1@gmail.com> |
|---|---|
| date | Thu, 21 Jan 2010 14:53:01 +0000 |
| parents | 2836:dfb5fa77d437 |
| children | 2838:586893d29eaf |
| files | net/httpserver.lua |
| diffstat | 1 files changed, 11 insertions(+), 4 deletions(-) [+] |
line wrap: on
line diff
--- a/net/httpserver.lua Thu Jan 21 13:14:52 2010 +0000 +++ b/net/httpserver.lua Thu Jan 21 14:53:01 2010 +0000 @@ -146,22 +146,29 @@ elseif request.state == "headers" then log("debug", "Reading headers...") local pos = startpos; - local headers = request.headers or {}; + local headers, headers_complete = request.headers; + if not headers then + headers = {}; + request.headers = headers; + end + for line in data:gmatch("(.-)\r\n") do startpos = (startpos or 1) + #line + 2; local k, v = line:match("(%S+): (.+)"); if k and v then headers[k:lower()] = v; --- log("debug", "Header: "..k:lower().." = "..v); + --log("debug", "Header: '"..k:lower().."' = '"..v.."'"); elseif #line == 0 then - request.headers = headers; + headers_complete = true; break; else log("debug", "Unhandled header line: "..line); end end - if not expectbody(request) then + if not headers_complete then return; end + + if not expectbody(request) then call_callback(request); return; end
