James Moger
2013-10-15 e1908933fa9968b11e9ea03d88730aee3140534b
src/main/java/com/gitblit/GitFilter.java
@@ -27,9 +27,9 @@
 * The GitFilter is an AccessRestrictionFilter which ensures that Git client
 * requests for push, clone, or view restricted repositories are authenticated
 * and authorized.
 *
 *
 * @author James Moger
 *
 *
 */
public class GitFilter extends AccessRestrictionFilter {
@@ -42,8 +42,8 @@
   /**
    * Extract the repository name from the url.
    *
    * @param url
    *
    * @param cloneUrl
    * @return repository name
    */
   public static String getRepositoryName(String value) {
@@ -59,7 +59,7 @@
   /**
    * Extract the repository name from the url.
    *
    *
    * @param url
    * @return repository name
    */
@@ -71,7 +71,7 @@
   /**
    * Analyze the url and returns the action of the request. Return values are
    * either "/git-receive-pack" or "/git-upload-pack".
    *
    *
    * @param serverUrl
    * @return action of the request
    */
@@ -92,35 +92,28 @@
      }
      return null;
   }
   /**
    * Determine if a non-existing repository can be created using this filter.
    *
    *
    * @return true if the server allows repository creation on-push
    */
   @Override
   protected boolean isCreationAllowed() {
      return GitBlit.getBoolean(Keys.git.allowCreateOnPush, true);
   }
   /**
    * 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;
            }
         }
      }
      // the log here has been moved into ReceiveHook to provide clients with
      // error messages
      return true;
   }
@@ -131,7 +124,7 @@
   /**
    * Determine if the repository requires authentication.
    *
    *
    * @param repository
    * @param action
    * @return true if authentication required
@@ -140,7 +133,7 @@
   protected boolean requiresAuthentication(RepositoryModel repository, String action) {
      if (gitUploadPack.equals(action)) {
         // send to client
         return repository.accessRestriction.atLeast(AccessRestrictionType.CLONE);
         return repository.accessRestriction.atLeast(AccessRestrictionType.CLONE);
      } else if (gitReceivePack.equals(action)) {
         // receive from client
         return repository.accessRestriction.atLeast(AccessRestrictionType.PUSH);
@@ -151,7 +144,7 @@
   /**
    * Determine if the user can access the repository and perform the specified
    * action.
    *
    *
    * @param repository
    * @param user
    * @param action
@@ -162,7 +155,7 @@
      if (!GitBlit.getBoolean(Keys.git.enableGitServlet, true)) {
         // Git Servlet disabled
         return false;
      }
      }
      if (action.equals(gitReceivePack)) {
         // Push request
         if (user.canPush(repository)) {
@@ -186,11 +179,11 @@
      }
      return true;
   }
   /**
    * An authenticated user with the CREATE role can create a repository on
    * push.
    *
    *
    * @param user
    * @param repository
    * @param action
@@ -210,7 +203,7 @@
            if (repository.contains("/../")) {
               logger.error(MessageFormat.format("Illegal relative path in repository name! {0}", repository));
               return null;
            }
            }
            // confirm valid characters in repository name
            Character c = StringUtils.findInvalidCharacter(repository);
@@ -231,7 +224,7 @@
            } else {
               // common repository, user default server settings
               model.authorizationControl = AuthorizationControl.fromName(GitBlit.getString(Keys.git.defaultAuthorizationControl, ""));
               model.accessRestriction = AccessRestrictionType.fromName(GitBlit.getString(Keys.git.defaultAccessRestriction, ""));
               model.accessRestriction = AccessRestrictionType.fromName(GitBlit.getString(Keys.git.defaultAccessRestriction, "PUSH"));
            }
            // create the repository
@@ -246,7 +239,7 @@
            logger.warn(MessageFormat.format("{0} is not permitted to create repository {1} ON-PUSH!", user.username, repository));
         }
      }
      // repository could not be created or action was not a push
      return null;
   }