You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@clerezza.apache.org by mi...@apache.org on 2010/08/25 10:58:39 UTC
svn commit: r988910 -
/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java
Author: mir
Date: Wed Aug 25 08:58:39 2010
New Revision: 988910
URL: http://svn.apache.org/viewvc?rev=988910&view=rev
Log:
CLEREZZA-287: unsuccessful authentication attempts are now logged on info level
Modified:
incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java
Modified: incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java
URL: http://svn.apache.org/viewvc/incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java?rev=988910&r1=988909&r2=988910&view=diff
==============================================================================
--- incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java (original)
+++ incubator/clerezza/trunk/org.apache.clerezza.parent/org.apache.clerezza.platform.security/src/main/java/org/apache/clerezza/platform/security/auth/AuthenticationCheckerImpl.java Wed Aug 25 08:58:39 2010
@@ -69,10 +69,10 @@ public class AuthenticationCheckerImpl i
NonLiteral agent = getAgentFromGraph(userName);
String storedPassword = getPasswordOfAgent(agent);
if (storedPassword.equals(PasswordUtil.convertPassword(password))) {
- logger.debug("password matches");
+ logger.debug("user {} successfully authenticated");
return true;
} else {
- logger.debug("password didn't match ");
+ logger.info("unsuccessful authentication attempt as user {}", userName);
return false;
}
}
@@ -83,7 +83,7 @@ public class AuthenticationCheckerImpl i
if (agents.hasNext()) {
agent = agents.next().getSubject();
} else {
- logger.debug("no user {} in graph", userName);
+ logger.info("unsuccessful authentication attempt as non-existent user {}", userName);
throw new NoSuchAgent();
}
return agent;