From 2fca824e349f5fecbf71d940c4521644e92cb0dd Mon Sep 17 00:00:00 2001 From: Paul Martin <paul@paulsputer.com> Date: Wed, 06 Apr 2016 14:49:09 -0400 Subject: [PATCH] Merge pull request #1039 from gitblit/962-Patchset-Revision-Delete --- src/main/java/com/gitblit/servlet/RpcFilter.java | 15 ++------------- 1 files changed, 2 insertions(+), 13 deletions(-) diff --git a/src/main/java/com/gitblit/servlet/RpcFilter.java b/src/main/java/com/gitblit/servlet/RpcFilter.java index 34474d5..355bcb9 100644 --- a/src/main/java/com/gitblit/servlet/RpcFilter.java +++ b/src/main/java/com/gitblit/servlet/RpcFilter.java @@ -128,7 +128,7 @@ return; } else { // check user access for request - if (user.canAdmin() || canAccess(user, requestType)) { + if (user.canAdmin() || !adminRequest) { // authenticated request permitted. // pass processing to the restricted servlet. newSession(authenticatedRequest, httpResponse); @@ -153,15 +153,4 @@ // pass processing to the restricted servlet. chain.doFilter(authenticatedRequest, httpResponse); } - - private boolean canAccess(UserModel user, RpcRequest requestType) { - switch (requestType) { - case GET_PROTOCOL: - return true; - case LIST_REPOSITORIES: - return true; - default: - return user.canAdmin(); - } - } -} \ No newline at end of file +} -- Gitblit v1.9.1