Implement mirror executor (issue-5)

James Moger

2013-11-13 c44dd099a432094a12131cf60dfc8a19f5aa8101

 src/main/java/com/gitblit/GitblitUserService.java

@@ -33,26 +33,24 @@
/**

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

 * point for custom user service implementations.

 * 
 *

 * This does seem a little convoluted, but the idea is to allow IUserService to

 * evolve with new methods and implementations without breaking custom

 * authentication implementations.

 * 
 *

 * The most common implementation of a custom IUserService is to only override

 * authentication and then delegate all other functionality to one of Gitblit's

 * user services. This class optimizes that use-case.

 * 
 *

 * Extending GitblitUserService allows for authentication customization without

 * having to keep-up-with IUSerService API changes.

 * 
 *

 * @author James Moger

 * 
 *

 */

public class GitblitUserService implements IUserService {



   protected IUserService serviceImpl;

	
   protected final String ExternalAccount = "#externalAccount";



   private final Logger logger = LoggerFactory.getLogger(GitblitUserService.class);



@@ -66,13 +64,9 @@
      logger.info("GUS delegating to " + serviceImpl.toString());

   }



   @SuppressWarnings("deprecation")

   protected IUserService createUserService(File realmFile) {

      IUserService service = null;

      if (realmFile.getName().toLowerCase().endsWith(".properties")) {

         // v0.5.0 - v0.7.0 properties-based realm file

         service = new FileUserService(realmFile);

      } else if (realmFile.getName().toLowerCase().endsWith(".conf")) {

      if (realmFile.getName().toLowerCase().endsWith(".conf")) {

         // v0.8.0+ config-based realm file

         service = new ConfigUserService(realmFile);

      }

@@ -93,25 +87,9 @@
         service.updateUserModel(admin);

      }



      if (service instanceof FileUserService) {

         // automatically create a users.conf realm file from the original

         // users.properties file

         File usersConfig = new File(realmFile.getParentFile(), "users.conf");

         if (!usersConfig.exists()) {

            logger.info(MessageFormat.format("Automatically creating {0} based on {1}",

                  usersConfig.getAbsolutePath(), realmFile.getAbsolutePath()));

            ConfigUserService configService = new ConfigUserService(usersConfig);

            for (String username : service.getAllUsernames()) {

               UserModel userModel = service.getUserModel(username);

               configService.updateUserModel(userModel);

            }

         }

         // issue suggestion about switching to users.conf

         logger.warn("Please consider using \"users.conf\" instead of the deprecated \"users.properties\" file");

      }

      return service;

   }

	


   @Override

   public String toString() {

      return getClass().getSimpleName();

@@ -160,7 +138,7 @@
      setAccountType(user);

      return user;

   }

	


   @Override

   public void logout(UserModel user) {

      serviceImpl.logout(user);

@@ -189,7 +167,7 @@
         if (!model.isLocalAccount() && !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);

@@ -202,7 +180,7 @@
         if (!model.isLocalAccount() && !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);

@@ -233,7 +211,7 @@
       for (UserModel user : users) {

          setAccountType(user);

       }

      return users; 
      return users;

   }



   @Override

@@ -277,7 +255,7 @@
      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);

@@ -315,16 +293,16 @@
   public boolean deleteRepositoryRole(String role) {

      return serviceImpl.deleteRepositoryRole(role);

   }

	


   protected boolean isLocalAccount(String username) {

      UserModel user = getUserModel(username);

      return user != null && user.isLocalAccount();

   }

	


   protected void setAccountType(UserModel user) {

      if (user != null) {

         if (!StringUtils.isEmpty(user.password)

               && !ExternalAccount.equalsIgnoreCase(user.password)

               && !Constants.EXTERNAL_ACCOUNT.equalsIgnoreCase(user.password)

               && !"StoredInLDAP".equalsIgnoreCase(user.password)) {

            user.accountType = AccountType.LOCAL;

         } else {

@@ -332,7 +310,7 @@
         }

      }

   }

	


   protected AccountType getAccountType() {

      return AccountType.LOCAL;

   }