From 7c9db9d2cd26812eb98b7d622afa977ebab762dd Mon Sep 17 00:00:00 2001 From: byronantak Date: Thu, 23 Jan 2025 17:20:56 +0000 Subject: [PATCH] I18N-1308: Add more debug logs to security methods (#228) Add more logging to help pinpoint where the failures are happening in the security layer --- .../l10n/mojito/security/UserDetailImplOAuth2UserService.java | 2 ++ .../com/box/l10n/mojito/security/UserDetailsServiceImpl.java | 3 +++ 2 files changed, 5 insertions(+) diff --git a/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailImplOAuth2UserService.java b/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailImplOAuth2UserService.java index c7e6bc9083..bdff2ee574 100644 --- a/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailImplOAuth2UserService.java +++ b/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailImplOAuth2UserService.java @@ -75,6 +75,7 @@ public UserDetailImplOAuth2UserService(SecurityConfig securityConfig, UserServic @Override public OAuth2User loadUser(OAuth2UserRequest userRequest) throws OAuth2AuthenticationException { + logger.debug("OAuth2UserRequest flow initiated"); Assert.notNull(userRequest, "userRequest cannot be null"); if (!StringUtils.hasText( @@ -145,6 +146,7 @@ public OAuth2User loadUser(OAuth2UserRequest userRequest) throws OAuth2Authentic Map userAttributes = response.getBody(); // This is the part of the implementation that diverge from {@link DefaultOAuth2UserService} + logger.debug("Successful OAuth2UserRequest flow"); logger.debug("user attributes: {}", userAttributes); return getOAuth2UserDetailsImpl(userRequest, userNameAttributeName, userAttributes); } diff --git a/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailsServiceImpl.java b/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailsServiceImpl.java index a2c6136b0c..8bb0a132fc 100644 --- a/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailsServiceImpl.java +++ b/webapp/src/main/java/com/box/l10n/mojito/security/UserDetailsServiceImpl.java @@ -26,12 +26,15 @@ public class UserDetailsServiceImpl implements UserDetailsService { @Override public UserDetails loadUserByUsername(String username) throws UsernameNotFoundException { + logger.debug("Attempting user database authentication for username: {}", username); User user = userRepository.findByUsername(username); if (user == null) { + logger.debug("User not found: {}", username); throw new UsernameNotFoundException("User [" + username + "] is not found in the database."); } + logger.debug("Database authentication user found: {}", user.getUsername()); return new UserDetailsImpl(user); } }