[java-shib-idp2 COMMIT] in /branches/REL_2: doc/RELEASE-NOTES.txt src/main/java/edu/internet2/middleware/shibboleth/i...

[noreply at shibboleth.net]

Author: putmanb
Date: Wed Jan 27 20:03:08 2016
New Revision: 3221

URL: http://svn.shibboleth.net/view/java-shib-idp2?rev=3221&view=rev
Log:
SIDP-631: Raise JAAS LoginHandler succeed/fail log from DEBUG to INFO

Modified:
    branches/REL_2/doc/RELEASE-NOTES.txt
    branches/REL_2/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java

Modified: branches/REL_2/doc/RELEASE-NOTES.txt
URL: http://svn.shibboleth.net/view/java-shib-idp2/branches/REL_2/doc/RELEASE-NOTES.txt?rev=3221&r1=3220&r2=3221&view=diff
==============================================================================
--- branches/REL_2/doc/RELEASE-NOTES.txt	(original)
+++ branches/REL_2/doc/RELEASE-NOTES.txt	Wed Jan 27 20:03:08 2016
@@ -8,6 +8,7 @@
 =============================================
 [SIDP-617] - Prevent No-login-context errors due to duplicate logins
 [SIDP-623] - NPE after error relating to SAML 2 artifact resolution
+[SIDP-631] - Raise JAAS LoginHandler succeed/fail log from DEBUG to INFO
 
 Changes in Release 2.4.4
 =============================================

Modified: branches/REL_2/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java
URL: http://svn.shibboleth.net/view/java-shib-idp2/branches/REL_2/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java?rev=3221&r1=3220&r2=3221&view=diff
==============================================================================
--- branches/REL_2/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java	(original)
+++ branches/REL_2/src/main/java/edu/internet2/middleware/shibboleth/idp/authn/provider/UsernamePasswordLoginServlet.java	Wed Jan 27 20:03:08 2016
@@ -175,7 +175,7 @@
                     jaasConfigName, cbh);
 
             jaasLoginCtx.login();
-            log.debug("Successfully authenticated user {}", username);
+            log.info("Successfully authenticated user {}", username);
 
             Subject loginSubject = jaasLoginCtx.getSubject();
 
@@ -191,10 +191,10 @@
             request.setAttribute(LoginHandler.SUBJECT_KEY, userSubject);
             request.setAttribute(LoginHandler.AUTHENTICATION_METHOD_KEY, authenticationMethod);
         } catch (LoginException e) {
-            log.debug("User authentication for " + username + " failed", e);
+            log.info("User authentication for " + username + " failed", e);
             throw e;
         } catch (Throwable e) {
-            log.debug("User authentication for " + username + " failed", e);
+            log.info("User authentication for " + username + " failed", e);
             throw new LoginException("unknown authentication error");
         }
     }