From 0b953c80b3fb43a4fcd559591067e719ec346d44 Mon Sep 17 00:00:00 2001 From: James Moger <james.moger@gitblit.com> Date: Sat, 01 Mar 2014 09:20:23 -0500 Subject: [PATCH] Move push authorization from the ARF to the receive pack --- src/main/java/com/gitblit/servlet/GitFilter.java | 11 ++--------- 1 files changed, 2 insertions(+), 9 deletions(-) diff --git a/src/main/java/com/gitblit/servlet/GitFilter.java b/src/main/java/com/gitblit/servlet/GitFilter.java index 402f72d..f9c062d 100644 --- a/src/main/java/com/gitblit/servlet/GitFilter.java +++ b/src/main/java/com/gitblit/servlet/GitFilter.java @@ -191,15 +191,8 @@ return false; } if (action.equals(gitReceivePack)) { - // Push request - if (user.canPush(repository)) { - return true; - } else { - // user is unauthorized to push to this repository - logger.warn(MessageFormat.format("user {0} is not authorized to push to {1}", - user.username, repository)); - return false; - } + // push permissions are enforced in the receive pack + return true; } else if (action.equals(gitUploadPack)) { // Clone request if (user.canClone(repository)) { -- Gitblit v1.9.1