From e77c633d64c1bd52d3427f6e4fb449b60a3a34c9 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Ren=C3=A9=20Pfeuffer?= Date: Wed, 22 Aug 2018 12:29:45 +0200 Subject: [PATCH] Fix permission check for read permissions --- .../sonia/scm/api/v2/resources/PermissionRootResource.java | 3 +++ 1 file changed, 3 insertions(+) diff --git a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/PermissionRootResource.java b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/PermissionRootResource.java index d08f2e7057..d8e1c56e29 100644 --- a/scm-webapp/src/main/java/sonia/scm/api/v2/resources/PermissionRootResource.java +++ b/scm-webapp/src/main/java/sonia/scm/api/v2/resources/PermissionRootResource.java @@ -14,6 +14,7 @@ import sonia.scm.repository.Repository; import sonia.scm.repository.RepositoryException; import sonia.scm.repository.RepositoryManager; import sonia.scm.repository.RepositoryNotFoundException; +import sonia.scm.repository.RepositoryPermissions; import sonia.scm.web.VndMediaType; import javax.inject.Inject; @@ -95,6 +96,7 @@ public class PermissionRootResource { @Path("{permission-name}") public Response get(@PathParam("namespace") String namespace, @PathParam("name") String name, @PathParam("permission-name") String permissionName) throws RepositoryException { Repository repository = load(namespace, name); + RepositoryPermissions.modify(repository).check(); return Response.ok( repository.getPermissions() .stream() @@ -125,6 +127,7 @@ public class PermissionRootResource { @Path("") public Response getAll(@PathParam("namespace") String namespace, @PathParam("name") String name) throws RepositoryNotFoundException { Repository repository = load(namespace, name); + RepositoryPermissions.modify(repository).check(); List permissionDtoList = repository.getPermissions() .stream() .map(per -> modelToDtoMapper.map(per, new NamespaceAndName(repository.getNamespace(), repository.getName())))