From 97a3108576c6a9d64c03e1455654dba88367992a Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Hoguin?= Date: Sat, 12 Jul 2014 12:09:43 +0200 Subject: Reply with 400 on header parsing crash This is a first step to improve the HTTP status codes returned by Cowboy on crashes. We will tweak it over time. Also fixes a small bug where two replies may have been sent when using loop handlers under rare conditions. --- src/cowboy_spdy.erl | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) (limited to 'src/cowboy_spdy.erl') diff --git a/src/cowboy_spdy.erl b/src/cowboy_spdy.erl index faea142..8da9613 100644 --- a/src/cowboy_spdy.erl +++ b/src/cowboy_spdy.erl @@ -422,7 +422,7 @@ execute(Req, Env, [Middleware|Tail]) -> {halt, Req2} -> cowboy_req:ensure_response(Req2, 204); {error, Status, Req2} -> - cowboy_req:maybe_reply(Status, Req2) + cowboy_req:reply(Status, Req2) end. -spec resume(cowboy_middleware:env(), [module()], @@ -437,7 +437,7 @@ resume(Env, Tail, Module, Function, Args) -> {halt, Req2} -> cowboy_req:ensure_response(Req2, 204); {error, Status, Req2} -> - cowboy_req:maybe_reply(Status, Req2) + cowboy_req:reply(Status, Req2) end. %% Reply functions used by cowboy_req. -- cgit v1.2.3