aboutsummaryrefslogtreecommitdiffstats
path: root/test/rfc7540_SUITE.erl
diff options
context:
space:
mode:
authorLoïc Hoguin <[email protected]>2017-05-23 14:09:38 +0200
committerLoïc Hoguin <[email protected]>2017-05-23 14:09:38 +0200
commit9ef4536656fd6c8bd9c80c6731460c057ba2b177 (patch)
tree324afb3f63449cf756f46878b40167e091f8f30e /test/rfc7540_SUITE.erl
parent4048f16365047ac9b16a5dbf4c3aae0b665a613c (diff)
downloadcowboy-9ef4536656fd6c8bd9c80c6731460c057ba2b177.tar.gz
cowboy-9ef4536656fd6c8bd9c80c6731460c057ba2b177.tar.bz2
cowboy-9ef4536656fd6c8bd9c80c6731460c057ba2b177.zip
Add many tests for RFC7540 5.1 and 5.1.1 and related fixes
Diffstat (limited to 'test/rfc7540_SUITE.erl')
-rw-r--r--test/rfc7540_SUITE.erl566
1 files changed, 566 insertions, 0 deletions
diff --git a/test/rfc7540_SUITE.erl b/test/rfc7540_SUITE.erl
index 6ce5253..727ea0c 100644
--- a/test/rfc7540_SUITE.erl
+++ b/test/rfc7540_SUITE.erl
@@ -1670,3 +1670,569 @@ continuation_wrong_stream_error(Config) ->
%% Receive a PROTOCOL_ERROR connection error.
{ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
ok.
+
+%% Stream states.
+
+idle_stream_reject_data(Config) ->
+ doc("DATA frames received on an idle stream must be rejected "
+ "with a PROTOCOL_ERROR connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a DATA frame on an idle stream.
+ ok = gen_tcp:send(Socket, cow_http2:data(1, fin, <<"Unexpected DATA frame.">>)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+idle_stream_accept_headers(Config) ->
+ doc("HEADERS frames received on an idle stream must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame on an idle stream.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive a HEADERS frame as a response.
+ {ok, << _:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+idle_stream_accept_priority(Config) ->
+ doc("PRIORITY frames received on an idle stream must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a PRIORITY frame on an idle stream.
+ ok = gen_tcp:send(Socket, cow_http2:priority(1, shared, 3, 123)),
+ %% Receive no error.
+ {error, timeout} = gen_tcp:recv(Socket, 7, 1000),
+ %% Send a HEADERS frame on the same stream.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive a HEADERS frame as a response.
+ {ok, << _:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+idle_stream_reject_rst_stream(Config) ->
+ doc("RST_STREAM frames received on an idle stream must be rejected "
+ "with a PROTOCOL_ERROR connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send an RST_STREAM frame on an idle stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, no_error)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+idle_stream_reject_push_promise(Config) ->
+ doc("PUSH_PROMISE frames received on an idle stream must be rejected "
+ "with a PROTOCOL_ERROR connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a PUSH_PROMISE frame on an idle stream.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:push_promise(1, 3, HeadersBlock)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+idle_stream_reject_window_update(Config) ->
+ doc("WINDOW_UPDATE frames received on an idle stream must be rejected "
+ "with a PROTOCOL_ERROR connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a WINDOW_UPDATE frame on an idle stream.
+ ok = gen_tcp:send(Socket, cow_http2:window_update(1, 12345)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+%reserved (local) - after sending PUSH_PROMISE:
+% An endpoint MUST NOT send any type of frame other than HEADERS,
+% RST_STREAM, or PRIORITY in this state.
+%%% how to test this?
+%
+% A PRIORITY or WINDOW_UPDATE frame MAY be received in this state.
+% Receiving any type of frame other than RST_STREAM, PRIORITY, or
+% WINDOW_UPDATE on a stream in this state MUST be treated as a
+% connection error (Section 5.4.1) of type PROTOCOL_ERROR.
+%%% we need to use a large enough file for this
+%
+%reserved_local_reject_data
+%reserved_local_reject_headers
+%reserved_local_accept_priority
+%reserved_local_accept_rst_stream
+%reserved_local_reject_push_promise %% do we even care? we reject it always
+%reserved_local_accept_window_update
+%
+%half-closed (remote):
+% If an endpoint receives additional frames, other than
+% WINDOW_UPDATE, PRIORITY, or RST_STREAM, for a stream that is in
+% this state, it MUST respond with a stream error (Section 5.4.2) of
+% type STREAM_CLOSED.
+
+half_closed_remote_reject_data(Config) ->
+ doc("DATA frames received on a half-closed (remote) stream must be rejected "
+ "with a STREAM_CLOSED stream error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with the FIN flag set.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Send a DATA frame on that now half-closed (remote) stream.
+ ok = gen_tcp:send(Socket, cow_http2:data(1, fin, <<"Unexpected DATA frame.">>)),
+ %% Receive a STREAM_CLOSED stream error.
+ {ok, << _:24, 3:8, _:8, 1:32, 5:32 >>} = gen_tcp:recv(Socket, 13, 6000),
+ ok.
+
+half_closed_remote_reject_headers(Config) ->
+ doc("HEADERS frames received on a half-closed (remote) stream must be rejected "
+ "with a STREAM_CLOSED stream error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with the FIN flag set.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Send a HEADERS frame on that now half-closed (remote) stream.
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive a STREAM_CLOSED stream error.
+ {ok, << _:24, 3:8, _:8, 1:32, 5:32 >>} = gen_tcp:recv(Socket, 13, 6000),
+ ok.
+
+half_closed_remote_accept_priority(Config) ->
+ doc("PRIORITY frames received on a half-closed stream must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with the FIN flag set.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Send a PRIORITY frame on that now half-closed (remote) stream.
+ ok = gen_tcp:send(Socket, cow_http2:priority(1, shared, 3, 123)),
+ %% Receive a HEADERS frame as a response.
+ {ok, << _:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+half_closed_remote_accept_rst_stream(Config) ->
+ doc("RST_STREAM frames received on a half-closed stream must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with the FIN flag set.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Send an RST_STREAM frame on that now half-closed (remote) stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, no_error)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+%% half_closed_remote_reject_push_promise
+%%
+%% We respond to all PUSH_PROMISE frames with a PROTOCOL_ERROR connection error
+%% because PUSH is disabled in that direction. We therefore cannot test other
+%% error conditions.
+
+half_closed_remote_accept_window_update(Config) ->
+ doc("WINDOW_UPDATE frames received on a half-closed stream must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with the FIN flag set.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Send a WINDOW_UPDATE frame on that now half-closed (remote) stream.
+ ok = gen_tcp:send(Socket, cow_http2:window_update(1, 12345)),
+ %% Receive a HEADERS frame as a response.
+ {ok, << _:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+rst_stream_closed_reject_data(Config) ->
+ doc("DATA frames received on a stream closed via RST_STREAM must be rejected "
+ "with a STREAM_CLOSED stream error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Send an RST_STREAM frame to close the stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Send a DATA frame on the now RST_STREAM closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:data(1, fin, <<"Unexpected DATA frame.">>)),
+ %% Receive a STREAM_CLOSED stream error.
+ {ok, << _:24, 3:8, _:8, 1:32, 5:32 >>} = gen_tcp:recv(Socket, 13, 6000),
+ ok.
+
+rst_stream_closed_reject_headers(Config) ->
+ doc("HEADERS frames received on a stream closed via RST_STREAM must be rejected "
+ "with a STREAM_CLOSED stream error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Send an RST_STREAM frame to close the stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Send a HEADERS frame on the now RST_STREAM closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Receive a STREAM_CLOSED stream error.
+ {ok, << _:24, 3:8, _:8, 1:32, 5:32 >>} = gen_tcp:recv(Socket, 13, 6000),
+ ok.
+
+rst_stream_closed_accept_priority(Config) ->
+ doc("PRIORITY frames received on a stream closed via RST_STREAM "
+ "must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Send an RST_STREAM frame to close the stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Send a PRIORITY frame on that now RST_STREAM closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:priority(1, shared, 3, 123)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+rst_stream_closed_ignore_rst_stream(Config) ->
+ doc("RST_STREAM frames received on a stream closed via RST_STREAM "
+ "must be ignored to avoid looping. (RFC7540 5.1, RFC7540 5.4.2)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Send an RST_STREAM frame to close the stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Send an extra RST_STREAM.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+%% rst_stream_closed_reject_push_promise
+%%
+%% We respond to all PUSH_PROMISE frames with a PROTOCOL_ERROR connection error
+%% because PUSH is disabled in that direction. We therefore cannot test other
+%% error conditions.
+
+rst_stream_closed_reject_window_update(Config) ->
+ doc("WINDOW_UPDATE frames received on a stream closed via RST_STREAM "
+ "must be rejected with a STREAM_CLOSED stream error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Send an RST_STREAM frame to close the stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Send a WINDOW_UPDATE frame on the now RST_STREAM closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:window_update(1, 12345)),
+ %% Receive a STREAM_CLOSED stream error.
+ {ok, << _:24, 3:8, _:8, 1:32, 5:32 >>} = gen_tcp:recv(Socket, 13, 6000),
+ ok.
+
+stream_closed_reject_data(Config) ->
+ doc("DATA frames received on a stream closed normally must be rejected "
+ "with a STREAM_CLOSED connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send a DATA frame on the now closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:data(1, fin, <<"Unexpected DATA frame.">>)),
+ %% Receive a STREAM_CLOSED connection error.
+ {ok, << _:24, 7:8, _:72, 5:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+stream_closed_reject_headers(Config) ->
+ doc("HEADERS frames received on a stream closed normally must be rejected "
+ "with a STREAM_CLOSED connection error. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send a HEADERS frame on the now closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive a STREAM_CLOSED connection error.
+ {ok, << _:24, 7:8, _:72, 5:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+stream_closed_accept_priority(Config) ->
+ doc("PRIORITY frames received on a stream closed normally must be accepted. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send a PRIORITY frame on the now closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:priority(1, shared, 3, 123)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+stream_closed_accept_rst_stream(Config) ->
+ doc("RST_STREAM frames received on a stream closed normally "
+ "must be accepted for a short period. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send an RST_STREAM frame on the now closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:rst_stream(1, cancel)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+%% stream_closed_reject_push_promise
+%%
+%% We respond to all PUSH_PROMISE frames with a PROTOCOL_ERROR connection error
+%% because PUSH is disabled in that direction. We therefore cannot test other
+%% error conditions.
+
+stream_closed_accept_window_update(Config) ->
+ doc("WINDOW_UPDATE frames received on a stream closed normally "
+ "must be accepted for a short period. (RFC7540 5.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, nofin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send a WINDOW_UPDATE frame on the now closed stream.
+ ok = gen_tcp:send(Socket, cow_http2:window_update(1, 12345)),
+ %% Receive nothing back.
+ {error, timeout} = gen_tcp:recv(Socket, 9, 6000),
+ ok.
+
+%% @todo While we accept RST_STREAM and WINDOW_UPDATE for a short period
+%% after the stream closed normally, we may want to reject the ones coming
+%% a significant amount of time after that.
+
+%% @todo Frames may arrive on a stream after we send an RST_STREAM for it.
+%% They must be ignored for a short period of time:
+%
+% If this state is reached as a result of sending a RST_STREAM
+% frame, the peer that receives the RST_STREAM might have already
+% sent -- or enqueued for sending -- frames on the stream that
+% cannot be withdrawn. An endpoint MUST ignore frames that it
+% receives on closed streams after it has sent a RST_STREAM frame.
+% An endpoint MAY choose to limit the period over which it ignores
+% frames and treat frames that arrive after this time as being in
+% error.
+
+%% @todo Ensure that rejected DATA frames result in the connection
+%% flow-control window being updated. How to test this?
+%
+% Flow-controlled frames (i.e., DATA) received after sending
+% RST_STREAM are counted toward the connection flow-control window.
+% Even though these frames might be ignored, because they are sent
+% before the sender receives the RST_STREAM, the sender will
+% consider the frames to count against the flow-control window.
+
+%% Stream identifiers.
+
+reject_streamid_even(Config) ->
+ doc("HEADERS frames received with an even-numbered streamid "
+ "must be rejected with a PROTOCOL_ERROR connection error. (RFC7540 5.1.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with an even-numbered streamid.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(2, fin, HeadersBlock)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+reject_streamid_0(Config) ->
+ doc("HEADERS frames received with streamid 0 (zero) "
+ "must be rejected with a PROTOCOL_ERROR connection error. (RFC7540 5.1.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with an streamid 0.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(0, fin, HeadersBlock)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+http_upgrade_reject_reuse_streamid_1(Config) ->
+ doc("Attempts to reuse streamid 1 after upgrading to HTTP/2 "
+ "must be rejected with a PROTOCOL_ERROR connection error. (RFC7540 5.1.1)"),
+ {ok, Socket} = gen_tcp:connect("localhost", config(port, Config), [binary, {active, false}]),
+ ok = gen_tcp:send(Socket, [
+ "GET / HTTP/1.1\r\n"
+ "Host: localhost\r\n"
+ "Connection: Upgrade, HTTP2-Settings\r\n"
+ "Upgrade: h2c\r\n"
+ "HTTP2-Settings: ", base64:encode(cow_http2:settings_payload(#{})), "\r\n",
+ "\r\n"]),
+ ok = do_recv_101(Socket),
+ %% Send a valid preface.
+ ok = gen_tcp:send(Socket, ["PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n", cow_http2:settings(#{})]),
+ %% Receive the server preface.
+ {ok, << Len:24 >>} = gen_tcp:recv(Socket, 3, 1000),
+ {ok, << 4:8, 0:40, _:Len/binary >>} = gen_tcp:recv(Socket, 6 + Len, 1000),
+ %% Send the SETTINGS ack.
+ ok = gen_tcp:send(Socket, cow_http2:settings_ack()),
+ %% Receive the SETTINGS ack, and the response HEADERS and DATA (Stream ID 1).
+ Received = lists:reverse(lists:foldl(fun(_, Acc) ->
+ case gen_tcp:recv(Socket, 9, 1000) of
+ {ok, << 0:24, 4:8, 1:8, 0:32 >>} ->
+ [settings_ack|Acc];
+ {ok, << SkipLen:24, 1:8, _:8, 1:32 >>} ->
+ {ok, _} = gen_tcp:recv(Socket, SkipLen, 1000),
+ [headers|Acc];
+ {ok, << SkipLen:24, 0:8, _:8, 1:32 >>} ->
+ {ok, _} = gen_tcp:recv(Socket, SkipLen, 1000),
+ [data|Acc]
+ end
+ end, [], [1, 2, 3])),
+ case Received of
+ [settings_ack, headers, data] -> ok;
+ [headers, settings_ack, data] -> ok;
+ [headers, data, settings_ack] -> ok
+ end,
+ %% Send a HEADERS frame with streamid 1.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(1, fin, HeadersBlock)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+reject_streamid_lower(Config) ->
+ doc("HEADERS frames received with streamid lower than the previous stream "
+ "must be rejected with a PROTOCOL_ERROR connection error. (RFC7540 5.1.1)"),
+ {ok, Socket} = do_handshake(Config),
+ %% Send a HEADERS frame with streamid 5.
+ {HeadersBlock, _} = cow_hpack:encode([
+ {<<":method">>, <<"GET">>},
+ {<<":scheme">>, <<"http">>},
+ {<<":authority">>, <<"localhost">>}, %% @todo Correct port number.
+ {<<":path">>, <<"/">>}
+ ]),
+ ok = gen_tcp:send(Socket, cow_http2:headers(5, fin, HeadersBlock)),
+ %% Receive the response.
+ {ok, << Length1:24, 1:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length1, 6000),
+ {ok, << Length2:24, 0:8, _:40 >>} = gen_tcp:recv(Socket, 9, 6000),
+ {ok, _} = gen_tcp:recv(Socket, Length2, 6000),
+ %% Send a HEADERS frame with streamid 3.
+ ok = gen_tcp:send(Socket, cow_http2:headers(3, fin, HeadersBlock)),
+ %% Receive a PROTOCOL_ERROR connection error.
+ {ok, << _:24, 7:8, _:72, 1:32 >>} = gen_tcp:recv(Socket, 17, 6000),
+ ok.
+
+%% @todo We need an option to limit the number of streams one can open
+%% on a connection. And we need to enforce it.
+%
+% Stream identifiers cannot be reused. Long-lived connections can
+% result in an endpoint exhausting the available range of stream
+% identifiers. A server
+% that is unable to establish a new stream identifier can send a GOAWAY
+% frame so that the client is forced to open a new connection for new
+% streams.