Authenticate the webapp against the same realm as the git servlet.

James Moger

2011-04-14 fc948cacef9c9b8c0a1e84cbc082ca67cd5f68d9

 src/com/gitblit/GitBlitServer.java

@@ -21,7 +21,7 @@
import org.eclipse.jetty.http.security.Constraint;

import org.eclipse.jetty.security.ConstraintMapping;

import org.eclipse.jetty.security.ConstraintSecurityHandler;

import org.eclipse.jetty.security.HashLoginService;

import org.eclipse.jetty.security.LoginService;

import org.eclipse.jetty.security.authentication.BasicAuthenticator;

import org.eclipse.jetty.server.Connector;

import org.eclipse.jetty.server.Handler;

@@ -40,7 +40,6 @@
import org.eclipse.jetty.webapp.WebAppContext;

import org.eclipse.jgit.http.server.GitServlet;





import com.beust.jcommander.JCommander;

import com.beust.jcommander.Parameter;

import com.beust.jcommander.ParameterException;

@@ -51,11 +50,6 @@


   private final static Logger logger = Log.getLogger(GitBlitServer.class.getSimpleName());

   private final static String border_star = "***********************************************************";

   private static boolean debugMode = false;



   public static boolean isDebugMode() {

      return debugMode;

   }



   public static void main(String[] args) {

      Params params = new Params();

@@ -112,6 +106,9 @@
    * Start Server.

    */

   private static void start(Params params) {

      // instantiate GitBlit

      GitBlit.self();

		
      PatternLayout layout = new PatternLayout(StoredSettings.getString("log4jPattern", "%-5p %d{MM-dd HH:mm:ss.SSS}  %-20.20c{1}  %m%n"));

      org.apache.log4j.Logger rootLogger = org.apache.log4j.Logger.getRootLogger();

      rootLogger.addAppender(new ConsoleAppender(layout));

@@ -124,7 +121,7 @@
      String osversion = System.getProperty("os.version");

      logger.info("Running on " + osname + " (" + osversion + ")");



      if (params.debug) {

      if (StoredSettings.getBoolean("debugMode", false)) {

         logger.warn("DEBUG Mode");

      }



@@ -134,7 +131,7 @@
         Connector httpConnector = createConnector(params.useNIO, params.port);

         connectors.add(httpConnector);

      }

		


      if (params.securePort > 0) {

         if (new File("keystore").exists()) {

            Connector secureConnector = createSSLConnector(params.useNIO, params.securePort, params.storePassword);

@@ -174,53 +171,64 @@
      FilterHolder wicketFilter = new FilterHolder(WicketFilter.class);

      wicketFilter.setInitParameter(ContextParamWebApplicationFactory.APP_CLASS_PARAM, GitBlitWebApp.class.getName());

      wicketFilter.setInitParameter(WicketFilter.FILTER_MAPPING_PARAM, wicketPathSpec);

      wicketFilter.setInitParameter(WicketFilter.IGNORE_PATHS_PARAM, "git/");

      rootContext.addFilter(wicketFilter, wicketPathSpec, FilterMapping.DEFAULT);



      // GIT Servlet

      // Git Servlet

      ServletHolder gitServlet = null;

      String gitServletPathSpec = "/git/*";

      ServletHolder gitServlet = rootContext.addServlet(GitServlet.class, gitServletPathSpec);

      gitServlet.setInitParameter("base-path", params.repositoriesFolder);

      gitServlet.setInitParameter("export-all", params.exportAll ? "1" : "0");



      if (StoredSettings.getBoolean("allowPushPull", true)) {

         gitServlet = rootContext.addServlet(GitServlet.class, gitServletPathSpec);

         gitServlet.setInitParameter("base-path", params.repositoriesFolder);

         gitServlet.setInitParameter("export-all", params.exportAll ? "1" : "0");

      }

		
      // Login Service

      LoginService loginService = null;

      String realmUsers = params.realmFile;



      // Authentication Realm

      if (realmUsers != null && new File(realmUsers).exists()) {

         logger.info("Setting up login service from " + realmUsers);

         JettyLoginService jettyLoginService = new JettyLoginService(realmUsers);

         GitBlit.self().setLoginService(jettyLoginService);

         loginService = jettyLoginService;

      }

		
      // Determine what handler to use

      Handler handler;

      if (realmUsers != null && new File(realmUsers).exists() && params.authenticateAccess) {

         List<String> list = StoredSettings.getStrings("gitRoles");

         String[] roles;

         if (list.size() == 0) {

            roles = new String[] { "*" };

      if (gitServlet != null) {

         if (loginService != null && params.authenticatePushPull) {

            // Authenticate Pull/Push

            String[] roles = new String[] { Constants.PULL_ROLE, Constants.PUSH_ROLE };

            logger.info("Authentication required for git servlet pull/push access");



            Constraint constraint = new Constraint();

            constraint.setName("auth");

            constraint.setAuthenticate(true);

            constraint.setRoles(roles);



            ConstraintMapping mapping = new ConstraintMapping();

            mapping.setPathSpec(gitServletPathSpec);

            mapping.setConstraint(constraint);



            ConstraintSecurityHandler security = new ConstraintSecurityHandler();

            security.addConstraintMapping(mapping);

            for (String role : roles) {

               security.addRole(role);

            }

            security.setAuthenticator(new BasicAuthenticator());

            security.setLoginService(loginService);

            security.setStrict(false);



            security.setHandler(rootContext);



            handler = security;

         } else {

            roles = list.toArray(new String[list.size()]);

            // Anonymous Pull/Push

            logger.info("Setting up anonymous git servlet pull/push access");

            handler = rootContext;

         }

         logger.info("Authentication required for GIT access");

         logger.info("Setting up realm from " + realmUsers);

         HashLoginService loginService = new HashLoginService(Constants.NAME, realmUsers);



         Constraint constraint = new Constraint();

         constraint.setName("auth");

         constraint.setAuthenticate(true);

         constraint.setRoles(roles);



         ConstraintMapping mapping = new ConstraintMapping();

         mapping.setPathSpec(gitServletPathSpec);

         mapping.setConstraint(constraint);



         ConstraintSecurityHandler security = new ConstraintSecurityHandler();

         security.addConstraintMapping(mapping);

         for (String role : roles) {

            security.addRole(role);

         }

         security.setAuthenticator(new BasicAuthenticator());

         security.setLoginService(loginService);

         security.setStrict(false);



         security.setHandler(rootContext);



         handler = security;

      } else {

         logger.info("Setting up anonymous access");

         logger.info("Git servlet pull/push disabled");

         handler = rootContext;

      }



@@ -350,13 +358,10 @@
      @Parameter(names = { "--temp" }, description = "Server temp folder")

      public String temp = StoredSettings.getString("tempFolder", "temp");



      @Parameter(names = { "--debug" }, description = "Run server in DEBUG mode")

      public Boolean debug = StoredSettings.getBoolean("debug", false);



      /*

       * GIT Servlet Parameters

       */

      @Parameter(names = { "--repos" }, description = "GIT Repositories Folder")

      @Parameter(names = { "--repos" }, description = "Git Repositories Folder")

      public String repositoriesFolder = StoredSettings.getString("repositoriesFolder", "repos");



      @Parameter(names = { "--exportAll" }, description = "Export All Found Repositories")

@@ -365,8 +370,8 @@
      /*

       * Authentication Parameters

       */

      @Parameter(names = { "--authenticateAccess" }, description = "Authenticate GIT access")

      public Boolean authenticateAccess = StoredSettings.getBoolean("authenticateAccess", true);

      @Parameter(names = { "--authenticatePushPull" }, description = "Authenticate Git Push/Pull access")

      public Boolean authenticatePushPull = StoredSettings.getBoolean("authenticatePushPull", true);



      @Parameter(names = { "--realm" }, description = "Users Realm Hash File")

      public String realmFile = StoredSettings.getString("realmFile", "users.properties");