Merge branch 'develop'

James Moger

2015-11-22 ed552ba47c02779c270ffd62841d6d1048dade70

 src/main/java/com/gitblit/servlet/AuthenticationFilter.java

@@ -21,6 +21,7 @@
import java.util.HashMap;

import java.util.Map;



import javax.servlet.Filter;
import javax.servlet.FilterChain;

import javax.servlet.FilterConfig;

import javax.servlet.ServletException;

@@ -35,13 +36,11 @@
import org.slf4j.LoggerFactory;



import com.gitblit.Constants;

import com.gitblit.dagger.DaggerFilter;

import com.gitblit.Constants.Role;
import com.gitblit.manager.IAuthenticationManager;

import com.gitblit.models.UserModel;

import com.gitblit.utils.DeepCopier;

import com.gitblit.utils.StringUtils;



import dagger.ObjectGraph;



/**

 * The AuthenticationFilter is a servlet filter that preprocesses requests that

@@ -52,7 +51,7 @@
 * @author James Moger

 *

 */

public abstract class AuthenticationFilter extends DaggerFilter {

public abstract class AuthenticationFilter implements Filter {


   protected static final String CHALLENGE = "Basic realm=\"" + Constants.NAME + "\"";



@@ -62,9 +61,16 @@


   protected IAuthenticationManager authenticationManager;



   protected AuthenticationFilter(IAuthenticationManager authenticationManager) {
      this.authenticationManager = authenticationManager;
   }

   @Override

   protected void inject(ObjectGraph dagger, FilterConfig filterConfig) {

      this.authenticationManager = dagger.get(IAuthenticationManager.class);

   public void init(FilterConfig filterConfig) throws ServletException {
   }

   @Override
   public void destroy() {
   }



   /**

@@ -167,7 +173,7 @@


      @Override

      public boolean isUserInRole(String role) {

         if (role.equals(Constants.ADMIN_ROLE)) {

         if (role.equals(Role.ADMIN.getRole())) {
            return user.canAdmin();

         }

         // Gitblit does not currently use actual roles in the traditional