James Moger
2012-02-25 40538c57dd574d831d044cda50a8999941dc0a24
src/com/gitblit/GitFilter.java
@@ -61,7 +61,7 @@
    * Analyze the url and returns the action of the request. Return values are
    * either "/git-receive-pack" or "/git-upload-pack".
    * 
    * @param url
    * @param serverUrl
    * @return action of the request
    */
   @Override
@@ -75,9 +75,30 @@
            return gitReceivePack;
         } else if (suffix.contains("?service=git-upload-pack")) {
            return gitUploadPack;
         } else {
            return gitUploadPack;
         }
      }
      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 (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;
   }
   /**
@@ -105,12 +126,13 @@
      if (!GitBlit.getBoolean(Keys.git.enableGitServlet, true)) {
         // Git Servlet disabled
         return false;
      }
      if (repository.isFrozen || repository.accessRestriction.atLeast(AccessRestrictionType.PUSH)) {
         boolean authorizedUser = user.canAccessRepository(repository.name);
      }
      boolean readOnly = repository.isFrozen;
      if (readOnly || repository.accessRestriction.atLeast(AccessRestrictionType.PUSH)) {
         boolean authorizedUser = user.canAccessRepository(repository);
         if (action.equals(gitReceivePack)) {
            // Push request
            if (!repository.isFrozen && authorizedUser) {
            if (!readOnly && authorizedUser) {
               // clone-restricted or push-authorized
               return true;
            } else {