Skip to content

Commit f9229e1

Browse files
authored
Fix deprecation warning for Logger.warn/1 (#80)
1 parent 6f0df1e commit f9229e1

File tree

2 files changed

+12
-2
lines changed

2 files changed

+12
-2
lines changed

lib/telemetry_metrics_statsd.ex

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -335,6 +335,10 @@ defmodule TelemetryMetricsStatsd do
335335

336336
Record.defrecordp(:hostent, Record.extract(:hostent, from_lib: "kernel/include/inet.hrl"))
337337

338+
# TODO: remove this when we depend on Elixir 1.11+, where Logger.warning/1
339+
# was introduced.
340+
@log_level_warning if macro_exported?(Logger, :warning, 1), do: :warning, else: :warn
341+
338342
@doc """
339343
Reporter's child spec.
340344
@@ -502,7 +506,8 @@ defmodule TelemetryMetricsStatsd do
502506
end
503507

504508
{:error, reason} ->
505-
Logger.warn(
509+
Logger.log(
510+
@log_level_warning,
506511
"Failed to resolve the hostname #{host}: #{inspect(reason)}. " <>
507512
"Using the previously resolved address of #{:inet.ntoa(current_address)}."
508513
)

lib/telemetry_metrics_statsd/formatter/standard.ex

Lines changed: 6 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,6 +7,10 @@ defmodule TelemetryMetricsStatsd.Formatter.Standard do
77

88
require Logger
99

10+
# TODO: remove this when we depend on Elixir 1.11+, where Logger.warning/1
11+
# was introduced.
12+
@log_level_warning if macro_exported?(Logger, :warning, 1), do: :warning, else: :warn
13+
1014
@impl true
1115
def format(metric, value, tags) do
1216
case format_metric_tags(tags) do
@@ -83,7 +87,8 @@ defmodule TelemetryMetricsStatsd.Formatter.Standard do
8387
do: [value |> round() |> :erlang.integer_to_binary(), "|c"]
8488

8589
defp format_counter_metric_value(%Metrics.Sum{}, value) do
86-
Logger.warn(
90+
Logger.log(
91+
@log_level_warning,
8792
"Unable to format negative value: #{inspect(value)} for reporting to StatsD Counter"
8893
)
8994

0 commit comments

Comments
 (0)