Remove @Inject references because they confuse JEE containers

James Moger

2013-12-10 cc47aad618137ee6481c8bc164d8074bd30fde9e

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

@@ -17,19 +17,18 @@


import java.text.MessageFormat;



import javax.inject.Inject;

import javax.inject.Singleton;

import javax.servlet.http.HttpServletRequest;



import com.gitblit.Constants;

import com.gitblit.Constants.AccessRestrictionType;

import com.gitblit.Constants.AuthorizationControl;

import com.gitblit.GitBlitException;

import com.gitblit.IStoredSettings;

import com.gitblit.Keys;

import com.gitblit.Constants.AccessRestrictionType;

import com.gitblit.Constants.AuthorizationControl;

import com.gitblit.Keys.git;

import com.gitblit.manager.IAuthenticationManager;

import com.gitblit.manager.IFederationManager;

import com.gitblit.manager.IRepositoryManager;

import com.gitblit.manager.IRuntimeManager;

import com.gitblit.manager.IAuthenticationManager;

import com.gitblit.manager.IUserManager;

import com.gitblit.models.RepositoryModel;

import com.gitblit.models.UserModel;

import com.gitblit.utils.StringUtils;

@@ -42,7 +41,6 @@
 * @author James Moger

 *

 */

@Singleton

public class GitFilter extends AccessRestrictionFilter {



   protected static final String gitReceivePack = "/git-receive-pack";

@@ -54,14 +52,21 @@


   private final IStoredSettings settings;



   @Inject

   private final IUserManager userManager;



   private final IFederationManager federationManager;



   public GitFilter(

         IRuntimeManager runtimeManager,

         IUserManager userManager,

         IAuthenticationManager authenticationManager,

         IRepositoryManager repositoryManager) {

         IRepositoryManager repositoryManager,

         IFederationManager federationManager) {



      super(runtimeManager, authenticationManager, repositoryManager);

      this.settings = runtimeManager.getSettings();

      this.userManager = userManager;

      this.federationManager = federationManager;

   }



   /**

@@ -118,6 +123,21 @@
   }



   /**

    * Returns the user making the request, if the user has authenticated.

    *

    * @param httpRequest

    * @return user

    */

   @Override

   protected UserModel getUser(HttpServletRequest httpRequest) {

      UserModel user = authenticationManager.authenticate(httpRequest, requiresClientCertificate());

      if (user == null) {

         user = federationManager.authenticate(httpRequest);

      }

      return user;

   }



   /**

    * Determine if a non-existing repository can be created using this filter.

    *

    * @return true if the server allows repository creation on-push