Rework LDAP implementation with unboundid. Also allows for an LDAP server ...

John Crygier

2012-04-10 f3b625d298bab922c64192c25914e352bd87e59e

 src/com/gitblit/GitblitUserService.java

@@ -25,6 +25,7 @@


import com.gitblit.models.TeamModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.DeepCopier;



/**

 * This class wraps the default user service and is recommended as the starting

@@ -112,6 +113,16 @@
   }



   @Override

   public boolean supportsCredentialChanges() {

      return serviceImpl.supportsCredentialChanges();

   }



   @Override

   public boolean supportsTeamMembershipChanges() {

      return serviceImpl.supportsTeamMembershipChanges();

   }



   @Override

   public boolean supportsCookies() {

      return serviceImpl.supportsCookies();

   }

@@ -143,9 +154,33 @@


   @Override

   public boolean updateUserModel(String username, UserModel model) {

      return serviceImpl.updateUserModel(username, model);

      if (supportsCredentialChanges()) {

         if (!supportsTeamMembershipChanges()) {

            //  teams are externally controlled - copy from original model

            UserModel existingModel = getUserModel(username);

				
            model = DeepCopier.copy(model);

            model.teams.clear();

            model.teams.addAll(existingModel.teams);

         }

         return serviceImpl.updateUserModel(username, model);

      }

      if (model.username.equals(username)) {

         // passwords are not persisted by the backing user service

         model.password = null;

         if (!supportsTeamMembershipChanges()) {

            //  teams are externally controlled- copy from original model

            UserModel existingModel = getUserModel(username);

				
            model = DeepCopier.copy(model);

            model.teams.clear();

            model.teams.addAll(existingModel.teams);

         }

         return serviceImpl.updateUserModel(username, model);

      }

      logger.error("Users can not be renamed!");

      return false;

   }



   @Override

   public boolean deleteUserModel(UserModel model) {

      return serviceImpl.deleteUserModel(model);

@@ -198,6 +233,14 @@


   @Override

   public boolean updateTeamModel(String teamname, TeamModel model) {

      if (!supportsTeamMembershipChanges()) {

         // teams are externally controlled - copy from original model

         TeamModel existingModel = getTeamModel(teamname);

			
         model = DeepCopier.copy(model);

         model.users.clear();

         model.users.addAll(existingModel.users);

      }

      return serviceImpl.updateTeamModel(teamname, model);

   }