feat: add support for elixir 1.15

Elixir 1.15 hard-depcreates Logger.warn.  Custom logger backends that
still use the legacy custom backend API must translate the log level to
avoid emitting a deprecation warning.

Co-authored-by: José Valim <[email protected]>

Author: dustinfarris

.github/workflows/main.yml

@@ -16,13 +16,25 @@ jobs:
     strategy:
       matrix:
         # https://hexdocs.pm/elixir/1.14/compatibility-and-deprecations.html#compatibility-between-elixir-and-erlang-otp
-        elixir: ['1.12.x', '1.13.x', '1.14.x']
-        otp: ['22.3.x', '23.3.x', '24.3.x', '25.2.x']
+        elixir: ['1.12.x', '1.13.x', '1.14.x', '1.15.x']
+        otp: ['22.3.x', '23.3.x', '24.3.x', '25.2.x', '26.0.x']
         exclude:
           - elixir: '1.12.x'
             otp: '25.2.x'
+          - elixir: '1.12.x'
+            otp: '26.0.x'
+          - elixir: '1.13.x'
+            otp: '25.2.x'
+          - elixir: '1.13.x'
+            otp: '26.0.x'
           - elixir: '1.14.x'
             otp: '22.3.x'
+          - elixir: '1.14.x'
+            otp: '26.0.x'
+          - elixir: '1.15.x'
+            otp: '22.3.x'
+          - elixir: '1.15.x'
+            otp: '23.3.x'
         include:
           - elixir: '1.10.x'
             otp: '22.3.x'
@@ -55,7 +67,7 @@ jobs:
         mix deps.get
     - name: Check Formatting
       run: mix format --check-formatted
-      if: matrix.elixir == '1.14.x'
+      if: matrix.elixir == '1.15.x'
 
     - name: Check Compile Warnings
       run: mix compile --warnings-as-errors
@@ -66,16 +78,16 @@ jobs:
     - name: Retrieve PLT Cache
       uses: actions/cache@v1
       id: plt-cache
-      if: matrix.elixir == '1.14.x'
+      if: matrix.elixir == '1.15.x'
       with:
         path: priv/plts
         key: ${{ runner.os }}-${{ matrix.otp }}-${{ matrix.elixir }}-plts-${{ hashFiles(format('{0}{1}', github.workspace, '/mix.lock')) }}
 
     - name: Create PLTs
-      if: steps.plt-cache.outputs.cache-hit != 'true' && matrix.elixir == '1.14.x'
+      if: steps.plt-cache.outputs.cache-hit != 'true' && matrix.elixir == '1.15.x'
       run: |
         mkdir -p priv/plts
         mix dialyzer --plt
     - name: Run dialyzer
       run: mix dialyzer --no-check --halt-exit-status
-      if: matrix.elixir == '1.14.x'
+      if: matrix.elixir == '1.15.x'

README.md

@@ -175,7 +175,7 @@ The full range of options is the following:
 | `source_code_path_pattern` | False  | `"**/*.ex"` | |
 | `filter` | False | | Module where the filter rules are defined (see [Filtering Exceptions](https://hexdocs.pm/sentry/Sentry.html#module-filtering-exceptions)) |
 | `json_library` | False | `Jason` | |
-| `log_level` | False | `:warn` | This sets the log level used when Sentry fails to send an event due to an invalid event or API error |
+| `log_level` | False | `:warning` | This sets the log level used when Sentry fails to send an event due to an invalid event or API error |
 | `max_breadcrumbs` | False | 100 | This sets the maximum number of breadcrumbs to send to Sentry when creating an event |
 
 Sentry uses the [hackney HTTP client](https://github.com/benoitc/hackney) for HTTP requests.  Sentry starts its own hackney pool named `:sentry_pool` with a default connection pool of 50, and a connection timeout of 5000 milliseconds.  The pool can be configured with the `hackney_pool_max_connections` and `hackney_pool_timeout` configuration keys.  If you need to set other [hackney configurations](https://github.com/benoitc/hackney/blob/master/doc/hackney.md#request5) for things like a proxy, using your own pool or response timeouts, the `hackney_opts` configuration is passed directly to hackney for each request.

lib/sentry/config.ex

@@ -18,6 +18,8 @@ defmodule Sentry.Config do
 
   @permitted_log_level_values ~w(debug info warning warn error)a
 
+  @warning_log_level if Version.compare(System.version(), "1.11.0") != :lt, do: :warning, else: :warn
+
   def validate_config! do
   end
 
@@ -198,8 +200,9 @@ defmodule Sentry.Config do
     get_config(:json_library, default: Jason, check_dsn: false)
   end
 
+
   def log_level do
-    get_config(:log_level, default: :warn, check_dsn: false)
+    get_config(:log_level, default: @warning_log_level, check_dsn: false)
   end
 
   def max_breadcrumbs do

lib/sentry/logger_backend.ex

@@ -34,8 +34,8 @@ defmodule Sentry.LoggerBackend do
   Example:
 
       config :logger, Sentry.LoggerBackend,
-        # Also send warn messages
-        level: :warn,
+        # Also send warning messages
+        level: :warning,
         # Send messages from Plug/Cowboy
         excluded_domains: [],
         # Include metadata added with `Logger.metadata([foo_bar: "value"])`
@@ -70,6 +70,8 @@ defmodule Sentry.LoggerBackend do
   end
 
   def handle_event({level, _gl, {Logger, msg, _ts, meta}}, state) do
+    level = maybe_ensure_warning_level(level)
+
     if Logger.compare_levels(level, state.level) != :lt and
          not excluded_domain?(meta[:domain], state) do
       log(level, msg, meta, state)
@@ -189,4 +191,10 @@ defmodule Sentry.LoggerBackend do
         "debug"
     end
   end
+
+  if Version.compare(System.version(), "1.11.0") != :lt do
+    defp maybe_ensure_warning_level(:warn), do: :warning
+  end
+
+  defp maybe_ensure_warning_level(level), do: level
 end

mix.lock

@@ -26,7 +26,7 @@
   "plug_cowboy": {:hex, :plug_cowboy, "2.4.1", "779ba386c0915027f22e14a48919a9545714f849505fa15af2631a0d298abf0f", [:mix], [{:cowboy, "~> 2.7", [hex: :cowboy, repo: "hexpm", optional: false]}, {:cowboy_telemetry, "~> 0.3", [hex: :cowboy_telemetry, repo: "hexpm", optional: false]}, {:plug, "~> 1.7", [hex: :plug, repo: "hexpm", optional: false]}, {:telemetry, "~> 0.4", [hex: :telemetry, repo: "hexpm", optional: false]}], "hexpm", "d72113b6dff7b37a7d9b2a5b68892808e3a9a752f2bf7e503240945385b70507"},
   "plug_crypto": {:hex, :plug_crypto, "1.2.2", "05654514ac717ff3a1843204b424477d9e60c143406aa94daf2274fdd280794d", [:mix], [], "hexpm", "87631c7ad914a5a445f0a3809f99b079113ae4ed4b867348dd9eec288cecb6db"},
   "ranch": {:hex, :ranch, "1.7.1", "6b1fab51b49196860b733a49c07604465a47bdb78aa10c1c16a3d199f7f8c881", [:rebar3], [], "hexpm", "451d8527787df716d99dc36162fca05934915db0b6141bbdac2ea8d3c7afc7d7"},
-  "ssl_verify_fun": {:hex, :ssl_verify_fun, "1.1.6", "cf344f5692c82d2cd7554f5ec8fd961548d4fd09e7d22f5b62482e5aeaebd4b0", [:make, :mix, :rebar3], [], "hexpm", "bdb0d2471f453c88ff3908e7686f86f9be327d065cc1ec16fa4540197ea04680"},
+  "ssl_verify_fun": {:hex, :ssl_verify_fun, "1.1.7", "354c321cf377240c7b8716899e182ce4890c5938111a1296add3ec74cf1715df", [:make, :mix, :rebar3], [], "hexpm", "fe4c190e8f37401d30167c8c405eda19469f34577987c76dde613e838bbc67f8"},
   "ssl_verify_hostname": {:hex, :ssl_verify_hostname, "1.0.5", "2e73e068cd6393526f9fa6d399353d7c9477d6886ba005f323b592d389fb47be", [:make], []},
   "telemetry": {:hex, :telemetry, "0.4.2", "2808c992455e08d6177322f14d3bdb6b625fbcfd233a73505870d8738a2f4599", [:rebar3], [], "hexpm", "2d1419bd9dda6a206d7b5852179511722e2b18812310d304620c7bd92a13fcef"},
   "unicode_util_compat": {:hex, :unicode_util_compat, "0.7.0", "bc84380c9ab48177092f43ac89e4dfa2c6d62b40b8bd132b1059ecc7232f9a78", [:rebar3], [], "hexpm", "25eee6d67df61960cf6a794239566599b09e17e668d3700247bc498638152521"},

test/event_test.exs

@@ -52,7 +52,7 @@ defmodule Sentry.EventTest do
                %{
                  context_line: nil,
                  filename: "timer.erl",
-                 function: ":timer.tc/1",
+                 function: ":timer.tc" <> _arity,
                  in_app: false,
                  lineno: _,
                  module: :timer,

test/logger_backend_test.exs

@@ -6,6 +6,8 @@ defmodule Sentry.LoggerBackendTest do
 
   alias Sentry.Envelope
 
+  @warning_log_level if Version.compare(System.version(), "1.11.0") != :lt, do: :warning, else: :warn
+
   setup do
     {:ok, _} = Logger.add_backend(Sentry.LoggerBackend)
 
@@ -399,8 +401,8 @@ defmodule Sentry.LoggerBackendTest do
     Logger.configure_backend(Sentry.LoggerBackend, capture_log_messages: false)
   end
 
-  test "sends warning messages when configured to :warn" do
-    Logger.configure_backend(Sentry.LoggerBackend, level: :warn, capture_log_messages: true)
+  test "sends warning messages when configured to #{@warning_log_level}" do
+    Logger.configure_backend(Sentry.LoggerBackend, level: @warning_log_level, capture_log_messages: true)
     bypass = Bypass.open()
     modify_env(:sentry, dsn: "http://public:secret@localhost:#{bypass.port}/1")
     pid = self()
@@ -421,7 +423,7 @@ defmodule Sentry.LoggerBackendTest do
 
     capture_log(fn ->
       Sentry.Context.set_user_context(%{user_id: 3})
-      Logger.warn("testing")
+      Logger.log(@warning_log_level, "testing")
       assert_receive("API called")
     end)
   after
@@ -546,7 +548,7 @@ defmodule Sentry.LoggerBackendTest do
   end
 
   test "sets event level to Logger message level" do
-    Logger.configure_backend(Sentry.LoggerBackend, level: :warn, capture_log_messages: true)
+    Logger.configure_backend(Sentry.LoggerBackend, level: @warning_log_level, capture_log_messages: true)
     bypass = Bypass.open()
     modify_env(:sentry, dsn: "http://public:secret@localhost:#{bypass.port}/1")
     pid = self()
@@ -566,8 +568,7 @@ defmodule Sentry.LoggerBackendTest do
     end)
 
     capture_log(fn ->
-      Logger.warn("warn")
-
+      Logger.log(@warning_log_level, "warn")
       assert_receive("API called")
     end)
   after