diff options
Diffstat (limited to 'src')
-rw-r--r-- | src/ranch.erl | 18 | ||||
-rw-r--r-- | src/ranch_acceptors_sup.erl | 10 | ||||
-rw-r--r-- | src/ranch_conns_sup.erl | 35 | ||||
-rw-r--r-- | src/ranch_conns_sup_sup.erl | 15 | ||||
-rw-r--r-- | src/ranch_listener_sup.erl | 13 | ||||
-rw-r--r-- | src/ranch_server.erl | 2 |
6 files changed, 58 insertions, 35 deletions
diff --git a/src/ranch.erl b/src/ranch.erl index 0e3541f..94727ea 100644 --- a/src/ranch.erl +++ b/src/ranch.erl @@ -291,16 +291,22 @@ set_max_connections(Ref, MaxConnections) -> get_transport_options(Ref) -> ranch_server:get_transport_options(Ref). --spec set_transport_options(ref(), opts()) -> ok | {error, running}. +-spec set_transport_options(ref(), opts()) -> ok | {error, term()}. set_transport_options(Ref, TransOpts0) -> TransOpts = normalize_opts(TransOpts0), - case get_status(Ref) of - suspended -> - ok = ranch_server:set_transport_options(Ref, TransOpts); - running -> - {error, running} + case validate_transport_opts(TransOpts) of + ok -> + ok = ranch_server:set_transport_options(Ref, TransOpts), + ok = apply_transport_options(Ref, TransOpts); + TransOptsError -> + TransOptsError end. +apply_transport_options(Ref, TransOpts) -> + _ = [ConnsSup ! {set_transport_options, TransOpts} + || {_, ConnsSup} <- ranch_server:get_connections_sups(Ref)], + ok. + -spec get_protocol_options(ref()) -> any(). get_protocol_options(Ref) -> ranch_server:get_protocol_options(Ref). diff --git a/src/ranch_acceptors_sup.erl b/src/ranch_acceptors_sup.erl index e08c2ec..b6531f8 100644 --- a/src/ranch_acceptors_sup.erl +++ b/src/ranch_acceptors_sup.erl @@ -18,15 +18,15 @@ -export([start_link/3]). -export([init/1]). --spec start_link(ranch:ref(), pos_integer(), module()) +-spec start_link(ranch:ref(), module(), module()) -> {ok, pid()}. -start_link(Ref, NumAcceptors, Transport) -> - supervisor:start_link(?MODULE, [Ref, NumAcceptors, Transport]). +start_link(Ref, Transport, Logger) -> + supervisor:start_link(?MODULE, [Ref, Transport, Logger]). -spec init([term()]) -> {ok, {supervisor:sup_flags(), [supervisor:child_spec()]}}. -init([Ref, NumAcceptors, Transport]) -> +init([Ref, Transport, Logger]) -> TransOpts = ranch_server:get_transport_options(Ref), - Logger = maps:get(logger, TransOpts, logger), + NumAcceptors = maps:get(num_acceptors, TransOpts, 10), NumListenSockets = maps:get(num_listen_sockets, TransOpts, 1), %% We temporarily put the logger in the process dictionary %% so that it can be used from ranch:filter_options. The diff --git a/src/ranch_conns_sup.erl b/src/ranch_conns_sup.erl index a585e3f..389deb0 100644 --- a/src/ranch_conns_sup.erl +++ b/src/ranch_conns_sup.erl @@ -18,12 +18,12 @@ -module(ranch_conns_sup). %% API. --export([start_link/4]). +-export([start_link/6]). -export([start_protocol/3]). -export([active_connections/1]). %% Supervisor internals. --export([init/5]). +-export([init/7]). -export([system_continue/3]). -export([system_terminate/4]). -export([system_code_change/4]). @@ -46,10 +46,10 @@ %% API. --spec start_link(ranch:ref(), pos_integer(), module(), module()) -> {ok, pid()}. -start_link(Ref, Id, Transport, Protocol) -> +-spec start_link(ranch:ref(), pos_integer(), module(), any(), module(), module()) -> {ok, pid()}. +start_link(Ref, Id, Transport, TransOpts, Protocol, Logger) -> proc_lib:start_link(?MODULE, init, - [self(), Ref, Id, Transport, Protocol]). + [self(), Ref, Id, Transport, TransOpts, Protocol, Logger]). %% We can safely assume we are on the same node as the supervisor. %% @@ -99,16 +99,14 @@ active_connections(SupPid) -> %% Supervisor internals. --spec init(pid(), ranch:ref(), pos_integer(), module(), module()) -> no_return(). -init(Parent, Ref, Id, Transport, Protocol) -> +-spec init(pid(), ranch:ref(), pos_integer(), module(), any(), module(), module()) -> no_return(). +init(Parent, Ref, Id, Transport, TransOpts, Protocol, Logger) -> process_flag(trap_exit, true), ok = ranch_server:set_connections_sup(Ref, Id, self()), MaxConns = ranch_server:get_max_connections(Ref), - TransOpts = ranch_server:get_transport_options(Ref), ConnType = maps:get(connection_type, TransOpts, worker), Shutdown = maps:get(shutdown, TransOpts, 5000), HandshakeTimeout = maps:get(handshake_timeout, TransOpts, 5000), - Logger = maps:get(logger, TransOpts, logger), ProtoOpts = ranch_server:get_protocol_options(Ref), ok = proc_lib:init_ack(Parent, {ok, self()}), loop(#state{parent=Parent, ref=Ref, conn_type=ConnType, @@ -166,8 +164,11 @@ loop(State=#state{parent=Parent, ref=Ref, conn_type=ConnType, {set_max_conns, MaxConns2} -> loop(State#state{max_conns=MaxConns2}, CurConns, NbChildren, Sleepers); + %% Upgrade the transport options. + {set_transport_options, TransOpts} -> + set_transport_options(State, CurConns, NbChildren, Sleepers, TransOpts); %% Upgrade the protocol options. - {set_opts, Opts2} -> + {set_protocol_options, Opts2} -> loop(State#state{opts=Opts2}, CurConns, NbChildren, Sleepers); {'EXIT', Parent, Reason} -> @@ -250,6 +251,20 @@ handshake(State=#state{ref=Ref, transport=Transport, handshake_timeout=Handshake loop(State, CurConns, NbChildren, Sleepers) end. +set_transport_options(State=#state{max_conns=MaxConns0}, CurConns, NbChildren, Sleepers0, TransOpts) -> + MaxConns1 = maps:get(max_connections, TransOpts, 1024), + HandshakeTimeout = maps:get(handshake_timeout, TransOpts, 5000), + Shutdown = maps:get(shutdown, TransOpts, 5000), + Sleepers1 = case MaxConns1 > MaxConns0 of + true -> + _ = [To ! self() || To <- Sleepers0], + []; + false -> + Sleepers0 + end, + loop(State#state{max_conns=MaxConns1, handshake_timeout=HandshakeTimeout, shutdown=Shutdown}, + CurConns, NbChildren, Sleepers1). + -spec terminate(#state{}, any(), non_neg_integer()) -> no_return(). terminate(#state{shutdown=brutal_kill}, Reason, _) -> kill_children(get_keys(active)), diff --git a/src/ranch_conns_sup_sup.erl b/src/ranch_conns_sup_sup.erl index 4b1dd47..61ace8e 100644 --- a/src/ranch_conns_sup_sup.erl +++ b/src/ranch_conns_sup_sup.erl @@ -19,19 +19,22 @@ -export([start_link/4]). -export([init/1]). --spec start_link(ranch:ref(), pos_integer(), ranch:opts(), module()) -> {ok, pid()}. -start_link(Ref, NumConnsSups, Transport, Protocol) -> +-spec start_link(ranch:ref(), module(), module(), module()) -> {ok, pid()}. +start_link(Ref, Transport, Protocol, Logger) -> ok = ranch_server:cleanup_connections_sups(Ref), supervisor:start_link(?MODULE, { - Ref, NumConnsSups, Transport, Protocol + Ref, Transport, Protocol, Logger }). --spec init({ranch:ref(), pos_integer(), module(), module()}) +-spec init({ranch:ref(), module(), module(), module()}) -> {ok, {supervisor:sup_flags(), [supervisor:child_spec()]}}. -init({Ref, NumConnsSups, Transport, Protocol}) -> +init({Ref, Transport, Protocol, Logger}) -> + TransOpts = ranch_server:get_transport_options(Ref), + NumAcceptors = maps:get(num_acceptors, TransOpts, 10), + NumConnsSups = maps:get(num_conns_sups, TransOpts, NumAcceptors), ChildSpecs = [#{ id => {ranch_conns_sup, N}, - start => {ranch_conns_sup, start_link, [Ref, N, Transport, Protocol]}, + start => {ranch_conns_sup, start_link, [Ref, N, Transport, TransOpts, Protocol, Logger]}, type => supervisor } || N <- lists:seq(1, NumConnsSups)], {ok, {#{intensity => 1 + ceil(math:log2(NumConnsSups))}, ChildSpecs}}. diff --git a/src/ranch_listener_sup.erl b/src/ranch_listener_sup.erl index f4d8ebc..393099f 100644 --- a/src/ranch_listener_sup.erl +++ b/src/ranch_listener_sup.erl @@ -21,28 +21,27 @@ -spec start_link(ranch:ref(), module(), any(), module(), any()) -> {ok, pid()}. start_link(Ref, Transport, TransOpts, Protocol, ProtoOpts) -> - NumAcceptors = maps:get(num_acceptors, TransOpts, 10), - NumConnsSups = maps:get(num_conns_sups, TransOpts, NumAcceptors), MaxConns = maps:get(max_connections, TransOpts, 1024), + Logger = maps:get(logger, TransOpts, logger), ranch_server:set_new_listener_opts(Ref, MaxConns, TransOpts, ProtoOpts, [Ref, Transport, TransOpts, Protocol, ProtoOpts]), supervisor:start_link(?MODULE, { - Ref, NumAcceptors, NumConnsSups, Transport, Protocol + Ref, Transport, Protocol, Logger }). --spec init({ranch:ref(), pos_integer(), pos_integer(), module(), module()}) +-spec init({ranch:ref(), module(), module(), module()}) -> {ok, {supervisor:sup_flags(), [supervisor:child_spec()]}}. -init({Ref, NumAcceptors, NumConnsSups, Transport, Protocol}) -> +init({Ref, Transport, Protocol, Logger}) -> ok = ranch_server:set_listener_sup(Ref, self()), ChildSpecs = [ #{ id => ranch_conns_sup_sup, - start => {ranch_conns_sup_sup, start_link, [Ref, NumConnsSups, Transport, Protocol]}, + start => {ranch_conns_sup_sup, start_link, [Ref, Transport, Protocol, Logger]}, type => supervisor }, #{ id => ranch_acceptors_sup, - start => {ranch_acceptors_sup, start_link, [Ref, NumAcceptors, Transport]}, + start => {ranch_acceptors_sup, start_link, [Ref, Transport, Logger]}, type => supervisor } ], diff --git a/src/ranch_server.erl b/src/ranch_server.erl index ae68e99..7bf8a96 100644 --- a/src/ranch_server.erl +++ b/src/ranch_server.erl @@ -202,7 +202,7 @@ handle_call({set_trans_opts, Ref, Opts}, _, State) -> {reply, ok, State}; handle_call({set_proto_opts, Ref, Opts}, _, State) -> ets:insert(?TAB, {{proto_opts, Ref}, Opts}), - _ = [ConnsSup ! {set_opts, Opts} || {_, ConnsSup} <- get_connections_sups(Ref)], + _ = [ConnsSup ! {set_protocol_options, Opts} || {_, ConnsSup} <- get_connections_sups(Ref)], {reply, ok, State}; handle_call(_Request, _From, State) -> {reply, ignore, State}. |