Merge pull request #1073 from gitblit/1062-DocEditorUpdates

Paul Martin

2016-04-30 a502d96a860456ec5e8c96761db70f7cabb74751

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

@@ -18,7 +18,11 @@
import java.io.IOException;
import java.text.MessageFormat;

import com.google.inject.Inject;
import com.google.inject.Singleton;
import javax.servlet.Filter;
import javax.servlet.FilterChain;
import javax.servlet.FilterConfig;
import javax.servlet.ServletException;
import javax.servlet.ServletRequest;
import javax.servlet.ServletResponse;
@@ -30,11 +34,8 @@

import com.gitblit.IStoredSettings;
import com.gitblit.Keys;
import com.gitblit.dagger.DaggerFilter;
import com.gitblit.manager.IAuthenticationManager;
import com.gitblit.models.UserModel;

import dagger.ObjectGraph;

/**
 * This filter enforces authentication via HTTP Basic Authentication, if the settings indicate so.
@@ -44,7 +45,8 @@
 * @author Laurens Vrijnsen
 *
 */
public class EnforceAuthenticationFilter extends DaggerFilter {
@Singleton
public class EnforceAuthenticationFilter implements Filter {

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

@@ -52,10 +54,21 @@

   private IAuthenticationManager authenticationManager;

   @Inject
   public EnforceAuthenticationFilter(
         IStoredSettings settings,
         IAuthenticationManager authenticationManager) {

      this.settings = settings;
      this.authenticationManager = authenticationManager;
   }

   @Override
   protected void inject(ObjectGraph dagger) {
      this.settings = dagger.get(IStoredSettings.class);
      this.authenticationManager = dagger.get(IAuthenticationManager.class);
   public void init(FilterConfig config) {
   }

   @Override
   public void destroy() {
   }

   /*
@@ -85,13 +98,5 @@
         // user is authenticated, or don't care, continue handling
         chain.doFilter(request, response);
      }
   }


   /*
    * @see javax.servlet.Filter#destroy()
    */
   @Override
   public void destroy() {
   }
}