Merge pull request #430 from getsentry/set-logger-event-to-log-level

Set logger event level to logger message level

Author: mitchellhenke

lib/sentry/logger_backend.ex

@@ -103,6 +103,7 @@ defmodule Sentry.LoggerBackend do
     opts =
       [
         event_source: :logger,
+        level: "#{level}",
         extra: %{logger_metadata: logger_metadata(meta, state), logger_level: level},
         result: :none
       ] ++ Map.to_list(sentry)

test/logger_backend_test.exs

@@ -415,6 +415,30 @@ defmodule Sentry.LoggerBackendTest do
     end)
   end
 
+  test "sets event level to Logger message level" do
+    Logger.configure_backend(Sentry.LoggerBackend, level: :warn, capture_log_messages: true)
+    bypass = Bypass.open()
+    modify_env(:sentry, dsn: "http://public:secret@localhost:#{bypass.port}/1")
+    pid = self()
+
+    Bypass.expect_once(bypass, fn conn ->
+      {:ok, body, conn} = Plug.Conn.read_body(conn)
+      json = Jason.decode!(body)
+      assert json["message"] == "warn"
+      assert json["level"] == "warn"
+      send(pid, "API called")
+      Plug.Conn.resp(conn, 200, ~s<{"id": "340"}>)
+    end)
+
+    capture_log(fn ->
+      Logger.warn("warn")
+
+      assert_receive("API called")
+    end)
+  after
+    Logger.configure_backend(Sentry.LoggerBackend, level: :error, capture_log_messages: false)
+  end
+
   def task(parent, fun \\ fn -> raise "oops" end) do
     mon = Process.monitor(parent)
     Process.unlink(parent)