James Moger
2014-06-30 94146dd495aad8157c423bfd9a1ae47c8e26b32c
src/main/java/com/gitblit/servlet/AuthenticationFilter.java
@@ -36,7 +36,7 @@
import org.slf4j.LoggerFactory;
import com.gitblit.Constants;
import com.gitblit.manager.ISessionManager;
import com.gitblit.manager.IAuthenticationManager;
import com.gitblit.models.UserModel;
import com.gitblit.utils.DeepCopier;
import com.gitblit.utils.StringUtils;
@@ -58,10 +58,18 @@
   protected transient Logger logger = LoggerFactory.getLogger(getClass());
   protected final ISessionManager sessionManager;
   protected IAuthenticationManager authenticationManager;
   protected AuthenticationFilter(ISessionManager sessionManager) {
      this.sessionManager = sessionManager;
   protected AuthenticationFilter(IAuthenticationManager authenticationManager) {
      this.authenticationManager = authenticationManager;
   }
   @Override
   public void init(FilterConfig filterConfig) throws ServletException {
   }
   @Override
   public void destroy() {
   }
   /**
@@ -108,7 +116,7 @@
    * @return user
    */
   protected UserModel getUser(HttpServletRequest httpRequest) {
      UserModel user = sessionManager.authenticate(httpRequest, requiresClientCertificate());
      UserModel user = authenticationManager.authenticate(httpRequest, requiresClientCertificate());
      return user;
   }
@@ -135,20 +143,6 @@
            }
         }
      }
   }
   /**
    * @see javax.servlet.Filter#init(javax.servlet.FilterConfig)
    */
   @Override
   public void init(final FilterConfig config) throws ServletException {
   }
   /**
    * @see javax.servlet.Filter#destroy()
    */
   @Override
   public void destroy() {
   }
   /**
@@ -184,7 +178,7 @@
         // Gitblit does not currently use actual roles in the traditional
         // servlet container sense.  That is the reason this is marked
         // deprecated, but I may want to revisit this.
         return user.canAccessRepository(role);
         return user.hasRepositoryPermission(role);
      }
      @Override
@@ -192,4 +186,4 @@
         return user;
      }
   }
}
}