Ticket Reference handling #1048

Paul Martin

2016-04-27 c2188a840bc4153ae92112b04b2e06a90d3944aa

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

@@ -24,6 +24,7 @@
import java.text.MessageFormat;
import java.util.ArrayList;
import java.util.List;
import java.util.Set;

import javax.naming.Context;
import javax.naming.InitialContext;
@@ -44,6 +45,7 @@
import com.gitblit.guice.WebModule;
import com.gitblit.manager.IAuthenticationManager;
import com.gitblit.manager.IFederationManager;
import com.gitblit.manager.IFilestoreManager;
import com.gitblit.manager.IGitblit;
import com.gitblit.manager.IManager;
import com.gitblit.manager.INotificationManager;
@@ -51,7 +53,9 @@
import com.gitblit.manager.IProjectManager;
import com.gitblit.manager.IRepositoryManager;
import com.gitblit.manager.IRuntimeManager;
import com.gitblit.manager.IServicesManager;
import com.gitblit.manager.IUserManager;
import com.gitblit.tickets.ITicketService;
import com.gitblit.transport.ssh.IPublicKeyManager;
import com.gitblit.utils.ContainerUtils;
import com.gitblit.utils.StringUtils;
@@ -166,7 +170,7 @@
         // if the base folder dosen't match the default assume they don't want to use express,
         // this allows for other containers to customise the basefolder per context.
         String defaultBase = Constants.contextFolder$ + "/WEB-INF/data";
         String base = lookupBaseFolderFromJndi();
         String base = getBaseFolderPath(defaultBase);
         if (!StringUtils.isEmpty(System.getenv("OPENSHIFT_DATA_DIR")) && defaultBase.equals(base)) {
            // RedHat OpenShift
            baseFolder = configureExpress(context, webxmlSettings, contextFolder, runtimeSettings);
@@ -199,7 +203,10 @@
      startManager(injector, IRepositoryManager.class);
      startManager(injector, IProjectManager.class);
      startManager(injector, IFederationManager.class);
      startManager(injector, ITicketService.class);
      startManager(injector, IGitblit.class);
      startManager(injector, IServicesManager.class);
      startManager(injector, IFilestoreManager.class);

      // start the plugin manager last so that plugins can depend on
      // deterministic access to all other managers in their start() methods
@@ -231,6 +238,24 @@
      return null;
   }

   protected String getBaseFolderPath(String defaultBaseFolder) {
      // try a system property or a JNDI property
      String specifiedBaseFolder = System.getProperty("GITBLIT_HOME", lookupBaseFolderFromJndi());

      if (!StringUtils.isEmpty(System.getenv("GITBLIT_HOME"))) {
         // try an environment variable
         specifiedBaseFolder = System.getenv("GITBLIT_HOME");
      }

      if (!StringUtils.isEmpty(specifiedBaseFolder)) {
         // use specified base folder path
         return specifiedBaseFolder;
      }

      // use default base folder path
      return defaultBaseFolder;
   }

   protected <X extends IManager> X loadManager(Injector injector, Class<X> clazz) {
      X x = injector.getInstance(clazz);
      return x;
@@ -239,9 +264,13 @@
   protected <X extends IManager> X startManager(Injector injector, Class<X> clazz) {
      X x = loadManager(injector, clazz);
      logManager(clazz);
      x.start();
      managers.add(x);
      return x;
      return startManager(x);
   }

   protected <X extends IManager> X startManager(X x) {
       x.start();
       managers.add(x);
       return x;
   }

   protected void logManager(Class<? extends IManager> clazz) {
@@ -324,9 +353,9 @@
      logger.debug("configuring Gitblit WAR");
      logger.info("WAR contextFolder is " + ((contextFolder != null) ? contextFolder.getAbsolutePath() : "<empty>"));

      String path = webxmlSettings.getString(Constants.baseFolder, Constants.contextFolder$ + "/WEB-INF/data");
      String webXmlPath = webxmlSettings.getString(Constants.baseFolder, Constants.contextFolder$ + "/WEB-INF/data");

      if (path.contains(Constants.contextFolder$) && contextFolder == null) {
      if (webXmlPath.contains(Constants.contextFolder$) && contextFolder == null) {
         // warn about null contextFolder (issue-199)
         logger.error("");
         logger.error(MessageFormat.format("\"{0}\" depends on \"{1}\" but \"{2}\" is returning NULL for \"{1}\"!",
@@ -336,21 +365,16 @@
         logger.error("");
      }

      String baseFromJndi = lookupBaseFolderFromJndi();
      if (!StringUtils.isEmpty(baseFromJndi)) {
         path = baseFromJndi;
      }
      String baseFolderPath = getBaseFolderPath(webXmlPath);

      File base = com.gitblit.utils.FileUtils.resolveParameter(Constants.contextFolder$, contextFolder, path);
      base.mkdirs();
      File baseFolder = com.gitblit.utils.FileUtils.resolveParameter(Constants.contextFolder$, contextFolder, baseFolderPath);
      baseFolder.mkdirs();

      // try to extract the data folder resource to the baseFolder
      File localSettings = new File(base, "gitblit.properties");
      if (!localSettings.exists()) {
         extractResources(context, "/WEB-INF/data/", base);
      }
      extractResources(context, "/WEB-INF/data/", baseFolder);

      // delegate all config to baseFolder/gitblit.properties file
      File localSettings = new File(baseFolder, "gitblit.properties");
      FileSettings fileSettings = new FileSettings(localSettings.getAbsolutePath());

      // merge the stored settings into the runtime settings
@@ -359,7 +383,7 @@
      // the target file for runtimeSettings is set to "localSettings".
      runtimeSettings.merge(fileSettings);

      return base;
      return baseFolder;
   }

   /**
@@ -432,7 +456,12 @@
   }

   protected void extractResources(ServletContext context, String path, File toDir) {
      for (String resource : context.getResourcePaths(path)) {
      Set<String> resources = context.getResourcePaths(path);
      if (resources == null) {
         logger.warn("There are no WAR resources to extract from {}", path);
         return;
      }
      for (String resource : resources) {
         // extract the resource to the directory if it does not exist
         File f = new File(toDir, resource.substring(path.length()));
         if (!f.exists()) {