From f65fcffbfcbec6817b4bc09e5e82b8d224f2e78c Mon Sep 17 00:00:00 2001 From: Jamil Date: Sun, 1 Jun 2025 12:32:08 -0700 Subject: [PATCH] fix(portal): fix sentry before_send when message is nil (#9349) A regression was introduced in #9242 where it appears that some Sentry events don't contain messages, so the filtering module is updated only to act on events with messages. --- elixir/apps/domain/lib/domain/telemetry/sentry.ex | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) diff --git a/elixir/apps/domain/lib/domain/telemetry/sentry.ex b/elixir/apps/domain/lib/domain/telemetry/sentry.ex index 6f5dd4042..b7b5eca46 100644 --- a/elixir/apps/domain/lib/domain/telemetry/sentry.ex +++ b/elixir/apps/domain/lib/domain/telemetry/sentry.ex @@ -3,9 +3,9 @@ defmodule Domain.Telemetry.Sentry do nil end - def before_send(event) do + def before_send(%{message: message} = event) when is_binary(message) do if String.contains?( - event.message, + message, "Node ~p not responding **~n** Removing (timedout) connection" ) do # This happens when libcluster loses connection to a node, which is normal during deploys. @@ -15,4 +15,6 @@ defmodule Domain.Telemetry.Sentry do event end end + + def before_send(event), do: event end