Use Guice annotations, not javax.inject annotations

James Moger

2014-06-12 cdb2fe64280a517cf1d13013614bb8ab73cd0ac0

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

@@ -58,10 +58,18 @@


   protected transient Logger logger = LoggerFactory.getLogger(getClass());



   protected final IAuthenticationManager authenticationManager;

   protected IAuthenticationManager authenticationManager;



   protected AuthenticationFilter(IAuthenticationManager authenticationManager) {

      this.authenticationManager = authenticationManager;

   }



   @Override

   public void init(FilterConfig filterConfig) throws ServletException {

   }



   @Override

   public void destroy() {

   }



   /**

@@ -91,8 +99,12 @@
    * @return url

    */

   protected String getFullUrl(HttpServletRequest httpRequest) {

      String servletUrl = httpRequest.getContextPath() + httpRequest.getServletPath();

      String url = httpRequest.getRequestURI().substring(servletUrl.length());

      String contextPath = httpRequest.getContextPath();

      String servletPath = httpRequest.getServletPath();

      String pathInfo = httpRequest.getPathInfo();

      String servletUrl = contextPath + servletPath;

      String requestURI = httpRequest.getRequestURI();

      String url = requestURI.substring(servletUrl.length());

      String params = httpRequest.getQueryString();

      if (url.length() > 0 && url.charAt(0) == '/') {

         url = url.substring(1);

@@ -138,20 +150,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() {

   }



   /**

    * Wraps a standard HttpServletRequest and overrides user principal methods.

    */

   public static class AuthenticatedRequest extends HttpServletRequestWrapper {

@@ -184,7 +182,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 +190,4 @@
         return user;

      }

   }

}
}