feat: add tags_from_metadata option to LoggerHandler

Adds a new option `tags_from_metadata` that allows specifying sentry tags based on already-set Logger metadata keys.

Author: icehaunter

lib/sentry/logger_handler.ex

@@ -44,6 +44,14 @@ defmodule Sentry.LoggerHandler do
       If set to `:all`, all metadata will be included.
       """
     ],
+    tags_from_metadata: [
+      type: {:list, :atom},
+      default: [],
+      doc: """
+      Use this to include logger metadata as tags in reports. Metadata under the specified keys
+      in those keys will be added as tags to the event.
+      """
+    ],
     capture_log_messages: [
       type: :boolean,
       default: false,
@@ -220,6 +228,7 @@ defmodule Sentry.LoggerHandler do
     :level,
     :excluded_domains,
     :metadata,
+    :tags_from_metadata,
     :capture_log_messages,
     :rate_limiting,
     :sync_threshold
@@ -321,7 +330,8 @@ defmodule Sentry.LoggerHandler do
             log_level,
             log_meta[:sentry],
             log_meta,
-            config.metadata
+            config.metadata,
+            config.tags_from_metadata
           )
 
         log_unfiltered(log_event, sentry_opts, config)

lib/sentry/logger_utils.ex

@@ -10,18 +10,27 @@ defmodule Sentry.LoggerUtils do
 
   require Logger
 
-  @spec build_sentry_options(Logger.level(), keyword() | nil, map(), [atom()] | :all) ::
+  @spec build_sentry_options(
+          Logger.level(),
+          keyword() | nil,
+          map(),
+          [atom()] | :all,
+          [atom()]
+        ) ::
           keyword()
-  def build_sentry_options(level, sentry_context, meta, allowed_meta) do
+  def build_sentry_options(level, sentry_context, meta, allowed_meta, allowed_tags \\ []) do
     default_extra =
       Map.merge(
         %{logger_metadata: logger_metadata(meta, allowed_meta), logger_level: level},
         Map.take(meta, [:domain])
       )
 
+    default_tags = logger_metadata(meta, allowed_tags)
+
     (sentry_context || get_sentry_options_from_callers(meta[:callers]) || %{})
     |> Map.new()
     |> Map.update(:extra, default_extra, &Map.merge(&1, default_extra))
+    |> Map.update(:tags, default_tags, &Map.merge(&1, default_tags))
     |> Map.merge(%{
       event_source: :logger,
       level: elixir_logger_level_to_sentry_level(level),

test/sentry/logger_handler_test.exs

@@ -206,6 +206,36 @@ defmodule Sentry.LoggerHandlerTest do
     end
   end
 
+  describe "with logger metadata as tags" do
+    @tag handler_config: %{capture_log_messages: true, tags_from_metadata: [:string, :number]}
+    test "includes configured Logger metadata as tags", %{sender_ref: ref} do
+      Logger.metadata(string: "value", number: 42, other: "ignored")
+      Logger.error("Testing error")
+
+      assert_receive {^ref, event}
+      assert event.tags == %{string: "value", number: 42}
+    end
+
+    @tag handler_config: %{capture_log_messages: true, tags_from_metadata: []}
+    test "does not include Logger metadata as tags when disabled",
+         %{sender_ref: ref} do
+      Logger.metadata(string: "value", number: 42)
+      Logger.error("Testing error")
+
+      assert_receive {^ref, event}
+      assert event.tags == %{}
+    end
+
+    @tag handler_config: %{capture_log_messages: true, tags_from_metadata: [:string, :number]}
+    test "merges configured tags with explicitly set tags", %{sender_ref: ref} do
+      Logger.metadata(string: "value", number: 42)
+      Logger.error("Testing error", sentry: [tags: %{explicit: "tag"}])
+
+      assert_receive {^ref, event}
+      assert event.tags == %{string: "value", number: 42, explicit: "tag"}
+    end
+  end
+
   describe "with a crashing GenServer" do
     setup do
       %{test_genserver: start_supervised!(TestGenServer, restart: :temporary)}