Do not grant fork CLONE permissions to users/teams with implied regex CLONE...

James Moger

2014-02-26 ba2f9aa95ee55f3672cd59474c65b959d0fe7fb5

 src/main/java/com/gitblit/git/GitblitReceivePackFactory.java

@@ -26,9 +26,9 @@
import org.slf4j.Logger;
import org.slf4j.LoggerFactory;

import com.gitblit.Gitblit;
import com.gitblit.IStoredSettings;
import com.gitblit.Keys;
import com.gitblit.manager.IGitblit;
import com.gitblit.models.RepositoryModel;
import com.gitblit.models.UserModel;
import com.gitblit.utils.HttpUtils;
@@ -47,9 +47,9 @@

   private final IStoredSettings settings;

   private final Gitblit gitblit;
   private final IGitblit gitblit;

   public GitblitReceivePackFactory(Gitblit gitblit) {
   public GitblitReceivePackFactory(IGitblit gitblit) {
      super();
      this.settings = gitblit.getSettings();
      this.gitblit = gitblit;
@@ -63,7 +63,6 @@
      String repositoryName = "";
      String origin = "";
      String gitblitUrl = "";
      String repositoryUrl = "";
      int timeout = 0;

      if (req instanceof HttpServletRequest) {
@@ -72,7 +71,6 @@
         repositoryName = request.getAttribute("gitblitRepositoryName").toString();
         origin = request.getRemoteHost();
         gitblitUrl = HttpUtils.getGitblitURL(request);
         repositoryUrl = request.getRequestURI();

         // determine pushing user
         String username = request.getRemoteUser();
@@ -98,11 +96,15 @@
         throw new ServiceNotEnabledException();
      }

      String url = settings.getString(Keys.web.canonicalUrl, null);
      if (StringUtils.isEmpty(url)) {
         url = gitblitUrl;
      }
		
      final RepositoryModel repository = gitblit.getRepositoryModel(repositoryName);

      final GitblitReceivePack rp = new GitblitReceivePack(gitblit, db, repository, user);
      rp.setGitblitUrl(gitblitUrl);
      rp.setRepositoryUrl(repositoryUrl);
      rp.setGitblitUrl(url);
      rp.setRefLogIdent(new PersonIdent(user.username, user.username + "@" + origin));
      rp.setTimeout(timeout);