From a2ce62e1f360e9cdb2221cfa3b091c02bda857eb Mon Sep 17 00:00:00 2001 From: Laurens Vrijnsen <laurens.vrijnsen@sioux.eu> Date: Fri, 22 Mar 2013 07:36:52 -0400 Subject: [PATCH] Added enforced HTTP Basic Authentication --- src/com/gitblit/FileUserService.java | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/src/com/gitblit/FileUserService.java b/src/com/gitblit/FileUserService.java index 39c9a5d..32c24cc 100644 --- a/src/com/gitblit/FileUserService.java +++ b/src/com/gitblit/FileUserService.java @@ -20,6 +20,7 @@ import java.io.IOException; import java.text.MessageFormat; import java.util.ArrayList; +import java.util.Collection; import java.util.Collections; import java.util.HashSet; import java.util.List; @@ -270,12 +271,12 @@ /** * Updates/writes all specified user objects. * - * @param model a list of user models + * @param models a list of user models * @return true if update is successful * @since 1.2.0 */ @Override - public boolean updateUserModels(List<UserModel> models) { + public boolean updateUserModels(Collection<UserModel> models) { try { Properties allUsers = read(); for (UserModel model : models) { @@ -796,7 +797,10 @@ repositories.add(role); } } - team.addRepositoryPermissions(repositories); + if (!team.canAdmin) { + // only read permissions for non-admin teams + team.addRepositoryPermissions(repositories); + } team.addUsers(users); team.addMailingLists(mailingLists); team.preReceiveScripts.addAll(preReceive); @@ -994,7 +998,7 @@ * @return true if update is successful * @since 1.2.0 */ - public boolean updateTeamModels(List<TeamModel> models) { + public boolean updateTeamModels(Collection<TeamModel> models) { try { Properties allUsers = read(); for (TeamModel model : models) { -- Gitblit v1.9.1