Embrace @Inject for Managers, Servlets, and Filters

James Moger

2014-05-16 1b34b05f5b8d2d0fd51819b286e22d7ddd27cb27

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

@@ -16,13 +16,12 @@
package com.gitblit.servlet;



import java.io.IOException;

import java.io.UnsupportedEncodingException;

import java.net.URLDecoder;

import java.security.Principal;

import java.util.Enumeration;

import java.util.HashMap;

import java.util.Map;



import javax.servlet.Filter;

import javax.servlet.FilterChain;

import javax.servlet.FilterConfig;

import javax.servlet.ServletException;

@@ -37,13 +36,10 @@
import org.slf4j.LoggerFactory;



import com.gitblit.Constants;

import com.gitblit.dagger.DaggerFilter;

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

@@ -54,7 +50,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 + "\"";



@@ -64,9 +60,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() {

   }



   /**

@@ -103,12 +106,6 @@
         url = url.substring(1);

      }

      String fullUrl = url + (StringUtils.isEmpty(params) ? "" : ("?" + params));

      try {

         fullUrl = URLDecoder.decode(fullUrl, "UTF-8");

      } catch (UnsupportedEncodingException e) {

         logger.warn("UTF-8 decoding of URL failed: "+fullUrl, e);

         e.printStackTrace();

      }

      return fullUrl;

   }