Copilot review (PR #4, commit 489abc8, src/httpd.erl).
Request timeouts are implemented via the recv_timeout socket option mapped to handle_timeout/1 (closes a connection with a buffered partial request). There is currently no test asserting this behavior.
Fix: add an integration test that starts the server with a small request_timeout, sends a partial request (e.g. headers without the full body, or a truncated request line), and verifies the socket is closed after the timeout — while confirming an idle keep-alive/WebSocket connection is NOT closed.
Validated against current code: no such test exists in test/.
Copilot review (PR #4, commit 489abc8,
src/httpd.erl).Request timeouts are implemented via the
recv_timeoutsocket option mapped tohandle_timeout/1(closes a connection with a buffered partial request). There is currently no test asserting this behavior.Fix: add an integration test that starts the server with a small
request_timeout, sends a partial request (e.g. headers without the full body, or a truncated request line), and verifies the socket is closed after the timeout — while confirming an idle keep-alive/WebSocket connection is NOT closed.Validated against current code: no such test exists in
test/.