You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sz...@apache.org on 2009/05/21 23:35:43 UTC
svn commit: r777273 -
/directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java
Author: szoerner
Date: Thu May 21 21:35:42 2009
New Revision: 777273
URL: http://svn.apache.org/viewvc?rev=777273&view=rev
Log:
Enhanced logging and comments
Modified:
directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java
Modified: directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java
URL: http://svn.apache.org/viewvc/directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java?rev=777273&r1=777272&r2=777273&view=diff
==============================================================================
--- directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java (original)
+++ directory/sandbox/szoerner/apacheds-tomcatrealm/src/main/java/org/apache/directory/tomcatrealm/EmbeddedApacheDsRealm.java Thu May 21 21:35:42 2009
@@ -144,19 +144,21 @@
Principal principal = null;
if (log.isDebugEnabled()) {
- log.debug("try to authenticat user " + username);
+ log.debug("Try to authenticate user [" + username + "]");
}
LdapDN userDN = this.searchEntryByUsername(username);
if (userDN != null) {
if (log.isDebugEnabled()) {
- log.debug("Found user with DN " + userDN.getUpName());
+ log.debug("Found user [" + username + "] with DN ["
+ + userDN.getUpName() + "]");
}
if (this.authenticate(userDN, credentials)) {
- log.debug("Authentication succesful for user " + username);
+ log.debug("Authentication succesful for user [" + username
+ + "]");
List<String> roles = this.searchRolesForUserDn(userDN);
principal = new GenericPrincipal(this, username, credentials,
@@ -212,7 +214,7 @@
adminSession.unbind();
} catch (Exception e) {
- e.printStackTrace();
+ log.error(e);
throw new RuntimeException(e);
}
return dn;
@@ -232,6 +234,11 @@
protected List<String> searchRolesForUserDn(LdapDN userDn) {
List<String> roles = new ArrayList<String>();
+ if (log.isDebugEnabled()) {
+ log.debug("search roles for user with DN [" + userDn.getUpName()
+ + "]");
+ }
+
try {
CoreSession adminSession = directoryService.getAdminSession();
@@ -254,9 +261,15 @@
adminSession.unbind();
} catch (Exception e) {
- e.printStackTrace();
+ log.error(e);
throw new RuntimeException(e);
}
+
+ if (log.isDebugEnabled()) {
+ log.debug(roles.size() + " roles found for user with DN ["
+ + userDn.getUpName() + "]:" + roles);
+ }
+
return roles;
}