From 14e4c2cc6eec1fe46983437d3ef9b9656406865e Mon Sep 17 00:00:00 2001
From: jomu <joern@muehlencord.de>
Date: Tue, 6 Nov 2018 00:37:19 +0100
Subject: [PATCH] improved logging

---
 .../shared/account/presentation/LoginView.java         | 10 ++++++++--
 1 file changed, 8 insertions(+), 2 deletions(-)

diff --git a/account/src/main/java/de/muehlencord/shared/account/presentation/LoginView.java b/account/src/main/java/de/muehlencord/shared/account/presentation/LoginView.java
index 7bb411c..eaf5094 100644
--- a/account/src/main/java/de/muehlencord/shared/account/presentation/LoginView.java
+++ b/account/src/main/java/de/muehlencord/shared/account/presentation/LoginView.java
@@ -51,19 +51,25 @@ public class LoginView implements Serializable {
         // "Remember Me" built-in:
         token.setRememberMe(rememberMe);
         Subject currentUser = SecurityUtils.getSubject();
-        LOGGER.info("Submitting login with username of " + username);
+        LOGGER.info("Trying to login user {}", username);
 
         try {
             currentUser.login(token);
+            LOGGER.info("User {} logged in", username);
             // user logged in, update account entity
             AccountEntity account = accountService.getAccountEntity(username, true);
             accountService.updateLogin(account);
 
             // redirect to home
             ExternalContext ec = FacesContext.getCurrentInstance().getExternalContext();
+            ServletResponse servletResponse = (ServletResponse) ec.getResponse();
             String fallbackUrl = "/web/index.xhtml"; // TODO make configurable
             // ec.redirect(url);
-            WebUtils.redirectToSavedRequest((ServletRequest) ec.getRequest(), (ServletResponse) ec.getResponse(), fallbackUrl);
+            if (LOGGER.isTraceEnabled()) {
+                LOGGER.trace("redirecting to {}, fallbackUrl={}", servletResponse.toString(), fallbackUrl);
+            }
+
+            WebUtils.redirectToSavedRequest((ServletRequest) ec.getRequest(), servletResponse, fallbackUrl);
         } catch (IOException | AuthenticationException ex) {
             // Could catch a subclass of AuthenticationException if you like
             String hint = "Error while authenticating user " + username;