Add a few logging statements

git-svn-id: https://subversion.switch.ch/svn/shibboleth/java-idp/branches/REL_2@2789 ab3bd59b-922f-494d-bb5f-6f0a3c29deca

diff --git a/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java b/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java
index 50d3b82..86809bd 100644 (file)
--- a/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java
+++ b/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java
@@ -157,9 +157,11 @@ public class UsernamePasswordLoginServlet extends HttpServlet {
      * @return true of authentication succeeds, false if not
      */
     protected boolean authenticateUser(HttpServletRequest request) {
+        String username = DatatypeHelper.safeTrimOrNullString(request.getParameter(usernameAttribute));
+        String password = DatatypeHelper.safeTrimOrNullString(request.getParameter(passwordAttribute));
+
         try {
-            String username = DatatypeHelper.safeTrimOrNullString(request.getParameter(usernameAttribute));
-            String password = DatatypeHelper.safeTrimOrNullString(request.getParameter(passwordAttribute));
+            log.debug("Attempting to authenticate user {}", username);
 
             SimpleCallbackHandler cbh = new SimpleCallbackHandler(username, password);
 
@@ -188,7 +190,7 @@ public class UsernamePasswordLoginServlet extends HttpServlet {
 
             return true;
         } catch (LoginException e) {
-            log.debug("User authentication failed", e);
+            log.debug("User authentication for {} failed", new Object[] {username}, e);
             return false;
         }
     }