From 5c315ab1425965dfb2c10b274185a3fdf2645d1d Mon Sep 17 00:00:00 2001
From: =?UTF-8?q?Lo=C3=AFc=20Hoguin?= <essen@ninenines.eu>
Date: Fri, 30 Nov 2012 16:44:57 +0100
Subject: Return a 400 error if the Accept header was incorrect

---
 src/cowboy_rest.erl | 9 +++++----
 1 file changed, 5 insertions(+), 4 deletions(-)

(limited to 'src/cowboy_rest.erl')

diff --git a/src/cowboy_rest.erl b/src/cowboy_rest.erl
index 446558c..721acb9 100644
--- a/src/cowboy_rest.erl
+++ b/src/cowboy_rest.erl
@@ -211,14 +211,15 @@ content_types_provided(Req, State) ->
 		    CTP2 = [normalize_content_types(P) || P <- CTP],
 			State2 = State#state{
 				handler_state=HandlerState, content_types_p=CTP2},
-			{ok, Accept, Req3} = cowboy_req:parse_header(<<"accept">>, Req2),
-			case Accept of
-				undefined ->
+			case cowboy_req:parse_header(<<"accept">>, Req2) of
+				{error, badarg} ->
+					respond(Req2, State2, 400);
+				{ok, undefined, Req3} ->
 					{PMT, _Fun} = HeadCTP = hd(CTP2),
 					languages_provided(
 						cowboy_req:set_meta(media_type, PMT, Req3),
 						State2#state{content_type_a=HeadCTP});
-				Accept ->
+				{ok, Accept, Req3} ->
 					Accept2 = prioritize_accept(Accept),
 					choose_media_type(Req3, State2, Accept2)
 			end
-- 
cgit v1.2.3