Merge pull request #1073 from gitblit/1062-DocEditorUpdates

Paul Martin

2016-04-30 a502d96a860456ec5e8c96761db70f7cabb74751

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

@@ -16,18 +16,23 @@
package com.gitblit.auth;

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

import javax.servlet.http.HttpServletRequest;

import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import com.gitblit.Constants.AccountType;
import com.gitblit.Constants.Role;
import com.gitblit.Constants.AuthenticationType;
import com.gitblit.IStoredSettings;
import com.gitblit.manager.IRuntimeManager;
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 {
@@ -71,6 +76,8 @@
      return serviceName;
   }

   public abstract AuthenticationType getAuthenticationType();

   protected void setCookie(UserModel user, char [] password) {
      // create a user cookie
      if (StringUtils.isEmpty(user.cookie) && !ArrayUtils.isEmpty(password)) {
@@ -79,38 +86,67 @@
   }

   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();

   /**
    * Used to handle requests for requests for pages requiring authentication.
    * This allows authentication to occur based on the contents of the request
    * itself.
    *
    * @param httpRequest
    * @return
    */
   public abstract UserModel authenticate(HttpServletRequest httpRequest);

   /**
    * Used to authentication user/password credentials, both for login form
    * and HTTP Basic authentication processing.
    *
    * @param username
    * @param password
    * @return
    */
   public abstract UserModel authenticate(String username, char[] password);

   public abstract AccountType getAccountType();

   /**
    * Does the user service support changes to credentials?
    * Returns true if the users's credentials can be changed.
    *
    * @return true or false
    * @return true if the authentication provider supports credential changes
    * @since 1.0.0
    */
   public abstract boolean supportsCredentialChanges();
@@ -119,7 +155,7 @@
    * Returns true if the user's display name can be changed.
    *
    * @param user
    * @return true if the user service supports display name changes
    * @return true if the authentication provider supports display name changes
    */
   public abstract boolean supportsDisplayNameChanges();

@@ -127,7 +163,7 @@
    * Returns true if the user's email address can be changed.
    *
    * @param user
    * @return true if the user service supports email address changes
    * @return true if the authentication provider supports email address changes
    */
   public abstract boolean supportsEmailAddressChanges();

@@ -135,7 +171,7 @@
    * Returns true if the user's team memberships can be changed.
    *
    * @param user
    * @return true if the user service supports team membership changes
    * @return true if the authentication provider supports team membership changes
    */
   public abstract boolean supportsTeamMembershipChanges();

@@ -167,6 +203,16 @@
          super(serviceName);
       }

      @Override
      public UserModel authenticate(HttpServletRequest httpRequest) {
         return null;
      }

      @Override
      public AuthenticationType getAuthenticationType() {
         return AuthenticationType.CREDENTIALS;
      }

       @Override
      public void stop() {

@@ -190,6 +236,11 @@
      }

      @Override
      public UserModel authenticate(HttpServletRequest httpRequest) {
         return null;
      }

      @Override
      public UserModel authenticate(String username, char[] password) {
         return null;
      }
@@ -200,6 +251,11 @@
      }

      @Override
      public AuthenticationType getAuthenticationType() {
         return null;
      }

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