diff options
author | Loïc Hoguin <[email protected]> | 2014-12-17 15:09:24 +0200 |
---|---|---|
committer | Loïc Hoguin <[email protected]> | 2014-12-17 15:09:24 +0200 |
commit | eb7be77916f99d3b323db80af42a14bf7e4e5c2e (patch) | |
tree | 0a6115d33d38a8140af9a83d77040f47f5625291 | |
parent | bc447c91450b54af97fcc18ada06f952eddec66c (diff) | |
download | cowlib-eb7be77916f99d3b323db80af42a14bf7e4e5c2e.tar.gz cowlib-eb7be77916f99d3b323db80af42a14bf7e4e5c2e.tar.bz2 cowlib-eb7be77916f99d3b323db80af42a14bf7e4e5c2e.zip |
Add cow_http_hd:parse_upgrade/1
From RFC7230.
Compared to the current Cowboy implementation we added support
for protocol-version. We return protocol-name/protocol-version
as a single binary as these will typically be matched directly.
-rw-r--r-- | src/cow_http_hd.erl | 81 |
1 files changed, 81 insertions, 0 deletions
diff --git a/src/cow_http_hd.erl b/src/cow_http_hd.erl index 4d94ca6..7200e8e 100644 --- a/src/cow_http_hd.erl +++ b/src/cow_http_hd.erl @@ -28,6 +28,7 @@ -export([parse_last_modified/1]). -export([parse_max_forwards/1]). -export([parse_transfer_encoding/1]). +-export([parse_upgrade/1]). -type media_type() :: {binary(), binary(), [{binary(), binary()}]}. -export_type([media_type/0]). @@ -1150,6 +1151,86 @@ horse_parse_transfer_encoding_custom() -> ). -endif. +%% @doc Parse the Upgrade header. +%% +%% It is unclear from the RFC whether the values here are +%% case sensitive. +%% +%% We handle them in a case insensitive manner because they +%% are described as case insensitive in the Websocket RFC. + +-spec parse_upgrade(binary()) -> [binary()]. +parse_upgrade(Upgrade) -> + nonempty(protocol_list(Upgrade, [])). + +protocol_list(<<>>, Acc) -> lists:reverse(Acc); +protocol_list(<< $\s, R/bits >>, Acc) -> protocol_list(R, Acc); +protocol_list(<< $\t, R/bits >>, Acc) -> protocol_list(R, Acc); +protocol_list(<< $,, R/bits >>, Acc) -> protocol_list(R, Acc); +protocol_list(<< C, R/bits >>, Acc) when ?IS_TOKEN(C) -> + case C of + ?INLINE_LOWERCASE(protocol_name, R, Acc, <<>>) + end. + +protocol_name(<<>>, Acc, P) -> lists:reverse([P|Acc]); +protocol_name(<< $\s, R/bits >>, Acc, P) -> protocol_list_sep(R, [P|Acc]); +protocol_name(<< $\t, R/bits >>, Acc, P) -> protocol_list_sep(R, [P|Acc]); +protocol_name(<< $,, R/bits >>, Acc, P) -> protocol_list(R, [P|Acc]); +protocol_name(<< $/, C, R/bits >>, Acc, P) -> + case C of + ?INLINE_LOWERCASE(protocol_version, R, Acc, << P/binary, $/ >>) + end; +protocol_name(<< C, R/bits >>, Acc, P) when ?IS_TOKEN(C) -> + case C of + ?INLINE_LOWERCASE(protocol_name, R, Acc, P) + end. + +protocol_version(<<>>, Acc, P) -> lists:reverse([P|Acc]); +protocol_version(<< $\s, R/bits >>, Acc, P) -> protocol_list_sep(R, [P|Acc]); +protocol_version(<< $\t, R/bits >>, Acc, P) -> protocol_list_sep(R, [P|Acc]); +protocol_version(<< $,, R/bits >>, Acc, P) -> protocol_list(R, [P|Acc]); +protocol_version(<< C, R/bits >>, Acc, P) when ?IS_TOKEN(C) -> + case C of + ?INLINE_LOWERCASE(protocol_version, R, Acc, P) + end. + +protocol_list_sep(<<>>, Acc) -> lists:reverse(Acc); +protocol_list_sep(<< $\s, R/bits >>, Acc) -> protocol_list_sep(R, Acc); +protocol_list_sep(<< $\t, R/bits >>, Acc) -> protocol_list_sep(R, Acc); +protocol_list_sep(<< $,, R/bits >>, Acc) -> protocol_list(R, Acc). + +-ifdef(TEST). +protocols() -> + ?LET(P, + oneof([token(), [token(), $/, token()]]), + iolist_to_binary(P)). + +prop_parse_upgrade() -> + ?FORALL(L, + non_empty(list(protocols())), + begin + << _, Upgrade/binary >> = iolist_to_binary([[$,, P] || P <- L]), + ResL = parse_upgrade(Upgrade), + CheckedL = [?INLINE_LOWERCASE_BC(P) =:= ResP || {P, ResP} <- lists:zip(L, ResL)], + [true] =:= lists:usort(CheckedL) + end). + +parse_upgrade_test_() -> + Tests = [ + {<<"HTTP/2.0, SHTTP/1.3, IRC/6.9, RTA/x11">>, + [<<"http/2.0">>, <<"shttp/1.3">>, <<"irc/6.9">>, <<"rta/x11">>]}, + {<<"HTTP/2.0">>, [<<"http/2.0">>]} + ], + [{V, fun() -> R = parse_transfer_encoding(V) end} || {V, R} <- Tests]. + +parse_upgrade_error_test_() -> + Tests = [ + <<>> + ], + [{V, fun() -> {'EXIT', _} = (catch parse_upgrade(V)) end} + || V <- Tests]. +-endif. + %% Internal. %% Only return if the list is not empty. |