From 7e9de8e95b7b67ae7b078c2ec2d1ff16e522cd53 Mon Sep 17 00:00:00 2001 From: Moritz Date: Wed, 21 Jan 2026 08:02:33 +0100 Subject: [PATCH] Add logging for fail-open email uniqueness validations Log warnings when query errors occur in email uniqueness checks to improve visibility of data integrity issues --- .../user/validations/email_not_used_by_other_member.ex | 8 +++++++- .../member/validations/email_not_used_by_other_user.ex | 8 +++++++- 2 files changed, 14 insertions(+), 2 deletions(-) diff --git a/lib/mv/accounts/user/validations/email_not_used_by_other_member.ex b/lib/mv/accounts/user/validations/email_not_used_by_other_member.ex index 8c28af6..9f900eb 100644 --- a/lib/mv/accounts/user/validations/email_not_used_by_other_member.ex +++ b/lib/mv/accounts/user/validations/email_not_used_by_other_member.ex @@ -91,7 +91,13 @@ defmodule Mv.Accounts.User.Validations.EmailNotUsedByOtherMember do {:ok, _} -> {:error, field: :email, message: "is already used by another member", value: email} - {:error, _} -> + {:error, reason} -> + require Logger + + Logger.warning( + "Email uniqueness validation query failed for user email '#{email}': #{inspect(reason)}. Allowing operation to proceed (fail-open)." + ) + :ok end end diff --git a/lib/mv/membership/member/validations/email_not_used_by_other_user.ex b/lib/mv/membership/member/validations/email_not_used_by_other_user.ex index 3e6ae58..78af70c 100644 --- a/lib/mv/membership/member/validations/email_not_used_by_other_user.ex +++ b/lib/mv/membership/member/validations/email_not_used_by_other_user.ex @@ -63,7 +63,13 @@ defmodule Mv.Membership.Member.Validations.EmailNotUsedByOtherUser do {:ok, _} -> {:error, field: :email, message: "is already used by another user", value: email} - {:error, _} -> + {:error, reason} -> + require Logger + + Logger.warning( + "Email uniqueness validation query failed for member email '#{email}': #{inspect(reason)}. Allowing operation to proceed (fail-open)." + ) + :ok end end