Skip to content

Commit 24888cf

Browse files
authored
Drop Logger backends (#4862)
* drop console logger backend * ah, switch Sentry too
1 parent 7f4c2b6 commit 24888cf

2 files changed

Lines changed: 7 additions & 16 deletions

File tree

config/runtime.exs

Lines changed: 4 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -26,26 +26,15 @@ log_level =
2626
|> get_var_from_path_or_env("LOG_LEVEL", default_log_level)
2727
|> String.to_existing_atom()
2828

29-
config :logger,
30-
level: log_level,
31-
backends: [:console]
32-
33-
config :logger, Sentry.LoggerBackend,
34-
capture_log_messages: true,
35-
level: :error
29+
config :logger, level: log_level
30+
config :logger, :default_formatter, metadata: [:request_id]
3631

3732
case String.downcase(log_format) do
3833
"standard" ->
39-
config :logger, :console,
40-
format: "$time $metadata[$level] $message\n",
41-
metadata: [:request_id]
34+
config :logger, :default_formatter, format: "$time $metadata[$level] $message\n"
4235

4336
"json" ->
44-
config :logger, :console,
45-
format: {ExJsonLogger, :format},
46-
metadata: [
47-
:request_id
48-
]
37+
config :logger, :default_formatter, format: {ExJsonLogger, :format}
4938
end
5039

5140
# Listen IP supports IPv4 and IPv6 addresses.

lib/plausible/application.ex

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -226,7 +226,9 @@ defmodule Plausible.Application do
226226
end
227227

228228
def setup_sentry() do
229-
Logger.add_backend(Sentry.LoggerBackend)
229+
:logger.add_handler(:sentry_handler, Sentry.LoggerHandler, %{
230+
config: %{capture_log_messages: true, level: :error}
231+
})
230232

231233
:telemetry.attach_many(
232234
"oban-errors",

0 commit comments

Comments
 (0)