diff options
| author | Loïc Hoguin <[email protected]> | 2013-01-06 19:49:01 +0100 |
|---|---|---|
| committer | Loïc Hoguin <[email protected]> | 2013-01-06 19:49:01 +0100 |
| commit | a013becc66b50db038c1f7f3539040b4482bba18 (patch) | |
| tree | 78b95a24fa5959a89125752b18d6c9ea42a0424f /test | |
| parent | 638d0345d1c738a6a4e73ce291f5ca6bfa4d0b5d (diff) | |
| download | cowboy-a013becc66b50db038c1f7f3539040b4482bba18.tar.gz cowboy-a013becc66b50db038c1f7f3539040b4482bba18.tar.bz2 cowboy-a013becc66b50db038c1f7f3539040b4482bba18.zip | |
Add protection against slowloris vulnerability
This changes the behavior of the `timeout` protocol option to
mean "Time in which the full request line and headers must be
received". The default of 5s should be fine for all normal uses.
This change has no noticeable impact on performance and is thus
enabled by default for everyone. It can be disabled by setting
`timeout` to `infinity` although that is definitely not encouraged.
Inspired by the contribution from @naryl on github.
Diffstat (limited to 'test')
| -rw-r--r-- | test/http_SUITE.erl | 32 |
1 files changed, 32 insertions, 0 deletions
diff --git a/test/http_SUITE.erl b/test/http_SUITE.erl index 7ce0835..607178f 100644 --- a/test/http_SUITE.erl +++ b/test/http_SUITE.erl @@ -60,6 +60,8 @@ -export([set_resp_body/1]). -export([set_resp_header/1]). -export([set_resp_overwrite/1]). +-export([slowloris/1]). +-export([slowloris2/1]). -export([static_attribute_etag/1]). -export([static_function_etag/1]). -export([static_mimetypes_function/1]). @@ -110,6 +112,8 @@ groups() -> set_resp_body, set_resp_header, set_resp_overwrite, + slowloris, + slowloris2, static_attribute_etag, static_function_etag, static_mimetypes_function, @@ -812,6 +816,34 @@ set_resp_overwrite(Config) -> {<<"server">>, <<"DesireDrive/1.0">>} = lists:keyfind(<<"server">>, 1, Headers). +slowloris(Config) -> + Client = ?config(client, Config), + Transport = ?config(transport, Config), + {ok, Client2} = cowboy_client:connect( + Transport, "localhost", ?config(port, Config), Client), + try + [begin + {ok, _} = cowboy_client:raw_request([C], Client2), + receive after 25 -> ok end + end || C <- "GET / HTTP/1.1\r\nHost: localhost\r\n" + "User-Agent: Mozilla/5.0 (Windows; U; Windows NT 6.0; en-US)\r\n" + "Cookie: name=aaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaaa\r\n\r\n"], + error(failure) + catch error:{badmatch, _} -> + ok + end. + +slowloris2(Config) -> + Client = ?config(client, Config), + Transport = ?config(transport, Config), + {ok, Client2} = cowboy_client:connect( + Transport, "localhost", ?config(port, Config), Client), + {ok, _} = cowboy_client:raw_request("GET / HTTP/1.1\r\n", Client2), + receive after 300 -> ok end, + {ok, _} = cowboy_client:raw_request("Host: localhost\r\n", Client2), + receive after 300 -> ok end, + {ok, 408, _, _} = cowboy_client:response(Client2). + static_attribute_etag(Config) -> Client = ?config(client, Config), {ok, Client2} = cowboy_client:request(<<"GET">>, |