Merge pull request #980 from mrjoel/mrjoel-httpheaders

James Moger

2015-12-10 7b7b0d54b606e5a7d63ea39ec8918968f612d61d

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

@@ -18,11 +18,14 @@
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;
@@ -73,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)) {
@@ -116,14 +121,32 @@

   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();
@@ -132,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();

@@ -140,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();

@@ -148,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();

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

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

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

       @Override
      public void stop() {

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

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

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

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

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