You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2015/06/02 20:36:46 UTC

[21/51] [partial] directory-fortress-core git commit: FC-109 - rename rbac package to impl

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rbac/UserP.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/UserP.java b/src/main/java/org/apache/directory/fortress/core/rbac/UserP.java
deleted file mode 100755
index 00f136f..0000000
--- a/src/main/java/org/apache/directory/fortress/core/rbac/UserP.java
+++ /dev/null
@@ -1,869 +0,0 @@
-/*
- *   Licensed to the Apache Software Foundation (ASF) under one
- *   or more contributor license agreements.  See the NOTICE file
- *   distributed with this work for additional information
- *   regarding copyright ownership.  The ASF licenses this file
- *   to you under the Apache License, Version 2.0 (the
- *   "License"); you may not use this file except in compliance
- *   with the License.  You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- *   Unless required by applicable law or agreed to in writing,
- *   software distributed under the License is distributed on an
- *   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- *   KIND, either express or implied.  See the License for the
- *   specific language governing permissions and limitations
- *   under the License.
- *
- */
-package org.apache.directory.fortress.core.rbac;
-
-
-import java.util.ArrayList;
-import java.util.List;
-import java.util.Set;
-
-import org.apache.commons.lang.StringUtils;
-import org.apache.directory.fortress.core.model.AdminRole;
-import org.apache.directory.fortress.core.model.OrgUnit;
-import org.apache.directory.fortress.core.model.PwPolicy;
-import org.apache.directory.fortress.core.model.Role;
-import org.apache.directory.fortress.core.model.Session;
-import org.apache.directory.fortress.core.model.User;
-import org.apache.directory.fortress.core.model.UserAdminRole;
-import org.apache.directory.fortress.core.model.UserRole;
-import org.apache.directory.fortress.core.util.ObjUtil;
-import org.apache.directory.fortress.core.util.attr.AttrHelper;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-
-import org.apache.directory.fortress.core.GlobalErrIds;
-import org.apache.directory.fortress.core.GlobalIds;
-import org.apache.directory.fortress.core.model.ObjectFactory;
-import org.apache.directory.fortress.core.PasswordException;
-import org.apache.directory.fortress.core.SecurityException;
-import org.apache.directory.fortress.core.ValidationException;
-import org.apache.directory.fortress.core.model.VUtil;
-import org.apache.directory.fortress.core.util.time.CUtil;
-
-
-/**
- * Process module for the User entity.  This class performs data validations and error mapping.  It is typically called
- * by internal Fortress manager classes ({@link AdminMgrImpl}, {@link AccessMgrImpl},
- * {@link ReviewMgrImpl}, ...) and not intended for external non-Fortress clients.  This class will accept,
- * {@link org.apache.directory.fortress.core.model.User}, validate its contents and forward on to it's corresponding DAO class {@link org.apache.directory.fortress.core.rbac.UserDAO}.
- * <p>
- * Class will throw {@link SecurityException} to caller in the event of security policy, data constraint violation or system
- * error internal to DAO object. This class will forward DAO exceptions ({@link org.apache.directory.fortress.core.FinderException},
- * {@link org.apache.directory.fortress.core.CreateException},{@link org.apache.directory.fortress.core.UpdateException},{@link org.apache.directory.fortress.core.RemoveException}),
- *  or {@link org.apache.directory.fortress.core.ValidationException} as {@link SecurityException}s with appropriate
- * error id from {@link org.apache.directory.fortress.core.GlobalErrIds}.
- * <p>
- * This class is thread safe.
- * </p>
- *
- * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
- */
-public final class UserP
-{
-    //private static final boolean IS_SESSION_PROPS_ENABLED = Config.getBoolean( "user.session.props.enabled", false );
-    private static final String CLS_NM = UserP.class.getName();
-    private static UserDAO uDao = new UserDAO();
-    private static final Logger LOG = LoggerFactory.getLogger( CLS_NM );
-    private static final PolicyP policyP = new PolicyP();
-    private static final AdminRoleP admRoleP = new AdminRoleP();
-    private static final OrgUnitP orgUnitP = new OrgUnitP();
-
-
-    /**
-     * Package private constructor.
-     */
-    UserP()
-    {
-    }
-
-
-    /**
-     * Takes a User entity that contains full or partial userId OR a full internal userId for search.
-     *
-     * @param user contains all or partial userId or full internal userId.
-     * @return List of type User containing fully populated matching User entities.  If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<User> search( User user ) throws SecurityException
-    {
-        return uDao.findUsers( user );
-    }
-
-
-    List<User> search( OrgUnit ou, boolean limitSize ) throws SecurityException
-    {
-        return uDao.findUsers( ou, limitSize );
-    }
-
-
-    /**
-     * Search according to full or partial search string that maps to Fortress userid.
-     * This search is used by RealmMgr for Websphere.
-     *
-     * @param user contains full or partial userId.
-     * @param limit     specify the max number of records to return in result set.
-     * @return List of type String containing userId of all matching User entities. If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<String> search( User user, int limit ) throws SecurityException
-    {
-        return uDao.findUsers( user, limit );
-    }
-
-
-    /**
-     * Return a list of Users that are authorized the given Role.
-     *
-     * @param role contains the role name targeted for search.
-     * @return List of type User containing fully populated matching User entities. If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<User> getAuthorizedUsers( Role role ) throws SecurityException
-    {
-        return uDao.getAuthorizedUsers( role );
-    }
-
-
-    /**
-     * Return a list of Users that are authorized the given Role.
-     *
-     * @param roles contains the set of role names targeted for search.
-     * @param contextId maps to sub-tree in DIT, for example ou=contextId, dc=jts, dc = com.
-     * @return Set of type String containing the userId's for matching User entities. If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    Set<String> getAssignedUsers( Set<String> roles, String contextId ) throws SecurityException
-    {
-        return uDao.getAssignedUsers( roles, contextId );
-    }
-
-
-    /**
-     * Return a list of Users that are authorized the given Role.
-     * In RBAC the word "authorized" implies the hierarchical role relations graph is considered in result set.
-     * This search is used by RealmMgr for Websphere.
-     *
-     * @param role
-     * @param limit specify the max number of records to return in result set.
-     * @return list of type String of userIds. If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<String> getAuthorizedUsers( Role role, int limit ) throws SecurityException
-    {
-        return uDao.getAuthorizedUsers( role, limit );
-    }
-
-
-    /**
-     * Return a list of Users assigned the given RBAC role.
-     * "Assigned" implies the hierarchical role relation graph will NOT be considered in result set.
-     *
-     * @param role contains name of RBAC role used for search.
-     * @return List of fully populated User entities matching target search. If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<User> getAssignedUsers( Role role ) throws SecurityException
-    {
-        return uDao.getAssignedUsers( role );
-    }
-
-
-    /**
-     * Return a list of Users assigned the given Administrative role.
-     * "Assigned" implies the hierarchical role relation graph will NOT be considered in result set.
-     *
-     * @param role contains name of Admin role used for search.
-     * @return List of fully populated User entities matching target search.  If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<User> getAssignedUsers( AdminRole role ) throws SecurityException
-    {
-        return uDao.getAssignedUsers( role );
-    }
-
-
-    /**
-     * Return the list of User's RBAC roles.
-     *
-     * @param user contains full userId for target operation.
-     * @return List of type String containing RBAC role names.  If no records found this will be empty.
-     * @throws SecurityException in the event of DAO search error.
-     */
-    List<String> getAssignedRoles( User user ) throws SecurityException
-    {
-        return uDao.getRoles( user );
-    }
-
-
-    /**
-     * Return a fully populated User entity for a given userId.  If the User entry is not found a SecurityException
-     * will be thrown.
-     *
-     * @param user  contains full userId value.
-     * @param isRoles return user's assigned roles if "true".
-     * @return User entity containing all attributes associated with User in directory.
-     * @throws SecurityException in the event of User not found or DAO search error.
-     */
-    User read( User user, boolean isRoles ) throws SecurityException
-    {
-        return uDao.getUser( user, isRoles );
-    }
-
-
-    /**
-     * Adds a new User entity to directory.  The User entity input object will be validated to ensure that:
-     * userId is present, orgUnitId is valid, roles (optiona) are valid, reasonability checks on all of the
-     * other populated values.
-     *
-     * @param entity User entity contains data targeted for insertion.
-     * @return User entity copy of input + additional attributes (internalId) that were added by op.
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    User add( User entity ) throws SecurityException
-    {
-        return add( entity, true );
-    }
-
-
-    /**
-     * Adds a new User entity to directory.
-     * The User entity input object will be validated to ensure that: userId is present, orgUnitId is valid,
-     * roles (optiona) are valid, reasonability checks on all of the other populated values.
-     *
-     * @param entity   User entity contains data targeted for insertion.
-     * @param validate if false will skip the validations described above.
-     * @return User entity copy of input + additional attributes (internalId)
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    User add( User entity, boolean validate ) throws SecurityException
-    {
-        if ( validate )
-        {
-            // Ensure the input data is valid.
-            validate( entity, false );
-        }
-
-        entity = uDao.create( entity );
-
-        return entity;
-    }
-
-
-    /**
-     * Update existing user's attributes with the input entity.  Null or empty attributes will be ignored.
-     * This method will ignore userId as input as change userId is not allowed.  If password is changed
-     * OpenLDAP password policy will not be evaluated on behalf of the user.
-     * Other User entity input data can be changed and will also be validated beforehand to ensure that:
-     * orgUnitId is valid, roles (optional) are valid, reasonability checks will be performed on all of the populated fields.
-     *
-     * @param entity User entity contains data targeted for insertion.
-     * @return User entity copy of input
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    User update( User entity ) throws SecurityException
-    {
-        return update( entity, true );
-    }
-
-
-    /**
-     * Update existing user's attributes with the input entity.  Null or empty attributes will be ignored.
-     * This method will ignore userId or password as input.  The former is not allowed and latter is performed by other
-     * methods in this class.
-     * Other User entity input data can be changed and will also be validated beforehand to ensure that:
-     * orgUnitId is valid, roles (optional) are valid, reasonability checks will be performed on all of the populated fields.
-     *
-     * @param entity   User entity contains data targeted for insertion.
-     * @param validate if false will skip the validations described above.
-     * @return User entity copy of input
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    /**
-     * Update existing user's attributes with the input entity.  Null or empty attributes will be ignored.
-     * This method will ignore userId or password as input.  The former is not allowed and latter is performed by other
-     * methods in this class.
-     * Other User entity input data can be changed and will also be validated beforehand to ensure that:
-     * orgUnitId is valid, roles (optional) are valid, reasonability checks will be performed on all of the populated fields.
-     *
-     * @param entity   User entity contains data targeted for insertion.
-     * @param validate if false will skip the validations described above.
-     * @return User entity copy of input
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    User update( User entity, boolean validate ) throws SecurityException
-    {
-        if ( validate )
-        {
-            // Ensure the input data is valid.
-            validate( entity, true );
-        }
-        entity = uDao.update( entity );
-        return entity;
-    }
-
-
-    /**
-     * Method performs a "soft" delete.  It disables User entity and flags as "deleted".  User must exist in directory
-     * prior to making this call.
-     *
-     * @param user Contains the userId of the user targeted for deletion.
-     * @return String contains user DN
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    String softDelete( User user ) throws SecurityException
-    {
-        // Ensure this user isn't listed in Fortress config as a system user that can't be removed via API.
-        // Is there a match between this userId and a Fortress system user?
-        User checkUser = read( user, true );
-        if ( ObjUtil.isNotNullOrEmpty( checkUser.isSystem() ) && checkUser.isSystem() )
-        {
-            String warning = "softDelete userId [" + user.getUserId()
-                + "] can't be removed due to policy violation, rc=" + GlobalErrIds.USER_PLCY_VIOLATION;
-            throw new SecurityException( GlobalErrIds.USER_PLCY_VIOLATION, warning );
-        }
-        user.setDescription( "DELETED" );
-        User outUser = uDao.update( user );
-        return outUser.getDn();
-    }
-
-
-    /**
-     * This method performs a "hard" delete.  It completely removes all data associated with this user from the directory.
-     * User entity must exist in directory prior to making this call else exception will be thrown.
-     *
-     * @param user Contains the userid of the user targeted for deletion.
-     * @return String contains user DN
-     * @throws SecurityException in the event of data validation or DAO system error.
-     */
-    String delete( User user ) throws SecurityException
-    {
-        // Ensure this user isn't listed in Fortress config as a system user that can't be removed via API.
-        // Is there a match between this userId and a Fortress system user?
-        User checkUser = read( user, true );
-        if ( ObjUtil.isNotNullOrEmpty( checkUser.isSystem() ) && checkUser.isSystem() )
-        {
-            String warning = "delete userId [" + user.getUserId()
-                + "] can't be removed due to policy violation, rc=" + GlobalErrIds.USER_PLCY_VIOLATION;
-            throw new SecurityException( GlobalErrIds.USER_PLCY_VIOLATION, warning );
-        }
-        return uDao.remove( user );
-    }
-
-
-    /**
-     * Removes the user's association from OpenLDAP password policy.  Once this association is removed, the User
-     * password policy will default to that which is default for ldap server.
-     *
-     * @param user contains the userId for target user.
-     * @throws SecurityException in the event of DAO error.
-     */
-    void deletePwPolicy( User user ) throws SecurityException
-    {
-        uDao.deletePwPolicy( user );
-    }
-
-
-    /**
-     * This method performs authentication only.  It does not activate RBAC roles in session.  It will evaluate
-     * password policies.
-     *
-     * @param user  Contains the userid of the user signing on along with password.
-     * @return Session object will be returned if authentication successful.  This will not contain user's roles.
-     * @throws SecurityException in the event of data validation failure, security policy violation or DAO error.
-     */
-    Session authenticate( User user ) throws SecurityException
-    {
-        Session session;
-        session = uDao.checkPassword( user );
-
-        if ( !session.isAuthenticated() )
-        {
-            String info = "UserP.authenticate failed  for userId [" + user.getUserId() + "] reason code ["
-                + session.getErrorId() + "] msg [" + session.getMsg() + "]";
-            throw new PasswordException( session.getErrorId(), info );
-        }
-
-        CUtil.validateConstraints( session, CUtil.ConstraintType.USER, false );
-
-        return session;
-    }
-
-
-    /**
-     * CreateSession
-     * <p>
-     * This method is called by AccessMgr and is not intended for use outside Fortress core.  The successful
-     * result is Session object that contains target user's RBAC and Admin role activations.  In addition to checking
-     * user password validity it will apply configured password policy checks.  Method may also store parms passed in for
-     * audit trail..
-     * <ul>
-     * <li> authenticate user password
-     * <li> password policy evaluation with OpenLDAP PwPolicy
-     * <li> evaluate temporal constraints on User and UserRole entities.
-     * <li> allow selective role activations into User RBAC Session.
-     * <li> require valid password if trusted == false.
-     * <li> will disallow any user who is locked out due to OpenLDAP pw policy, regardless of trusted flag being set as parm on API.
-     * <li> return User's RBAC Session containing User and UserRole attributes.
-     * <li> throw a SecurityException for authentication failures, other policy violations, data validation errors or system failure.
-     * </ul>
-     * </p>
-     * <p>
-     * The function is valid if and only if:
-     * <ul>
-     * <li> the user is a member of the USERS data set
-     * <li> the password is supplied (unless trusted).
-     * <li> the (optional) active role set is a subset of the roles authorized for that user.
-     * </ul>
-     * </p>
-     * <p>
-     * The User parm contains the following (* indicates required)
-     * <ul>
-     * <li> String userId*
-     * <li> char[] password
-     * <li> List<UserRole> userRoles contains a list of RBAC role names authorized for user and targeted for activation within this session.
-     * <li> List<UserAdminRole> userAdminRoles contains a list of Admin role names authorized for user and targeted for activation.
-     * <li> Properties logonProps collection of auditable name/value pairs to store.  For example hostname:myservername or ip:192.168.1.99
-     * </ul>
-     * </p>
-     * <p>
-     * Notes:
-     * <ul>
-     * <li> roles that violate Dynamic Separation of Duty Relationships will not be activated into session.
-     * <li> role activations will proceed in same order as supplied to User entity setter.
-     * </ul>
-     * </p>
-     *
-     * @param user    Contains userId, password (optional if "trusted"), optional User RBAC Roles: List<UserRole> rolesToBeActivated., optional User Admin Roles: List<UserAdminRole> adminRolesToBeActivated.
-     * @param trusted if true password is not required.
-     * @return Session object will contain authentication result code, RBAC and Admin role activations, OpenLDAP pw policy output and more.
-     * @throws SecurityException in the event of data validation failure, security policy violation or DAO error.
-     */
-    Session createSession( User user, boolean trusted ) throws SecurityException
-    {
-        Session session;
-        if ( trusted )
-        {
-            // Create the rbac session without authentication of password.
-            session = createSessionTrusted( user );
-            // Check user temporal constraints.  This op usually performed during authentication.
-            CUtil.validateConstraints( session, CUtil.ConstraintType.USER, false );
-        }
-        else
-        {
-            // Create the rbac session if the user authentication succeeds:
-            VUtil.assertNotNullOrEmpty( user.getPassword(), GlobalErrIds.USER_PW_NULL, CLS_NM + ".createSession" );
-            session = createSession( user );
-        }
-        // Did the caller pass in a set of roles for selective activation?
-        if ( ObjUtil.isNotNullOrEmpty( user.getRoles() ) )
-        {
-            // Process selective activation of user's RBAC roles into session:
-            List<UserRole> rlsActual = session.getRoles();
-            List<UserRole> rlsFinal = new ArrayList<>();
-            session.setRoles( rlsFinal );
-            // Activate only the intersection between assigned and roles passed into this method:
-            for ( UserRole role : user.getRoles() )
-            {
-                int indx = rlsActual.indexOf( role );
-                if ( indx != -1 )
-                {
-                    UserRole candidateRole = rlsActual.get( indx );
-                    rlsFinal.add( candidateRole );
-                }
-            }
-        }
-        // Check role temporal constraints + activate roles:
-        CUtil.validateConstraints( session, CUtil.ConstraintType.ROLE, true );
-        return session;
-    }
-
-
-    /**
-     * Called internal to this class only.  Will do all of the session activations of the public method
-     * in addition to the password validation.
-     *
-     * @param inUser   Contains userId that represents rDn of node in ldap directory.
-     * @return Session object will contain authentication result code, RBAC and Admin role activations, OpenLDAP pw policy output and more.
-     * @throws SecurityException in the event of data validation failure, security policy violation or DAO error.
-     */
-    private Session createSession( User inUser )
-        throws SecurityException
-    {
-        // read user entity:
-        User user = read( inUser, true );
-        user.setContextId( inUser.getContextId() );
-
-        // authenticate password, check pw policies and validate user temporal constraints:
-        Session session = authenticate( inUser );
-
-        // Set the user entity into the session object:
-        session.setUser( user );
-        return session;
-    }
-
-
-    /**
-     * Trusted session creation method called internal to this class only.  Will do all of the session activations of the public method
-     *
-     * @param inUser Contains userId that represents rDn of node in ldap directory.
-     * @return Session object will contain authentication result code, RBAC and Admin role activations, OpenLDAP pw policy output and more.
-     * @throws SecurityException in the event of data validation failure, security policy violation or DAO error.
-     */
-    private Session createSessionTrusted( User inUser )
-        throws SecurityException
-    {
-        User user = read( inUser, true );
-        user.setContextId( inUser.getContextId() );
-
-        if ( user.isLocked() )
-        {
-            String warning = "createSession failed for userId [" + inUser.getUserId()
-                + "] reason user is locked";
-            LOG.warn( warning );
-            throw new SecurityException( GlobalErrIds.USER_LOCKED_BY_CONST, warning );
-        }
-
-        Session session = new ObjectFactory().createSession();
-        session.setUserId( inUser.getUserId() );
-        // Set this flag to false because user's password was not authenticated.
-        session.setAuthenticated( false );
-        session.setUser( user );
-        return session;
-    }
-
-
-    /**
-     * Method will set the OpenLDAP pwlocked attribute which will lock user from being able to signon to the system.
-     *
-     * @param user Contains userId that represents rDn of node in ldap directory.
-     * @throws SecurityException in the event of DAO error.
-     */
-    void lock( User user ) throws SecurityException
-    {
-        uDao.lock( user );
-    }
-
-
-    /**
-     * Method will reset the OpenLDAP pwlocked attribute which will unlock user and allow to signon to the system.
-     *
-     * @param user Contains userId that represents rDn of node in ldap directory.
-     * @throws SecurityException in the event of DAO  error.
-     */
-    void unlock( User user ) throws SecurityException
-    {
-        uDao.unlock( user );
-    }
-
-
-    /**
-     * Method will change the user's password and validate user's pw policy in OpenLDAP.
-     *
-     * @param entity      contains userId and old password.
-     * @param newPassword contains the new password which must pass the password policy constraints.
-     * @throws SecurityException in the event of data validation failure, password policy violation or DAO error.
-     */
-    void changePassword( User entity, char[] newPassword ) throws SecurityException
-    {
-        String userId = entity.getUserId();
-        boolean result = uDao.changePassword( entity, newPassword );
-        if ( !result )
-        {
-            LOG.warn( "changePassword failed for user [{}]", userId );
-        }
-    }
-
-
-    /**
-     * Perform password reset on user entity.  This will change the User password and set the reset flag
-     * in OpenLDAP will will force the user to change their password at next logon time.
-     *
-     * @param user contains the userId and the new password.
-     * @throws SecurityException in the event of DAO error.
-     */
-    void resetPassword( User user ) throws SecurityException
-    {
-        uDao.resetUserPassword( user );
-    }
-
-
-    /**
-     * This command assigns a user to a role.
-     * <p>
-     * <ul>
-     * <li> The command is valid if and only if:
-     * <li> The user is a member of the USERS data set
-     * <li> The role is a member of the ROLES data set
-     * <li> The user is not already assigned to the role
-     * <li> The SSD constraints are satisfied after assignment.
-     * </ul>
-     * </p>
-     * <p>
-     * Successful completion of this op, the following occurs:
-     * </p>
-     * <ul>
-     * <li> User entity (resides in people container) has role assignment added to aux object class attached to actual user record.
-     * <li> Role entity (resides in role container) has userId added as role occupant.
-     * <li> (optional) Temporal constraints may be associated with <code>ftUserAttrs</code> aux object class based on:
-     * <ul>
-     * <li> timeout - number in seconds of session inactivity time allowed.
-     * <li> beginDate - YYYYMMDD - determines date when role may be activated.
-     * <li> endDate - YYMMDD - indicates latest date role may be activated.
-     * <li> beginLockDate - YYYYMMDD - determines beginning of enforced inactive status
-     * <li> endLockDate - YYMMDD - determines end of enforced inactive status.
-     * <li> beginTime - HHMM - determines begin hour role may be activated in user's session.
-     * <li> endTime - HHMM - determines end hour role may be activated in user's session.*
-     * <li> dayMask - 1234567, 1 = Sunday, 2 = Monday, etc - specifies which day of week role may be activated.
-     * </ul>
-     * </ul>
-     *
-     * @param uRole entity contains userId and role name for targeted assignment.
-     * @return String containing the user's DN.  This value is used to update the "roleOccupant" attribute on associated role entity.
-     * @throws SecurityException in the event data error in user or role objects or system error.
-     */
-    String assign( UserRole uRole ) throws SecurityException
-    {
-        validate( uRole );
-        // "assign" custom Fortress role data, i.e. temporal constraints, onto the user node:
-        return uDao.assign( uRole );
-    }
-
-
-    /**
-     * This command deletes the assignment of the User from the Role entities. The command is
-     * valid if and only if the user is a member of the USERS data set, the role is a member of
-     * the ROLES data set, and the user is assigned to the role.
-     * Any sessions that currently have this role activated will not be effected.
-     * Successful completion includes:
-     * User entity in USER data set has role assignment removed.
-     * Role entity in ROLE data set has userId removed as role occupant.
-     * (optional) Temporal constraints will be removed from user aux object if set prior to call.
-     *
-     * @param uRole entity contains userId and RBAC Role name for targeted assignment.
-     * @return String containing the user's DN.  This value is used to remove the "roleOccupant" attribute on associated RBAC Role entity.
-     * @throws SecurityException - in the event data error in user or role objects or system error.
-     */
-    String deassign( UserRole uRole ) throws SecurityException
-    {
-        validate( uRole );
-        // "deassign" custom Fortress role data from the user's node:
-        return uDao.deassign( uRole );
-    }
-
-
-    /**
-     * This command assigns a user to an admin role.
-     * Successful completion of this op, the following occurs:
-     * </p>
-     * <ul>
-     * <li> User entity (resides in people container) has role assignment added to aux object class attached to actual user record.
-     * <li> AdminRole entity (resides in admin role container) has userId added as role occupant.
-     * <li> (optional) Temporal constraints may be associated with <code>ftUserAttrs</code> aux object class based on:
-     * <ul>
-     * <li> timeout - number in seconds of session inactivity time allowed.
-     * <li> beginDate - YYYYMMDD - determines date when role may be activated.
-     * <li> endDate - YYMMDD - indicates latest date role may be activated.
-     * <li> beginLockDate - YYYYMMDD - determines beginning of enforced inactive status
-     * <li> endLockDate - YYMMDD - determines end of enforced inactive status.
-     * <li> beginTime - HHMM - determines begin hour role may be activated in user's session.
-     * <li> endTime - HHMM - determines end hour role may be activated in user's session.*
-     * <li> dayMask - 1234567, 1 = Sunday, 2 = Monday, etc - specifies which day of week role may be activated.
-     * </ul>
-     * </ul>
-     *
-     * @param uRole entity contains userId and Admin Role name for targeted assignment.
-     * @return String containing the user's DN.  This value is used to update the "roleOccupant" attribute on associated Admin Role entity.
-     * @throws SecurityException in the event data error in user or role objects or system error.
-     */
-    String assign( UserAdminRole uRole ) throws SecurityException
-    {
-        validate( uRole );
-        // Assign custom Fortress role data, i.e. temporal constraints, onto the user node:
-        return uDao.assign( uRole );
-    }
-
-
-    /**
-     * This method removes assigned admin role from user entity.  Both user and admin role entities must exist and have role relationship
-     * before calling this method.
-     * Successful completion:
-     * del Role to User assignment in User data set
-     * AND
-     * User to Role assignment in Admin Role data set.
-     *
-     * @param uRole entity contains userId and Admin Role name for targeted assignment.
-     * @return String containing the user's DN.  This value is used to remove the "roleOccupant" attribute on associated Admin Role entity.
-     * @throws SecurityException - in the event data error in user or role objects or system error.
-     */
-    String deassign( UserAdminRole uRole ) throws SecurityException
-    {
-        validate( uRole );
-        // Deassign custom Fortress role data from the user's node:
-        return uDao.deassign( uRole );
-    }
-
-
-    /**
-     * Ensure that the passed in variable has the correct fields set.
-     *
-     * @param uRole - name and userId must be checked.
-     * @throws ValidationException - if either are null or empty.
-     */
-    private void validate( UserRole uRole ) throws ValidationException
-    {
-        if ( !StringUtils.isNotEmpty( uRole.getUserId() ) )
-        {
-            throw new ValidationException( GlobalErrIds.USER_ID_NULL, CLS_NM + ".validate userId is NULL" );
-        }
-        if ( !StringUtils.isNotEmpty( uRole.getName() ) )
-        {
-            throw new ValidationException( GlobalErrIds.ROLE_NM_NULL, CLS_NM + ".validate name is NULL" );
-        }
-    }
-
-
-    /**
-     * Method will perform various validations to ensure the integrity of the User entity targeted for insertion
-     * or updating in directory.  For example the ou attribute will be "read" from the OrgUnit dataset to ensure
-     * that it is valid.  Data reasonability checks will be performed on all non-null attributes.
-     * This method will also copy the source constraints to target entity iff the target input entity does not have set
-     * prior to calling.
-     *
-     * @param entity   User entity contains data targeted for insertion or update.  The input role constraints will be accepted.
-     * @param isUpdate if true update operation is being performed which specifies a different set of targeted attributes.
-     * @throws SecurityException in the event of data validation error or DAO error on Org validation.
-     */
-    private void validate( User entity, boolean isUpdate )
-        throws SecurityException
-    {
-        if ( !isUpdate )
-        {
-            // the UserId attribute is required on User:
-            VUtil.userId( entity.getUserId() );
-
-            // the cn attribute is optional as input.  entity will default to userId if cn not set by caller on add:
-            if ( StringUtils.isNotEmpty( entity.getCn() ) )
-            {
-                VUtil.safeText( entity.getCn(), GlobalIds.CN_LEN );
-            }
-            // the sn attribute is optional as input.  entity will default to userId if sn not set by caller on add:
-            if ( StringUtils.isNotEmpty( entity.getSn() ) )
-            {
-                VUtil.safeText( entity.getSn(), GlobalIds.SN_LEN );
-            }
-            // password is not required on user object but user cannot execute AccessMgr or DelAccessMgr methods w/out pw.
-            if ( ObjUtil.isNotNullOrEmpty( entity.getPassword() ) )
-            {
-                VUtil.password( entity.getPassword() );
-            }
-            // the OU attribute is required:
-            if ( !StringUtils.isNotEmpty( entity.getOu() ) )
-            {
-                String error = "OU validation failed, null or empty value";
-                throw new ValidationException( GlobalErrIds.ORG_NULL_USER, error );
-            }
-            VUtil.orgUnit( entity.getOu() );
-            // ensure ou exists in the OS-U pool:
-            OrgUnit ou = new OrgUnit( entity.getOu(), OrgUnit.Type.USER );
-            ou.setContextId( entity.getContextId() );
-            if ( !orgUnitP.isValid( ou ) )
-            {
-                String error = "validate detected invalid orgUnit name [" + entity.getOu()
-                    + "] adding user with userId [" + entity.getUserId() + "]";
-                throw new ValidationException( GlobalErrIds.USER_OU_INVALID, error );
-            }
-            // description attribute is optional:
-            if ( StringUtils.isNotEmpty( entity.getDescription() ) )
-            {
-                VUtil.description( entity.getDescription() );
-            }
-        }
-        else
-        {
-            // on User update, all attributes are optional:
-            if ( StringUtils.isNotEmpty( entity.getCn() ) )
-            {
-                VUtil.safeText( entity.getCn(), GlobalIds.CN_LEN );
-            }
-            if ( StringUtils.isNotEmpty( entity.getSn() ) )
-            {
-                VUtil.safeText( entity.getSn(), GlobalIds.SN_LEN );
-            }
-            if ( ObjUtil.isNotNullOrEmpty( entity.getPassword() ) )
-            {
-                VUtil.password( entity.getPassword() );
-            }
-            if ( StringUtils.isNotEmpty( entity.getOu() ) )
-            {
-                VUtil.orgUnit( entity.getOu() );
-                // ensure ou exists in the OS-U pool:
-                OrgUnit ou = new OrgUnit( entity.getOu(), OrgUnit.Type.USER );
-                ou.setContextId( entity.getContextId() );
-                if ( !orgUnitP.isValid( ou ) )
-                {
-                    String error = "validate detected invalid orgUnit name [" + entity.getOu()
-                        + "] updating user wth userId [" + entity.getUserId() + "]";
-                    throw new ValidationException( GlobalErrIds.USER_OU_INVALID, error );
-                }
-            }
-            if ( StringUtils.isNotEmpty( entity.getDescription() ) )
-            {
-                VUtil.description( entity.getDescription() );
-            }
-        }
-
-        // 1 OpenLDAP password policy name must be valid if set:
-        if ( StringUtils.isNotEmpty( entity.getPwPolicy() ) )
-        {
-            PwPolicy policy = new PwPolicy( entity.getPwPolicy() );
-            policy.setContextId( entity.getContextId() );
-            if ( !policyP.isValid( policy ) )
-            {
-                String error = "validate detected invalid OpenLDAP policy name [" + entity.getPwPolicy()
-                    + "] for userId [" + entity.getUserId()
-                    + "]. Assignment is optional for User but must be valid if specified.";
-                throw new ValidationException( GlobalErrIds.USER_PW_PLCY_INVALID, error );
-            }
-        }
-
-        // 2 Validate constraints on User object:
-        CUtil.validate( entity );
-
-        // 3 Validate or copy constraints on RBAC roles:
-        if ( ObjUtil.isNotNullOrEmpty( entity.getRoles() ) )
-        {
-            RoleP rp = new RoleP();
-            List<UserRole> roles = entity.getRoles();
-            for ( UserRole ure : roles )
-            {
-                Role inRole = new Role( ure.getName() );
-                inRole.setContextId( entity.getContextId() );
-                Role role = rp.read( inRole );
-                CUtil.validateOrCopy( role, ure );
-            }
-        }
-
-        // 4 Validate and copy constraints on Administrative roles:
-        if ( ObjUtil.isNotNullOrEmpty( entity.getAdminRoles() ) )
-        {
-            List<UserAdminRole> uRoles = entity.getAdminRoles();
-            for ( UserAdminRole uare : uRoles )
-            {
-                AdminRole inRole = new AdminRole( uare.getName() );
-                inRole.setContextId( entity.getContextId() );
-                AdminRole outRole = admRoleP.read( inRole );
-                CUtil.validateOrCopy( outRole, uare );
-
-                // copy the ARBAC AdminRole attributes to UserAdminRole:
-                AttrHelper.copyAdminAttrs( outRole, uare );
-            }
-        }
-    }
-}
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java b/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
deleted file mode 100755
index 2d39ff1..0000000
--- a/src/main/java/org/apache/directory/fortress/core/rbac/UsoUtil.java
+++ /dev/null
@@ -1,311 +0,0 @@
-/*
- *   Licensed to the Apache Software Foundation (ASF) under one
- *   or more contributor license agreements.  See the NOTICE file
- *   distributed with this work for additional information
- *   regarding copyright ownership.  The ASF licenses this file
- *   to you under the Apache License, Version 2.0 (the
- *   "License"); you may not use this file except in compliance
- *   with the License.  You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- *   Unless required by applicable law or agreed to in writing,
- *   software distributed under the License is distributed on an
- *   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- *   KIND, either express or implied.  See the License for the
- *   specific language governing permissions and limitations
- *   under the License.
- *
- */
-package org.apache.directory.fortress.core.rbac;
-
-
-import java.util.List;
-import java.util.Set;
-import java.util.TreeSet;
-import java.util.concurrent.locks.ReadWriteLock;
-
-import org.apache.commons.lang.StringUtils;
-import org.apache.directory.fortress.core.model.Graphable;
-import org.apache.directory.fortress.core.model.Hier;
-import org.apache.directory.fortress.core.model.OrgUnit;
-import org.apache.directory.fortress.core.model.Relationship;
-import org.apache.directory.fortress.core.util.ObjUtil;
-import org.jgrapht.graph.SimpleDirectedGraph;
-import org.slf4j.Logger;
-import org.slf4j.LoggerFactory;
-import org.apache.directory.fortress.core.GlobalIds;
-import org.apache.directory.fortress.core.SecurityException;
-import org.apache.directory.fortress.core.ValidationException;
-import org.apache.directory.fortress.core.model.VUtil;
-import org.apache.directory.fortress.core.util.cache.Cache;
-import org.apache.directory.fortress.core.util.cache.CacheMgr;
-
-
-/**
- * This utility wraps {@link HierUtil} methods to provide hierarchical functionality using the {@link org.apache.directory.fortress.core.model.OrgUnit} data set for User type {@link org.apache.directory.fortress.core.model.OrgUnit.Type#USER}.
- * The {@code cn=Hierarchies, ou=OS-U} data contains User OU pools is stored within a data cache, {@link #usoCache}, contained within this class.  The parent-child edges are contained in LDAP,
- * in {@code ftParents} attribute.  The ldap data is retrieved {@link OrgUnitP#getAllDescendants(org.apache.directory.fortress.core.model.OrgUnit)} and loaded into {@code org.jgrapht.graph.SimpleDirectedGraph}.
- * The graph...
- * <ol>
- * <li>is stored as singleton in this class with vertices of {@code String}, and edges, as {@link org.apache.directory.fortress.core.model.Relationship}s</li>
- * <li>utilizes open source library, see <a href="http://www.jgrapht.org/">JGraphT</a>.</li>
- * <li>contains a general hierarchical data structure i.e. allows multiple inheritance with parents.</li>
- * <li>is a simple directed graph thus does not allow cycles.</li>
- * </ol>
- * After update is performed to ldap, the singleton is refreshed with latest info.
- * <p/>
- * Static methods on this class are intended for use by other Fortress classes, i.e. {@link DelAdminMgrImpl}.
- * and cannot be directly invoked by outside programs.
- * <p/>
- * This class contains singleton that can be updated but is thread safe.
- * <p/>
- *
- * @author <a href="mailto:dev@directory.apache.org">Apache Directory Project</a>
- */
-public final class UsoUtil
-{
-    private static final Cache usoCache;
-    private static final OrgUnitP orgUnitP = new OrgUnitP();
-    private static final String CLS_NM = UsoUtil.class.getName();
-    private static final Logger LOG = LoggerFactory.getLogger( CLS_NM );
-
-    /**
-     * Initialize the User OU hierarchies.  This will read the {@link org.apache.directory.fortress.core.model.Hier} data set from ldap and load into
-     * the JGraphT simple digraph that referenced statically within this class.
-     */
-    static
-    {
-        CacheMgr cacheMgr = CacheMgr.getInstance();
-        usoCache = cacheMgr.getCache( "fortress.uso" );
-    }
-
-    /**
-     * Private constructor
-     *
-     */
-    private UsoUtil()
-    {
-    }
-
-    /**
-     * Recursively traverse the {@link org.apache.directory.fortress.core.model.OrgUnit} graph and return all of the descendants of a given parent {@link org.apache.directory.fortress.core.model.OrgUnit#name}.
-     *
-     * @param name {@link org.apache.directory.fortress.core.model.OrgUnit#name} on 'ftOrgUnit' object class.
-     * @return Set of names of descendants {@link org.apache.directory.fortress.core.model.OrgUnit}s of given parent.
-     */
-    static Set<String> getDescendants( String name, String contextId )
-    {
-        return HierUtil.getDescendants( name, getGraph( contextId ) );
-    }
-
-
-    /**
-     * Recursively traverse the {@link org.apache.directory.fortress.core.model.OrgUnit.Type#USER} graph and return all of the ascendants of a given child ou.
-     *
-     * @param name maps to logical {@link org.apache.directory.fortress.core.model.OrgUnit#name} on 'ftOrgUnit' object class.
-     * @return Set of ou names that are ascendants of given child.
-     */
-    static Set<String> getAscendants( String name, String contextId )
-    {
-        return HierUtil.getAscendants( name, getGraph( contextId ) );
-    }
-
-
-    /**
-     * Traverse one level of the {@link org.apache.directory.fortress.core.model.OrgUnit} graph and return all of the children (direct descendants) of a given parent {@link org.apache.directory.fortress.core.model.OrgUnit#name}.
-     *
-     * @param name {@link org.apache.directory.fortress.core.model.OrgUnit#name} maps on 'ftOrgUnit' object class.
-     * @return Set of names of children {@link org.apache.directory.fortress.core.model.OrgUnit}s of given parent.
-     */
-    public static Set<String> getChildren( String name, String contextId )
-    {
-        return HierUtil.getChildren( name, getGraph( contextId ) );
-    }
-
-
-    /**
-     * Traverse one level of the {@link org.apache.directory.fortress.core.model.OrgUnit.Type#USER} graph and return all of the parents (direct ascendants) of a given child ou.
-     *
-     * @param name maps to logical {@link org.apache.directory.fortress.core.model.OrgUnit#name} on 'ftOrgUnit' object class.
-     * @return Set of ou names that are parents of given child.
-     */
-    static Set<String> getParents( String name, String contextId )
-    {
-        return HierUtil.getParents( name, getGraph( contextId ) );
-    }
-
-
-    /**
-     * Recursively traverse the {@link org.apache.directory.fortress.core.model.OrgUnit.Type#USER} graph and return number of children a given parent ou has.
-     *
-     * @param name maps to logical {@link org.apache.directory.fortress.core.model.OrgUnit#name} on 'ftOrgUnit' object class.
-     * @return int value contains the number of children of a given parent ou.
-     */
-    static int numChildren( String name, String contextId )
-    {
-        return HierUtil.numChildren( name, getGraph( contextId ) );
-    }
-
-
-    /**
-     * Return Set of {@link org.apache.directory.fortress.core.model.OrgUnit#name}s ascendants contained within {@link org.apache.directory.fortress.core.model.OrgUnit.Type#USER}.
-     *
-     * @param ous contains list of {@link org.apache.directory.fortress.core.model.OrgUnit}s.
-     * @return contains Set of all descendants.
-     */
-    static Set<String> getInherited( List<OrgUnit> ous, String contextId )
-    {
-        // create Set with case insensitive comparator:
-        Set<String> iOUs = new TreeSet<>( String.CASE_INSENSITIVE_ORDER );
-        
-        if ( ObjUtil.isNotNullOrEmpty( ous ) )
-        {
-            for ( OrgUnit ou : ous )
-            {
-                String name = ou.getName();
-                iOUs.add( name );
-                Set<String> parents = HierUtil.getAscendants( name, getGraph( contextId ) );
-                
-                if ( ObjUtil.isNotNullOrEmpty( parents ) )
-                {
-                    iOUs.addAll( parents );
-                }
-            }
-        }
-        
-        return iOUs;
-    }
-
-
-    /**
-     * This api is used by {@link DelAdminMgrImpl} to determine parentage for User OU processing.
-     * It calls {@link HierUtil#validateRelationship(org.jgrapht.graph.SimpleDirectedGraph, String, String, boolean)} to evaluate three OU relationship expressions:
-     * <ol>
-     * <li>If child equals parent</li>
-     * <li>If mustExist true and parent-child relationship exists</li>
-     * <li>If mustExist false and parent-child relationship does not exist</li>
-     * </ol>
-     * Method will throw {@link org.apache.directory.fortress.core.ValidationException} if rule check fails meaning caller failed validation
-     * attempt to add/remove hierarchical relationship failed.
-     *
-     * @param child     contains {@link org.apache.directory.fortress.core.model.OrgUnit#name} of child.
-     * @param parent    contains {@link org.apache.directory.fortress.core.model.OrgUnit#name} of parent.
-     * @param mustExist boolean is used to specify if relationship must be true.
-     * @throws org.apache.directory.fortress.core.ValidationException
-     *          in the event it fails one of the 3 checks.
-     */
-    static void validateRelationship( OrgUnit child, OrgUnit parent, boolean mustExist )
-        throws ValidationException
-    {
-        HierUtil.validateRelationship( getGraph( child.getContextId() ), child.getName(), parent.getName(), mustExist );
-    }
-
-
-    /**
-     * This api allows synchronized access to allow updates to hierarchical relationships.
-     * Method will update the hierarchical data set and reload the JGraphT simple digraph with latest.
-     *
-     * @param contextId maps to sub-tree in DIT, for example ou=contextId, dc=jts, dc = com.
-     * @param relationship contains parent-child relationship targeted for addition.
-     * @param op   used to pass the ldap op {@link org.apache.directory.fortress.core.model.Hier.Op#ADD}, {@link org.apache.directory.fortress.core.model.Hier.Op#MOD}, {@link org.apache.directory.fortress.core.model.Hier.Op#REM}
-     * @throws org.apache.directory.fortress.core.SecurityException in the event of a system error.
-     */
-    static void updateHier( String contextId, Relationship relationship, Hier.Op op ) throws SecurityException
-    {
-        HierUtil.updateHier( getGraph( contextId ), relationship, op );
-    }
-
-
-    /**
-     * Read this ldap record,{@code cn=Hierarchies, ou=OS-P} into this entity, {@link Hier}, before loading into this collection class,{@code org.jgrapht.graph.SimpleDirectedGraph}
-     * using 3rd party lib, <a href="http://www.jgrapht.org/">JGraphT</a>.
-     *
-     * @param contextId maps to sub-tree in DIT, for example ou=contextId, dc=jts, dc = com.
-     * @return handle to simple digraph containing user ou hierarchies.
-     */
-    private static SimpleDirectedGraph<String, Relationship> loadGraph( String contextId )
-    {
-        Hier inHier = new Hier( Hier.Type.ROLE );
-        inHier.setContextId( contextId );
-        LOG.info( "loadGraph initializing USO context [{}]", inHier.getContextId() );
-        
-        List<Graphable> descendants = null;
-        try
-        {
-            OrgUnit orgUnit = new OrgUnit();
-            orgUnit.setType( OrgUnit.Type.USER );
-            orgUnit.setContextId( contextId );
-            descendants = orgUnitP.getAllDescendants( orgUnit );
-        }
-        catch ( SecurityException se )
-        {
-            LOG.info( "loadGraph caught SecurityException={}", se );
-        }
-        
-        Hier hier = HierUtil.loadHier( contextId, descendants );
-        SimpleDirectedGraph<String, Relationship> graph;
-        
-        graph = HierUtil.buildGraph( hier );
-        usoCache.put( getKey( contextId ), graph );
-        
-        return graph;
-    }
-
-
-    /**
-     *
-     * @return handle to simple digraph containing user ou hierarchies.
-     */
-    private static SimpleDirectedGraph<String, Relationship> getGraph( String contextId )
-    {
-        ReadWriteLock hierLock = HierUtil.getLock( contextId, HierUtil.Type.USO );
-        String key = getKey( contextId );
-        
-        try
-        {
-            hierLock.readLock().lock();
-            SimpleDirectedGraph<String, Relationship> graph = ( SimpleDirectedGraph<String, Relationship> ) usoCache
-                .get( key );
-            
-            if ( graph == null )
-            {
-                try
-                {
-                    hierLock.readLock().unlock();
-                    hierLock.writeLock().lock();
-
-                    // TODO: determine why this (code that was commented out) creates a deadlock:
-                    //graph = ( SimpleDirectedGraph<String, Relationship> ) usoCache.get( key );
-
-                    //if ( graph == null )
-                    //{
-                        graph = loadGraph( contextId );
-                    //}
-                    hierLock.readLock().lock();
-                }
-                finally
-                {
-                    hierLock.writeLock().unlock();
-                }
-            }
-            
-            return graph;
-        }
-        finally
-        {
-            hierLock.readLock().unlock();
-        }
-    }
-
-
-    private static String getKey( String contextId )
-    {
-        String key = HierUtil.Type.USO.toString();
-        if ( StringUtils.isNotEmpty( contextId ) && !contextId.equalsIgnoreCase( GlobalIds.NULL ) )
-        {
-            key += ":" + contextId;
-        }
-        return key;
-    }
-}
\ No newline at end of file

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rbac/package.html
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rbac/package.html b/src/main/java/org/apache/directory/fortress/core/rbac/package.html
deleted file mode 100755
index 91cfc15..0000000
--- a/src/main/java/org/apache/directory/fortress/core/rbac/package.html
+++ /dev/null
@@ -1,52 +0,0 @@
-<!--
- *   Licensed to the Apache Software Foundation (ASF) under one
- *   or more contributor license agreements.  See the NOTICE file
- *   distributed with this work for additional information
- *   regarding copyright ownership.  The ASF licenses this file
- *   to you under the Apache License, Version 2.0 (the
- *   "License"); you may not use this file except in compliance
- *   with the License.  You may obtain a copy of the License at
- *
- *     http://www.apache.org/licenses/LICENSE-2.0
- *
- *   Unless required by applicable law or agreed to in writing,
- *   software distributed under the License is distributed on an
- *   "AS IS" BASIS, WITHOUT WARRANTIES OR CONDITIONS OF ANY
- *   KIND, either express or implied.  See the License for the
- *   specific language governing permissions and limitations
- *   under the License.
- *
--->
-<html>
-   <head>
-      <title>Package Documentation for org.apache.directory.fortress.core.rbac</title>
-   </head>
-   <body>
-      <p>
-      <ol>
-      <li>This package provides data entities and apis that adhere to standards like <a href="http://csrc.nist.gov/groups/SNS/rbac/documents/draft-rbac-implementation-std-v01.pdf">RBAC</a></li>
-      <li>This package provides data entities and APIs that adhere to <a href="http://profsandhu.com/journals/tissec/p113-oh.pdf">ARBAC02</a> model to provide delegated administration capabilities.</li>
-      <li>This package provides data entities and APIs to interrogate <a href="http://openldap.org/">OpenLDAP</a>'s slapd access log to obtain fortress audit trail and historical events.</li>
-      <li>This package uses <a href="http://www.jgrapht.org/">JGraphT</a> to provide hierarchical functionality that is reused within several other packages in the fortress system.</li>
-      </ol>
-      </p>
-      <p>
-         The <b>org.apache.directory.fortress.core.rbac</b> package data entities may be manipulated by outside
-          programs but the apis contained within this package are for fortress use only.
-          The entities include {@link org.apache.directory.fortress.core.model.User User},{@link org.apache.directory.fortress.core.model.Role Role}, {@link org.apache.directory.fortress.core.model.PermObj PermObj}, {@link org.apache.directory.fortress.core.model.Permission Permission}, and more.
-          See the corresponding javadoc contained with this package for more info.
-      </p>
-      <h3>Fortress Manager APIs implemented in this package</h3>
-       <ol>
-       <li><a href="AccessMgrImpl.html">AccessMgrImpl</a> - This implements using LDAP the runtime access control operations on objects that are provisioned <a href="http://csrc.nist.gov/groups/SNS/rbac/documents/draft-rbac-implementation-std-v01.pdf">RBAC</a> entities that reside in LDAP directory.</li>
-       <li><a href="AdminMgrImpl.html">AdminMgrImpl</a> - This implements using LDAP the administrative functions to provision Fortress <a href="http://csrc.nist.gov/groups/SNS/rbac/documents/draft-rbac-implementation-std-v01.pdf">RBAC</a> entities into the LDAP directory.</li>
-       <li><a href="ReviewMgrImpl.html">ReviewMgrImpl</a> - This implements using LDAP the administrative review functions on already provisioned Fortress <a href="http://csrc.nist.gov/groups/SNS/rbac/documents/draft-rbac-implementation-std-v01.pdf">RBAC</a> entities that reside in LDAP directory.</li>
-       <li><a href="DelegatedAccessMgrImpl.html">DelegatedAccessMgrImpl</a> - This implements using LDAP the APIs for performing runtime delegated access control operations on objects that are provisioned Fortress <a href="http://profsandhu.com/journals/tissec/p113-oh.pdf">ARBAC02</a> entities that reside in LDAP directory.</li>
-       <li><a href="DelegatedAdminMgrImpl.html">DelegatedAdminMgrImpl</a> - This implements using LDAP the <a href="http://profsandhu.com/journals/tissec/p113-oh.pdf">ARBAC02</a> DelegatedAdminMgr interface for performing policy administration of Fortress ARBAC entities that reside in LDAP directory.</li>
-       <li><a href="DelegatedReviewMgrImpl.html">DelegatedReviewMgrImpl</a> - This implements using LDAP the <a href="http://profsandhu.com/journals/tissec/p113-oh.pdf">ARBAC02</a> DelegatedReviewMgr interface for performing policy interrogation of provisioned Fortress ARBAC02 entities that reside in LDAP directory.</li>
-       <li><a href="PwPolicyMgrImpl.html">PwPolicyMgrImpl</a> - Performs CRUD on OpenLDAP password policies stored in directory.</li>
-       <li><a href="AuditMgrImpl.html">AuditMgrImpl</a> - Interrogates Fortress audit data stored in OpenLDAP.</li>
-       </ol>
-   </body>
-</html>
-

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/AccessMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/AccessMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/AccessMgrRestImpl.java
index d8128fa..1137b6b 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/AccessMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/AccessMgrRestImpl.java
@@ -24,8 +24,8 @@ import org.apache.directory.fortress.core.GlobalErrIds;
 import org.apache.directory.fortress.core.SecurityException;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.AccessMgrImpl;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.AccessMgrImpl;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Session;
 import org.apache.directory.fortress.core.model.User;
@@ -125,7 +125,7 @@ public class AccessMgrRestImpl extends Manageable implements AccessMgr
      * <li> fail for any user who is locked by OpenLDAP's policies {@link User#isLocked()}, regardless of trusted flag being set as parm on API.
      * <li> evaluate temporal {@link org.apache.directory.fortress.core.util.time.Constraint}(s) on {@link User}, {@link UserRole} and {@link org.apache.directory.fortress.core.model.UserAdminRole} entities.
      * <li> process selective role activations into User RBAC Session {@link User#roles}.
-     * <li> check Dynamic Separation of Duties {@link org.apache.directory.fortress.core.rbac.DSDChecker#validate(Session, org.apache.directory.fortress.core.util.time.Constraint, org.apache.directory.fortress.core.util.time.Time)} on {@link User#roles}.
+     * <li> check Dynamic Separation of Duties {@link org.apache.directory.fortress.core.impl.DSDChecker#validate(Session, org.apache.directory.fortress.core.util.time.Constraint, org.apache.directory.fortress.core.util.time.Time)} on {@link User#roles}.
      * <li> process selective administrative role activations {@link User#adminRoles}.
      * <li> return a {@link Session} containing {@link Session#getUser()}, {@link Session#getRoles()} and (if admin user) {@link Session#getAdminRoles()} if everything checks out good.
      * <li> throw a checked exception that will be {@link org.apache.directory.fortress.core.SecurityException} or its derivation.
@@ -198,7 +198,7 @@ public class AccessMgrRestImpl extends Manageable implements AccessMgr
     }
 
     /**
-     * Perform user rbac authorization.  This function returns a Boolean value meaning whether the subject of a given session is
+     * Perform user impl authorization.  This function returns a Boolean value meaning whether the subject of a given session is
      * allowed or not to perform a given operation on a given object. The function is valid if and
      * only if the session is a valid Fortress session, the object is a member of the OBJS data set,
      * and the operation is a member of the OPS data set. The session's subject has the permission

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/AdminMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/AdminMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/AdminMgrRestImpl.java
index 7323fdb..bbd68ff 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/AdminMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/AdminMgrRestImpl.java
@@ -33,7 +33,7 @@ import org.apache.directory.fortress.core.model.RoleRelationship;
 import org.apache.directory.fortress.core.model.SDSet;
 import org.apache.directory.fortress.core.model.User;
 import org.apache.directory.fortress.core.model.UserRole;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.VUtil;
 
 

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/AuditMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/AuditMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/AuditMgrRestImpl.java
index 3610592..ca62f90 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/AuditMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/AuditMgrRestImpl.java
@@ -26,7 +26,7 @@ import org.apache.directory.fortress.core.model.AuthZ;
 import org.apache.directory.fortress.core.model.Bind;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.Mod;
 import org.apache.directory.fortress.core.model.UserAudit;
 import org.apache.directory.fortress.core.model.VUtil;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/DelAdminMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/DelAdminMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/DelAdminMgrRestImpl.java
index 5ac9c14..9bea3a7 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/DelAdminMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/DelAdminMgrRestImpl.java
@@ -25,7 +25,7 @@ import org.apache.directory.fortress.core.model.AdminRole;
 import org.apache.directory.fortress.core.model.AdminRoleRelationship;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.OrgUnit;
 import org.apache.directory.fortress.core.model.OrgUnitRelationship;
 import org.apache.directory.fortress.core.model.PermGrant;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/DelReviewMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/DelReviewMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/DelReviewMgrRestImpl.java
index 30ed58e..f10482d 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/DelReviewMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/DelReviewMgrRestImpl.java
@@ -25,7 +25,7 @@ import org.apache.directory.fortress.core.DelReviewMgr;
 import org.apache.directory.fortress.core.model.AdminRole;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.OrgUnit;
 import org.apache.directory.fortress.core.model.UserAdminRole;
 import org.apache.directory.fortress.core.model.User;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/PwPolicyMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/PwPolicyMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/PwPolicyMgrRestImpl.java
index 768c02d..e45c571 100644
--- a/src/main/java/org/apache/directory/fortress/core/rest/PwPolicyMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/PwPolicyMgrRestImpl.java
@@ -25,7 +25,7 @@ import org.apache.directory.fortress.core.PwPolicyMgr;
 import org.apache.directory.fortress.core.SecurityException;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.PwPolicy;
 import org.apache.directory.fortress.core.model.VUtil;
 

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/rest/ReviewMgrRestImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/rest/ReviewMgrRestImpl.java b/src/main/java/org/apache/directory/fortress/core/rest/ReviewMgrRestImpl.java
index 2df59e0..bda0d26 100755
--- a/src/main/java/org/apache/directory/fortress/core/rest/ReviewMgrRestImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/rest/ReviewMgrRestImpl.java
@@ -24,7 +24,7 @@ import org.apache.directory.fortress.core.ReviewMgr;
 import org.apache.directory.fortress.core.SecurityException;
 import org.apache.directory.fortress.core.model.FortRequest;
 import org.apache.directory.fortress.core.model.FortResponse;
-import org.apache.directory.fortress.core.rbac.Manageable;
+import org.apache.directory.fortress.core.impl.Manageable;
 import org.apache.directory.fortress.core.model.OrgUnit;
 import org.apache.directory.fortress.core.model.PermObj;
 import org.apache.directory.fortress.core.model.Permission;
@@ -552,7 +552,7 @@ public class ReviewMgrRestImpl extends Manageable implements ReviewMgr
      * This function returns the set of users assigned to a given role. The function is valid if and
      * only if the role is a member of the ROLES data set.
      * The max number of users returned is constrained by limit argument.
-     * This method is used by the Websphere realm component.  This method does NOT use hierarchical rbac.
+     * This method is used by the Websphere realm component.  This method does NOT use hierarchical impl.
      * <h4>required parameters</h4>
      * <ul>
      * <li>{@link Role#name} - contains the name to use for the Role targeted for search.</li>

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java b/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
index 159ca61..e306f92 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/attr/AttrHelper.java
@@ -27,7 +27,6 @@ import org.apache.directory.fortress.core.GlobalIds;
 import org.apache.directory.fortress.core.model.Administrator;
 import org.apache.directory.fortress.core.model.AuthZ;
 import org.apache.directory.fortress.core.model.Permission;
-import org.apache.directory.fortress.core.model.VUtil;
 import org.apache.directory.fortress.core.util.ObjUtil;
 
 import java.text.ParseException;
@@ -38,7 +37,7 @@ import java.util.Properties;
 
 /**
  * Class contains static utilities for various conversions of ldap data to application entity and back.  These methods are called by the
- * Fortress DAO's, i.e. {@link org.apache.directory.fortress.core.rbac.UserDAO}.
+ * Fortress DAO's, i.e. {@link org.apache.directory.fortress.core.impl.UserDAO}.
  * These are low-level data utilities and no validations are performed.  These apis should not be called by outside programs.
  * <p/>
  * This class is thread safe.

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java b/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
index e2735e1..ac278b0 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/time/CUtil.java
@@ -54,7 +54,7 @@ import java.util.StringTokenizer;
  * <li>{@link org.apache.directory.fortress.core.GlobalIds#VALIDATOR_PROPS}2={@link Timeout}</li>
  * <li>{@link org.apache.directory.fortress.core.GlobalIds#VALIDATOR_PROPS}3={@link ClockTime}</li>
  * <li>{@link org.apache.directory.fortress.core.GlobalIds#VALIDATOR_PROPS}4={@link Day}</li>
- * <li>{@link org.apache.directory.fortress.core.GlobalIds#DSD_VALIDATOR_PROP}={@link org.apache.directory.fortress.core.rbac.DSDChecker}</li>
+ * <li>{@link org.apache.directory.fortress.core.GlobalIds#DSD_VALIDATOR_PROP}={@link org.apache.directory.fortress.core.impl.DSDChecker}</li>
  * </ol>
  * </p>
  *
@@ -474,7 +474,7 @@ public class CUtil
             {
                 if ( ObjUtil.isNotNullOrEmpty( session.getRoles() ) )
                 {
-                    // now check the constraint on every rbac role activation candidate contained within session object:
+                    // now check the constraint on every impl role activation candidate contained within session object:
                     ListIterator<UserRole> roleItems = session.getRoles().listIterator();
 
                     while ( roleItems.hasNext() )
@@ -515,7 +515,7 @@ public class CUtil
             }
         }
 
-        // now perform DSD validation on session's rbac roles:
+        // now perform DSD validation on session's impl roles:
         if ( checkDsd && DSDVALIDATOR != null && DSDVALIDATOR.length() > 0 && type == ConstraintType.ROLE
             && ObjUtil.isNotNullOrEmpty( session.getRoles() ) )
         {

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/main/java/org/apache/directory/fortress/core/util/time/Validator.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/util/time/Validator.java b/src/main/java/org/apache/directory/fortress/core/util/time/Validator.java
index 314b781..aa911fc 100755
--- a/src/main/java/org/apache/directory/fortress/core/util/time/Validator.java
+++ b/src/main/java/org/apache/directory/fortress/core/util/time/Validator.java
@@ -42,7 +42,7 @@ import org.apache.directory.fortress.core.model.Session;
  * <li>Days of week: {@link Day}</li>
  * <li>Timeout:      {@link Timeout}</li>
  * <li>Lock dates:   {@link LockDate}</li>
- * <li>DSDs:         {@link org.apache.directory.fortress.core.rbac.DSDChecker}</li>
+ * <li>DSDs:         {@link org.apache.directory.fortress.core.impl.DSDChecker}</li>
  * </ol>
  * </p>
  * <h4> Constraint Error Codes </h4>

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/AccelMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/AccelMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/AccelMgrConsole.java
index 1413c3b..f231229 100644
--- a/src/test/java/org/apache/directory/fortress/core/AccelMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/AccelMgrConsole.java
@@ -19,7 +19,7 @@
  */
 package org.apache.directory.fortress.core;
 
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Session;
 import org.apache.directory.fortress.core.model.User;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/AccessMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/AccessMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/AccessMgrConsole.java
index 902a897..e5cf5d8 100755
--- a/src/test/java/org/apache/directory/fortress/core/AccessMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/AccessMgrConsole.java
@@ -19,7 +19,7 @@
  */
 package org.apache.directory.fortress.core;
 
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.UserAdminRole;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Session;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/AdminMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/AdminMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/AdminMgrConsole.java
index 6b83c23..1a778df 100755
--- a/src/test/java/org/apache/directory/fortress/core/AdminMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/AdminMgrConsole.java
@@ -24,19 +24,18 @@ import org.apache.directory.fortress.core.util.Config;
 import org.apache.directory.fortress.core.example.Example;
 import org.apache.directory.fortress.core.example.ExampleAdminMgr;
 import org.apache.directory.fortress.core.example.ExampleAdminMgrFactory;
-import org.apache.directory.fortress.core.rbac.MyAnnotation;
-import org.apache.directory.fortress.core.rbac.PolicyTestData;
+import org.apache.directory.fortress.core.impl.MyAnnotation;
+import org.apache.directory.fortress.core.impl.PolicyTestData;
 import org.apache.directory.fortress.core.model.Hier;
 import org.apache.directory.fortress.core.model.PermObj;
 import org.apache.directory.fortress.core.model.Relationship;
 import org.apache.directory.fortress.core.model.Role;
 import org.apache.directory.fortress.core.model.SDSet;
 import org.apache.directory.fortress.core.model.Session;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.User;
 import org.apache.directory.fortress.core.model.UserRole;
-import org.apache.directory.fortress.core.model.VUtil;
 import org.apache.directory.fortress.core.util.ObjUtil;
 import org.apache.directory.fortress.core.util.time.Constraint;
 import org.slf4j.Logger;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/AuditMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/AuditMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/AuditMgrConsole.java
index 610d62f..57665b3 100755
--- a/src/test/java/org/apache/directory/fortress/core/AuditMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/AuditMgrConsole.java
@@ -24,7 +24,7 @@ import org.apache.directory.api.ldap.model.exception.LdapInvalidDnException;
 import org.apache.directory.fortress.core.model.Bind;
 import org.apache.directory.fortress.core.model.AuthZ;
 import org.apache.directory.fortress.core.model.Mod;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.UserAudit;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.util.attr.AttrHelper;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/DelegatedAccessMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/DelegatedAccessMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/DelegatedAccessMgrConsole.java
index 09d1090..49f99f5 100755
--- a/src/test/java/org/apache/directory/fortress/core/DelegatedAccessMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/DelegatedAccessMgrConsole.java
@@ -22,7 +22,7 @@ package org.apache.directory.fortress.core;
 
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Role;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.User;
 import org.apache.directory.fortress.core.model.Session;
 import org.apache.directory.fortress.core.model.VUtil;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/DelegatedAdminMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/DelegatedAdminMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/DelegatedAdminMgrConsole.java
index 05ddc1b..016bc87 100755
--- a/src/test/java/org/apache/directory/fortress/core/DelegatedAdminMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/DelegatedAdminMgrConsole.java
@@ -21,7 +21,7 @@ package org.apache.directory.fortress.core;
 
 import org.apache.directory.fortress.core.model.AdminRole;
 import org.apache.directory.fortress.core.model.OrgUnit;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.UserAdminRole;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Role;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/DelegatedReviewMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/DelegatedReviewMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/DelegatedReviewMgrConsole.java
index 62462a3..034184c 100755
--- a/src/test/java/org/apache/directory/fortress/core/DelegatedReviewMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/DelegatedReviewMgrConsole.java
@@ -21,7 +21,7 @@ package org.apache.directory.fortress.core;
 
 
 import org.apache.directory.fortress.core.model.AdminRole;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.UserAdminRole;
 import org.apache.directory.fortress.core.model.User;
 

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/GroupMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/GroupMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/GroupMgrConsole.java
index 7a3f53e..1e4d16c 100755
--- a/src/test/java/org/apache/directory/fortress/core/GroupMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/GroupMgrConsole.java
@@ -21,9 +21,6 @@ package org.apache.directory.fortress.core;
 
 import org.apache.commons.lang.StringUtils;
 import org.apache.directory.fortress.core.model.Group;
-import org.apache.directory.fortress.core.ldap.group.GroupMgr;
-import org.apache.directory.fortress.core.ldap.group.GroupMgrFactory;
-import org.apache.directory.fortress.core.model.VUtil;
 
 import java.util.Enumeration;
 import java.util.List;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/PolicyMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/PolicyMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/PolicyMgrConsole.java
index 8f878ae..7fab3ed 100755
--- a/src/test/java/org/apache/directory/fortress/core/PolicyMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/PolicyMgrConsole.java
@@ -21,7 +21,7 @@ package org.apache.directory.fortress.core;
 
 
 import org.apache.directory.fortress.core.model.PwPolicy;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/ReviewMgrConsole.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/ReviewMgrConsole.java b/src/test/java/org/apache/directory/fortress/core/ReviewMgrConsole.java
index f6aeb99..3a4c6cc 100755
--- a/src/test/java/org/apache/directory/fortress/core/ReviewMgrConsole.java
+++ b/src/test/java/org/apache/directory/fortress/core/ReviewMgrConsole.java
@@ -23,14 +23,13 @@ package org.apache.directory.fortress.core;
 import org.apache.directory.fortress.core.util.ObjUtil;
 import org.slf4j.LoggerFactory;
 import org.apache.directory.fortress.core.model.OrgUnit;
-import org.apache.directory.fortress.core.rbac.TestUtils;
+import org.apache.directory.fortress.core.impl.TestUtils;
 import org.apache.directory.fortress.core.model.UserAdminRole;
 import org.apache.directory.fortress.core.model.Address;
 import org.apache.directory.fortress.core.model.Permission;
 import org.apache.directory.fortress.core.model.Role;
 import org.apache.directory.fortress.core.model.User;
 import org.apache.directory.fortress.core.model.UserRole;
-import org.apache.directory.fortress.core.model.VUtil;
 import org.apache.directory.fortress.core.util.time.Constraint;
 
 import java.util.ArrayList;

http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/ba64d26a/src/test/java/org/apache/directory/fortress/core/group/GroupAntTest.java
----------------------------------------------------------------------
diff --git a/src/test/java/org/apache/directory/fortress/core/group/GroupAntTest.java b/src/test/java/org/apache/directory/fortress/core/group/GroupAntTest.java
index 62b00c3..fb5e916 100644
--- a/src/test/java/org/apache/directory/fortress/core/group/GroupAntTest.java
+++ b/src/test/java/org/apache/directory/fortress/core/group/GroupAntTest.java
@@ -28,8 +28,8 @@ import org.junit.runner.Result;
 import org.junit.runner.notification.Failure;
 import org.apache.directory.fortress.core.ant.Addgroup;
 import org.apache.directory.fortress.core.model.Group;
-import org.apache.directory.fortress.core.ldap.group.GroupMgr;
-import org.apache.directory.fortress.core.ldap.group.GroupMgrFactory;
+import org.apache.directory.fortress.core.GroupMgr;
+import org.apache.directory.fortress.core.GroupMgrFactory;
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;