Minor refactoring of user/team checksumming

James Moger

2015-03-07 45ed92bd767e9e88d70c14d81e8bbb7acd2e5fe2

 src/main/java/com/gitblit/auth/AuthenticationProvider.java

@@ -16,6 +16,7 @@
package com.gitblit.auth;

import java.io.File;
import java.math.BigInteger;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;
@@ -26,6 +27,9 @@
import com.gitblit.manager.IUserManager;
import com.gitblit.models.TeamModel;
import com.gitblit.models.UserModel;
import com.gitblit.utils.ArrayUtils;
import com.gitblit.utils.DeepCopier;
import com.gitblit.utils.StringUtils;

public abstract class AuthenticationProvider {

@@ -68,28 +72,48 @@
      return serviceName;
   }

   protected void setCookie(UserModel user, char [] password) {
      // create a user cookie
      if (StringUtils.isEmpty(user.cookie) && !ArrayUtils.isEmpty(password)) {
         user.cookie = StringUtils.getSHA1(user.username + new String(password));
      }
   }

   protected void updateUser(UserModel userModel) {
      // TODO implement user model change detection
      // account for new user and revised user
      final UserModel userLocalDB = userManager.getUserModel(userModel.getName());

      // username
      // displayname
      // email address
      // cookie
      // Establish the checksum of the current version of the user
      final BigInteger userCurrentCheck = DeepCopier.checksum(userModel);

      userManager.updateUserModel(userModel);
      // Establish the checksum of the stored version of the user
      final BigInteger userLocalDBcheck = DeepCopier.checksum(userLocalDB);

      // Compare the checksums
      if (!userCurrentCheck.equals(userLocalDBcheck)) {
         // If mismatch, save the new instance.
         userManager.updateUserModel(userModel);
      }
   }

   protected void updateTeam(TeamModel teamModel) {
      // TODO implement team model change detection
      // account for new team and revised team
      final TeamModel teamLocalDB = userManager.getTeamModel(teamModel.name);

      // memberships
      // Establish the checksum of the current version of the team
      final BigInteger teamCurrentCheck = DeepCopier.checksum(teamModel);

      userManager.updateTeamModel(teamModel);
      // Establish the checksum of the stored version of the team
      final BigInteger teamLocalDBcheck = DeepCopier.checksum(teamLocalDB);

      // Compare the checksums
      if (!teamCurrentCheck.equals(teamLocalDBcheck)) {
         // If mismatch, save the new instance.
         userManager.updateTeamModel(teamModel);
      }
   }

   public abstract void setup();

   public abstract void stop();

   public abstract UserModel authenticate(String username, char[] password);

@@ -136,6 +160,11 @@
       protected UsernamePasswordAuthenticationProvider(String serviceName) {
          super(serviceName);
       }

       @Override
      public void stop() {

      }
    }

    public static class NullProvider extends AuthenticationProvider {
@@ -146,6 +175,11 @@

      @Override
      public void setup() {

      }

      @Override
      public void stop() {

      }

@@ -161,22 +195,22 @@

      @Override
      public boolean supportsCredentialChanges() {
         return false;
         return true;
      }

      @Override
      public boolean supportsDisplayNameChanges() {
         return false;
         return true;
      }

      @Override
      public boolean supportsEmailAddressChanges() {
         return false;
         return true;
      }

      @Override
      public boolean supportsTeamMembershipChanges() {
         return false;
         return true;
      }
    }
}