From 76eb7fa5f5c62e5c756838bf5c2c7a0df7c6d7bd Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Lo=C3=AFc=20Hoguin?= Date: Sun, 12 May 2019 19:03:40 +0200 Subject: Properly document logger option; change default to logger We now require OTP-21+ therefore there's no need to keep the default of error_logger. --- src/ranch.erl | 7 +++---- src/ranch_acceptors_sup.erl | 2 +- src/ranch_conns_sup.erl | 2 +- 3 files changed, 5 insertions(+), 6 deletions(-) (limited to 'src') diff --git a/src/ranch.erl b/src/ranch.erl index fc6a3c8..a0fdea6 100644 --- a/src/ranch.erl +++ b/src/ranch.erl @@ -379,7 +379,7 @@ filter_user_options([], _) -> filter_options_warning(Opt) -> Logger = case get(logger) of - undefined -> error_logger; + undefined -> logger; Logger0 -> Logger0 end, log(warning, @@ -416,9 +416,8 @@ log(Level, Format, Args, #{logger := Logger}) when Logger =/= error_logger -> _ = Logger:Level(Format, Args), ok; -%% We use error_logger by default. Because error_logger does -%% not have all the levels we accept we have to do some -%% mapping to error_logger functions. +%% Because error_logger does not have all the levels +%% we accept we have to do some mapping to error_logger functions. log(Level, Format, Args, _) -> Function = case Level of emergency -> error_msg; diff --git a/src/ranch_acceptors_sup.erl b/src/ranch_acceptors_sup.erl index 8da93e3..965862a 100644 --- a/src/ranch_acceptors_sup.erl +++ b/src/ranch_acceptors_sup.erl @@ -25,7 +25,7 @@ start_link(Ref, NumAcceptors, Transport) -> init([Ref, NumAcceptors, Transport]) -> TransOpts = ranch_server:get_transport_options(Ref), - Logger = maps:get(logger, TransOpts, error_logger), + Logger = maps:get(logger, TransOpts, logger), NumListenSockets = maps:get(num_listen_sockets, TransOpts, 1), SocketOpts = maps:get(socket_opts, TransOpts, []), %% We temporarily put the logger in the process dictionary diff --git a/src/ranch_conns_sup.erl b/src/ranch_conns_sup.erl index 70947e1..a5c2206 100644 --- a/src/ranch_conns_sup.erl +++ b/src/ranch_conns_sup.erl @@ -108,7 +108,7 @@ init(Parent, Ref, Id, Transport, Protocol) -> 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, error_logger), + 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, -- cgit v1.2.3