Breaking update of the cowboy_req interface

Simplify the interface for most cowboy_req functions. They all return
a single value except the four body reading functions. The reply functions
now only return a Req value.

Access functions do not return a Req anymore.

Functions that used to cache results do not have a cache anymore.

The interface for accessing query string and cookies has therefore
been changed.

There are now three query string functions: qs/1 provides access
to the raw query string value; parse_qs/1 returns the query string
as a list of key/values; match_qs/2 returns a map containing the
values requested in the second argument, after applying constraints
and default value.

Similarly, there are two cookie functions: parse_cookies/1 and
match_cookies/2. More match functions will be added in future commits.

None of the functions return an error tuple anymore. It either works
or crashes. Cowboy will attempt to provide an appropriate status code
in the response of crashed handlers.

As a result, the content decode function has its return value changed
to a simple binary, and the body reading functions only return on success.

1 files changed, 15 insertions, 15 deletions

diff --git a/examples/rest_pastebin/src/toppage_handler.erl b/examples/rest_pastebin/src/toppage_handler.erl
index be2b803..506fc82 100644
--- a/examples/rest_pastebin/src/toppage_handler.erl
+++ b/examples/rest_pastebin/src/toppage_handler.erl
@@ -36,37 +36,37 @@ content_types_accepted(Req, State) ->
 
 resource_exists(Req, _State) ->
 	case cowboy_req:binding(paste_id, Req) of
-		{undefined, Req2} ->
-			{true, Req2, index};
-		{PasteID, Req2} ->
+		undefined ->
+			{true, Req, index};
+		PasteID ->
 			case valid_path(PasteID) and file_exists(PasteID) of
-				true -> {true, Req2, PasteID};
-				false -> {false, Req2, PasteID}
+				true -> {true, Req, PasteID};
+				false -> {false, Req, PasteID}
 			end
 	end.
 
 create_paste(Req, State) ->
 	PasteID = new_paste_id(),
-	{ok, [{<<"paste">>, Paste}], Req3} = cowboy_req:body_qs(Req),
+	{ok, [{<<"paste">>, Paste}], Req2} = cowboy_req:body_qs(Req),
 	ok = file:write_file(full_path(PasteID), Paste),
-	case cowboy_req:method(Req3) of
-		{<<"POST">>, Req4} ->
-			{{true, <<$/, PasteID/binary>>}, Req4, State};
-		{_, Req4} ->
-			{true, Req4, State}
+	case cowboy_req:method(Req2) of
+		<<"POST">> ->
+			{{true, <<$/, PasteID/binary>>}, Req2, State};
+		_ ->
+			{true, Req2, State}
 	end.
 
 paste_html(Req, index) ->
 	{read_file("index.html"), Req, index};
 paste_html(Req, Paste) ->
-	{Style, Req2} = cowboy_req:qs_val(<<"lang">>, Req, plain),
-	{format_html(Paste, Style), Req2, Paste}.
+	#{lang := Lang} = cowboy_req:match_qs(Req, [lang]),
+	{format_html(Paste, Lang), Req, Paste}.
 
 paste_text(Req, index) ->
 	{read_file("index.txt"), Req, index};
 paste_text(Req, Paste) ->
-	{Style, Req2} = cowboy_req:qs_val(<<"lang">>, Req, plain),
-	{format_text(Paste, Style), Req2, Paste}.
+	#{lang := Lang} = cowboy_req:match_qs(Req, [lang]),
+	{format_text(Paste, Lang), Req, Paste}.
 
 % Private