X-Git-Url: http://wamblee.org/gitweb/?a=blobdiff_plain;f=security%2Fsrc%2Fmain%2Fjava%2Forg%2Fwamblee%2Fusermgt%2FJaasUserAccessor.java;h=8102b82fe22a6dd447b920d2abbf6f41869e3575;hb=0d8d8f24656e585ee75558cfd6a4c661f8f14985;hp=56ae836e5feb02686bbca3b09e47e5b82eb27685;hpb=162af365e45e54e5e8d656be276914df2005eaec;p=utils

diff --git a/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java b/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
index 56ae836e..8102b82f 100644
--- a/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
+++ b/security/src/main/java/org/wamblee/usermgt/JaasUserAccessor.java
@@ -25,18 +25,20 @@ import javax.security.auth.Subject;
 /**
  * Implementation of the user accessor that retrieves user information
  * from JAAS. 
+ *
+ * @author Erik Brakkee
  */
 public class JaasUserAccessor implements UserAccessor {
 
     /**
      * User administration to use. 
      */
-    private UserAdministration _admin;
+    private UserAdministration admin;
 
     /**
      * Class of the JAAS user principal. 
      */
-    private Class _userPrincipalClass;
+    private Class userPrincipalClass;
 
     /**
      * Constructs user accessor. 
@@ -44,10 +46,10 @@ public class JaasUserAccessor implements UserAccessor {
      * @param aUserClassName Class name of the user principal. 
      */
     public JaasUserAccessor(UserAdministration aAdmin, String aUserClassName) {
-        _admin = aAdmin;
+        admin = aAdmin;
         try {
-            _userPrincipalClass = Class.forName(aUserClassName);
-            if ( !Principal.class.isAssignableFrom(_userPrincipalClass)) {
+            userPrincipalClass = Class.forName(aUserClassName);
+            if ( !Principal.class.isAssignableFrom(userPrincipalClass)) {
                 throw new IllegalArgumentException("Specified class '" + aUserClassName + "' is not a subclass of '" +
                         Principal.class.getName());
             }
@@ -68,7 +70,7 @@ public class JaasUserAccessor implements UserAccessor {
         }
         Principal userPrincipal = getUserPrincipal(subject);
       
-        return _admin.getUser(userPrincipal.getName());
+        return admin.getUser(userPrincipal.getName());
     }
 
     /**
@@ -81,17 +83,17 @@ public class JaasUserAccessor implements UserAccessor {
         Set<Principal> principals = subject.getPrincipals();
         Principal userPrincipal = null;  
         for ( Principal principal: principals) { 
-            if ( principal.getClass().equals(_userPrincipalClass)) { 
+            if ( principal.getClass().equals(userPrincipalClass)) { 
                 if ( userPrincipal != null ) { 
                     throw new IllegalArgumentException(
-                            "Multiple principals for class '" + _userPrincipalClass + "', subject: " + subject);                   
+                            "Multiple principals for class '" + userPrincipalClass + "', subject: " + subject);                   
                 }
                 userPrincipal = principal; 
             }
         }
         if ( userPrincipal == null ) { 
             throw new IllegalArgumentException(
-                    "No user principal found for class '" + _userPrincipalClass + "', subject: " + subject);
+                    "No user principal found for class '" + userPrincipalClass + "', subject: " + subject);
         }
         return userPrincipal;
     }