From 9339e88a5a26ff77a5e40149cbb1a5b339b7ec6a Mon Sep 17 00:00:00 2001
From: "Alessandro (Ale) Segala" <43508+ItalyPaleAle@users.noreply.github.com>
Date: Wed, 20 Aug 2025 15:01:53 -0700
Subject: [PATCH] fix: move audit log call before TX is committed (#854)

---
 backend/internal/service/webauthn_service.go | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/backend/internal/service/webauthn_service.go b/backend/internal/service/webauthn_service.go
index f6fd742a..f87652c6 100644
--- a/backend/internal/service/webauthn_service.go
+++ b/backend/internal/service/webauthn_service.go
@@ -263,13 +263,13 @@ func (s *WebAuthnService) VerifyLogin(ctx context.Context, sessionID string, cre
 		return model.User{}, "", err
 	}
 
+	s.auditLogService.CreateNewSignInWithEmail(ctx, ipAddress, userAgent, user.ID, tx)
+
 	err = tx.Commit().Error
 	if err != nil {
 		return model.User{}, "", err
 	}
 
-	s.auditLogService.CreateNewSignInWithEmail(ctx, ipAddress, userAgent, user.ID, tx)
-
 	return *user, token, nil
 }