James Moger
2012-09-29 1e1b85270f93b3bca624c99b478f3a9a23be2395
src/com/gitblit/GitFilter.java
@@ -32,12 +32,29 @@
 */
public class GitFilter extends AccessRestrictionFilter {
   protected final String gitReceivePack = "/git-receive-pack";
   protected static final String gitReceivePack = "/git-receive-pack";
   protected final String gitUploadPack = "/git-upload-pack";
   protected static final String gitUploadPack = "/git-upload-pack";
   protected final String[] suffixes = { gitReceivePack, gitUploadPack, "/info/refs", "/HEAD",
   protected static final String[] suffixes = { gitReceivePack, gitUploadPack, "/info/refs", "/HEAD",
         "/objects" };
   /**
    * Extract the repository name from the url.
    *
    * @param url
    * @return repository name
    */
   public static String getRepositoryName(String value) {
      String repository = value;
      // get the repository name from the url by finding a known url suffix
      for (String urlSuffix : suffixes) {
         if (repository.indexOf(urlSuffix) > -1) {
            repository = repository.substring(0, repository.indexOf(urlSuffix));
         }
      }
      return repository;
   }
   /**
    * Extract the repository name from the url.
@@ -47,14 +64,7 @@
    */
   @Override
   protected String extractRepositoryName(String url) {
      String repository = url;
      // get the repository name from the url by finding a known url suffix
      for (String urlSuffix : suffixes) {
         if (repository.indexOf(urlSuffix) > -1) {
            repository = repository.substring(0, repository.indexOf(urlSuffix));
         }
      }
      return repository;
      return GitFilter.getRepositoryName(url);
   }
   /**
@@ -81,16 +91,45 @@
      }
      return null;
   }
   /**
    * Determine if the repository can receive pushes.
    *
    * @param repository
    * @param action
    * @return true if the action may be performed
    */
   @Override
   protected boolean isActionAllowed(RepositoryModel repository, String action) {
      if (!StringUtils.isEmpty(action)) {
         if (action.equals(gitReceivePack)) {
            // Push request
            if (!repository.isBare) {
               logger.warn("Gitblit does not allow pushes to repositories with a working copy");
               return false;
            }
         }
      }
      return true;
   }
   /**
    * Determine if the repository requires authentication.
    * 
    * @param repository
    * @param action
    * @return true if authentication required
    */
   @Override
   protected boolean requiresAuthentication(RepositoryModel repository) {
      return repository.accessRestriction.atLeast(AccessRestrictionType.PUSH);
   protected boolean requiresAuthentication(RepositoryModel repository, String action) {
      if (gitUploadPack.equals(action)) {
         // send to client
         return repository.accessRestriction.atLeast(AccessRestrictionType.CLONE);
      } else if (gitReceivePack.equals(action)) {
         // receive from client
         return repository.accessRestriction.atLeast(AccessRestrictionType.PUSH);
      }
      return false;
   }
   /**
@@ -107,8 +146,8 @@
      if (!GitBlit.getBoolean(Keys.git.enableGitServlet, true)) {
         // Git Servlet disabled
         return false;
      }
      boolean readOnly = repository.isFrozen;
      }
      boolean readOnly = repository.isFrozen;
      if (readOnly || repository.accessRestriction.atLeast(AccessRestrictionType.PUSH)) {
         boolean authorizedUser = user.canAccessRepository(repository);
         if (action.equals(gitReceivePack)) {