Added AccessRestrictionFilter and simplified authentication.

James Moger

2011-06-14 8c9a2037b5c0fed881a3ad6dd9cff364eed603d9

 src/com/gitblit/GitBlit.java

@@ -20,7 +20,10 @@
import java.text.MessageFormat;

import java.util.ArrayList;

import java.util.Collections;

import java.util.HashMap;

import java.util.List;

import java.util.Map;

import java.util.Map.Entry;



import javax.servlet.ServletContextEvent;

import javax.servlet.ServletContextListener;

@@ -87,8 +90,8 @@
      return GITBLIT.storedSettings.getAllKeys(startingWith);

   }



   public boolean isDebugMode() {

      return storedSettings.getBoolean(Keys.web.debugMode, false);

   public static boolean isDebugMode() {

      return GITBLIT.storedSettings.getBoolean(Keys.web.debugMode, false);

   }



   public List<String> getOtherCloneUrls(String repositoryName) {

@@ -142,7 +145,7 @@


   public List<String> getRepositoryList() {

      return JGitUtils.getRepositoryList(repositoriesFolder, exportAll,

            storedSettings.getBoolean(Keys.git.nestedRepositories, true));

            storedSettings.getBoolean(Keys.git.searchRepositoriesSubfolders, true));

   }



   public Repository getRepository(String repositoryName) {

@@ -203,6 +206,7 @@
               "accessRestriction", null));

         model.showRemoteBranches = getConfig(config, "showRemoteBranches", false);

         model.isFrozen = getConfig(config, "isFrozen", false);

         model.showReadme = getConfig(config, "showReadme", false);

      }

      r.close();

      return model;

@@ -224,6 +228,10 @@
         boolean isCreate) throws GitBlitException {

      Repository r = null;

      if (isCreate) {

         // ensure created repository name ends with .git

         if (!repository.name.toLowerCase().endsWith(org.eclipse.jgit.lib.Constants.DOT_GIT_EXT)) {

            repository.name += org.eclipse.jgit.lib.Constants.DOT_GIT_EXT;

         }

         if (new File(repositoriesFolder, repository.name).exists()) {

            throw new GitBlitException(MessageFormat.format(

                  "Can not create repository ''{0}'' because it already exists.",

@@ -231,7 +239,7 @@
         }

         // create repository

         logger.info("create repository " + repository.name);

         r = JGitUtils.createRepository(repositoriesFolder, repository.name, true);

         r = JGitUtils.createRepository(repositoriesFolder, repository.name);

      } else {

         // rename repository

         if (!repositoryName.equalsIgnoreCase(repository.name)) {

@@ -278,6 +286,7 @@
               repository.accessRestriction.name());

         config.setBoolean("gitblit", null, "showRemoteBranches", repository.showRemoteBranches);

         config.setBoolean("gitblit", null, "isFrozen", repository.isFrozen);

         config.setBoolean("gitblit", null, "showReadme", repository.showReadme);

         try {

            config.save();

         } catch (IOException e) {

@@ -306,15 +315,39 @@
      return false;

   }



   public boolean renameRepository(RepositoryModel model, String newName) {

      File folder = new File(repositoriesFolder, model.name);

      if (folder.exists() && folder.isDirectory()) {

         File newFolder = new File(repositoriesFolder, newName);

         if (folder.renameTo(newFolder)) {

            return loginService.renameRole(model.name, newName);

   public String processCommitMessage(String repositoryName, String text) {

      String html = StringUtils.breakLinesForHtml(text);

      Map<String, String> map = new HashMap<String, String>();

      // global regex keys

      if (storedSettings.getBoolean(Keys.regex.global, false)) {

         for (String key : storedSettings.getAllKeys(Keys.regex.global)) {

            if (!key.equals(Keys.regex.global)) {

               String subKey = key.substring(key.lastIndexOf('.') + 1);

               map.put(subKey, storedSettings.getString(key, ""));

            }

         }

      }

      return false;



      // repository-specific regex keys

      List<String> keys = storedSettings.getAllKeys(Keys.regex._ROOT + "."

            + repositoryName.toLowerCase());

      for (String key : keys) {

         String subKey = key.substring(key.lastIndexOf('.') + 1);

         map.put(subKey, storedSettings.getString(key, ""));

      }



      for (Entry<String, String> entry : map.entrySet()) {

         String definition = entry.getValue().trim();

         String[] chunks = definition.split("!!!");

         if (chunks.length == 2) {

            html = html.replaceAll(chunks[0], chunks[1]);

         } else {

            logger.warn(entry.getKey()

                  + " improperly formatted.  Use !!! to separate match from replacement: "

                  + definition);

         }

      }

      return html;

   }



   public void configureContext(IStoredSettings settings) {

@@ -328,6 +361,8 @@
   @Override

   public void contextInitialized(ServletContextEvent contextEvent) {

      if (storedSettings == null) {

         // for running gitblit as a traditional webapp in a servlet

         // container

         WebXmlSettings webxmlSettings = new WebXmlSettings(contextEvent.getServletContext());

         configureContext(webxmlSettings);

      }

@@ -335,6 +370,6 @@


   @Override

   public void contextDestroyed(ServletContextEvent contextEvent) {

      logger.info("GitBlit context destroyed by servlet container.");

      logger.info("Gitblit context destroyed by servlet container.");

   }

}