aboutsummaryrefslogtreecommitdiffstats
path: root/src
diff options
context:
space:
mode:
authorFred Hebert <[email protected]>2014-01-14 13:30:58 -0500
committerFred Hebert <[email protected]>2014-07-14 10:11:03 -0400
commitb10b34a8f204ed409a6fc936e199557bfe3b0975 (patch)
treed911ecf2b32464ef67a12f02dec809300686dc42 /src
parent9980348a1cd74a41b3024f0b01d6fef301d8288b (diff)
downloadcowboy-b10b34a8f204ed409a6fc936e199557bfe3b0975.tar.gz
cowboy-b10b34a8f204ed409a6fc936e199557bfe3b0975.tar.bz2
cowboy-b10b34a8f204ed409a6fc936e199557bfe3b0975.zip
Support ad-hoc keep-alive for HTTP/1.0 Clients
Only go for keep-alive if they submit a 'connection: keep-alive' header in the request, keep behaviour the same otherwise. The new RFC 7230 (http://tools.ietf.org/html/rfc7230#section-6.3) states: If the received protocol is HTTP/1.0, the "keep-alive" connection option is present, the recipient is not a proxy, and the recipient wishes to honor the HTTP/1.0 "keep-alive" mechanism, the connection will persist after the current response; Even though clients are discouraged from doing so in Appendix A.1.2 (http://tools.ietf.org/html/rfc7230#appendix-A.1.2)
Diffstat (limited to 'src')
-rw-r--r--src/cowboy_req.erl7
1 files changed, 5 insertions, 2 deletions
diff --git a/src/cowboy_req.erl b/src/cowboy_req.erl
index 1394e72..981b321 100644
--- a/src/cowboy_req.erl
+++ b/src/cowboy_req.erl
@@ -174,13 +174,16 @@ new(Socket, Transport, Peer, Method, Path, Query,
method=Method, path=Path, qs=Query, version=Version,
headers=Headers, host=Host, port=Port, buffer=Buffer,
resp_compress=Compress, onresponse=OnResponse},
- case CanKeepalive and (Version =:= 'HTTP/1.1') of
+ case CanKeepalive of
false ->
Req#http_req{connection=close};
true ->
case lists:keyfind(<<"connection">>, 1, Headers) of
false ->
- Req; %% keepalive
+ case Version of
+ 'HTTP/1.1' -> Req; %% keepalive
+ 'HTTP/1.0' -> Req#http_req{connection=close}
+ end;
{_, ConnectionHeader} ->
Tokens = cow_http_hd:parse_connection(ConnectionHeader),
Connection = connection_to_atom(Tokens),