diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/cowboy_http.erl | 6 | ||||
-rw-r--r-- | src/cowboy_http2.erl | 9 | ||||
-rw-r--r-- | src/cowboy_stream.erl | 2 |
3 files changed, 5 insertions, 12 deletions
diff --git a/src/cowboy_http.erl b/src/cowboy_http.erl index 71d33ce..b1c8dbe 100644 --- a/src/cowboy_http.erl +++ b/src/cowboy_http.erl @@ -209,8 +209,7 @@ loop(State=#state{parent=Parent, socket=Socket, transport=Transport, opts=Opts, loop(State, Buffer); %% System messages. {'EXIT', Parent, Reason} -> - %% @todo We should exit gracefully, if possible. - exit(Reason); + terminate(State, {stop, {exit, Reason}, 'Parent process terminated.'}); {system, From, Request} -> sys:handle_system_msg(Request, From, Parent, ?MODULE, [], {State, Buffer}); %% Messages pertaining to a stream. @@ -1379,8 +1378,7 @@ system_continue(_, _, {State, Buffer}) -> -spec system_terminate(any(), _, _, {#state{}, binary()}) -> no_return(). system_terminate(Reason, _, _, {State, _}) -> - %% @todo We should exit gracefully, if possible. - terminate(State, Reason). + terminate(State, {stop, {exit, Reason}, 'sys:terminate/2,3 was called.'}). -spec system_code_change(Misc, _, _, _) -> {ok, Misc} when Misc::{#state{}, binary()}. system_code_change(Misc, _, _, _) -> diff --git a/src/cowboy_http2.erl b/src/cowboy_http2.erl index 97f7c02..c7176eb 100644 --- a/src/cowboy_http2.erl +++ b/src/cowboy_http2.erl @@ -165,8 +165,7 @@ loop(State=#state{parent=Parent, socket=Socket, transport=Transport, terminate(State, {socket_error, Reason, 'An error has occurred on the socket.'}); %% System messages. {'EXIT', Parent, Reason} -> - %% @todo We should exit gracefully. - exit(Reason); + terminate(State, {stop, {exit, Reason}, 'Parent process terminated.'}); {system, From, Request} -> sys:handle_system_msg(Request, From, Parent, ?MODULE, [], {State, Buffer}); %% Timeouts. @@ -239,7 +238,6 @@ frame(State=#state{http2_machine=HTTP2Machine0}, Frame) -> {stop, Frame, 'Client is going away.'}); {send, SendData, HTTP2Machine} -> send_data(maybe_ack(State#state{http2_machine=HTTP2Machine}, Frame), SendData); - %% @todo Yeah StreamID would be better out of this, {error, StreamID, Tuple, State} {error, {stream_error, StreamID, Reason, Human}, HTTP2Machine} -> stream_reset(State#state{http2_machine=HTTP2Machine}, StreamID, {stream_error, Reason, Human}); @@ -591,8 +589,6 @@ send_response(State0, StreamID, StatusCode, Headers, Body) -> end, case Size of 0 -> - %% @todo If IsFin=fin, we may want to skip the body and terminate the stream, - %% if we are no longer tracking the stream. State = send_headers(State0, StreamID, fin, StatusCode, Headers), maybe_terminate_stream(State, StreamID, fin); _ -> @@ -783,8 +779,7 @@ system_continue(_, _, {State, Buffer}) -> -spec system_terminate(any(), _, _, {#state{}, binary()}) -> no_return(). system_terminate(Reason, _, _, {State, _}) -> - %% @todo We should exit gracefully, if possible. - terminate(State, Reason). + terminate(State, {stop, {exit, Reason}, 'sys:terminate/2,3 was called.'}). -spec system_code_change(Misc, _, _, _) -> {ok, Misc} when Misc::{#state{}, binary()}. system_code_change(Misc, _, _, _) -> diff --git a/src/cowboy_stream.erl b/src/cowboy_stream.erl index b0b570e..92f66ba 100644 --- a/src/cowboy_stream.erl +++ b/src/cowboy_stream.erl @@ -50,7 +50,7 @@ | {socket_error, closed | atom(), human_reason()} | {stream_error, cow_http2:error(), human_reason()} | {connection_error, cow_http2:error(), human_reason()} - | {stop, cow_http2:frame(), human_reason()}. + | {stop, cow_http2:frame() | {exit, any()}, human_reason()}. -export_type([reason/0]). -type partial_req() :: map(). %% @todo Take what's in cowboy_req with everything? optional. |