From 47e0a5fb4c7788d5f45ed35ac5f6d36fb721d8bb Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Jean-S=C3=A9bastien=20P=C3=A9dron?= Date: Mon, 9 Aug 2021 18:03:11 +0200 Subject: [PATCH] Logging: Use `:` as the separator in verbosity mapping This is the same separator as the field mapping. I don't remember why I picked a different character... Now, it feels awkward and non-intuitive for users. --- .../src/rabbit_prelaunch_early_logging.erl | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/deps/rabbit/apps/rabbitmq_prelaunch/src/rabbit_prelaunch_early_logging.erl b/deps/rabbit/apps/rabbitmq_prelaunch/src/rabbit_prelaunch_early_logging.erl index 5b2c8e80b1d5..860a1e141bec 100644 --- a/deps/rabbit/apps/rabbitmq_prelaunch/src/rabbit_prelaunch_early_logging.erl +++ b/deps/rabbit/apps/rabbitmq_prelaunch/src/rabbit_prelaunch_early_logging.erl @@ -472,11 +472,11 @@ parse_json_field_mapping([], Mapping) -> %% @doc %% Parses the verbosity_map pattern. %% -%% The pattern is of the form: `debug=2 info=1 *=0'. +%% The pattern is of the form: `debug:2 info:1 *:0'. %% -%% `debug=2' means that the verbosity of the debug level is 2. +%% `debug:2' means that the verbosity of the debug level is 2. %% -%% `*=0' means that the verbosity of all non-mentionned levels is 0. +%% `*:0' means that the verbosity of all non-mentionned levels is 0. parse_json_verbosity_mapping("") -> #{}; @@ -484,7 +484,7 @@ parse_json_verbosity_mapping(RawMapping) -> parse_json_verbosity_mapping(string:split(RawMapping, " ", all), #{}). parse_json_verbosity_mapping([Entry | Rest], Mapping) -> - Mapping1 = case string:split(Entry, "=", leading) of + Mapping1 = case string:split(Entry, ":", leading) of ["*", VerbS] -> Verb = list_to_integer(VerbS), Mapping#{'$REST' => Verb};