Make rabbit_log:log/2 return 'ok' unconditionally

instead of relying on a Lager transform-driven return value.

This should avoid erlang/otp#4576 on Erlang 24.

Per discussion with @lhoguin.

1 files changed, 4 insertions, 2 deletions

diff --git a/deps/rabbit_common/src/rabbit_log.erl b/deps/rabbit_common/src/rabbit_log.erl
index 910389b9e0..577ecb00a0 100644
--- a/deps/rabbit_common/src/rabbit_log.erl
+++ b/deps/rabbit_common/src/rabbit_log.erl
@@ -74,7 +74,8 @@ log(Category, Level, Fmt, Args) when is_list(Args) ->
         default -> ?LAGER_SINK;
         _       -> make_internal_sink_name(Category)
     end,
-    lager:log(Sink, Level, self(), Fmt, Args).
+    lager:log(Sink, Level, self(), Fmt, Args),
+    ok.
 
 %% logger(3) handler.
 log(#{level := Level,
@@ -100,7 +101,8 @@ log(#{level := Level,
             lager:log(?LAGER_SINK, Level, Pid, "~ts", [String]);
         {Format, Args} when is_list(Format) ->
             lager:log(?LAGER_SINK, Level, Pid, Format, Args)
-    end.
+    end,
+    ok.
 
 make_internal_sink_name(channel)    -> rabbit_log_channel_lager_event;
 make_internal_sink_name(connection) -> rabbit_log_connection_lager_event;