From cb946fa57e9dd4ca0853f079331c73dc0331c1e7 Mon Sep 17 00:00:00 2001 From: Florian Zschocke <florian.zschocke@cycos.com> Date: Mon, 26 Aug 2013 06:30:53 -0400 Subject: [PATCH] Refactor logic for user repository path into one class. --- src/main/java/com/gitblit/GitFilter.java | 13 +++---------- 1 files changed, 3 insertions(+), 10 deletions(-) diff --git a/src/main/java/com/gitblit/GitFilter.java b/src/main/java/com/gitblit/GitFilter.java index a0d395b..8c6dd80 100644 --- a/src/main/java/com/gitblit/GitFilter.java +++ b/src/main/java/com/gitblit/GitFilter.java @@ -43,7 +43,7 @@ /** * Extract the repository name from the url. * - * @param url + * @param cloneUrl * @return repository name */ public static String getRepositoryName(String value) { @@ -112,15 +112,8 @@ */ @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; } -- Gitblit v1.9.1