bug4811: require valid user (non-anonymous) to ask users with permission; additionally require READ for those USERs returned

This commit is contained in:
Axel Uhl committed 2021-01-25 17:24:20 +01:00
1 parent 79d25e2a89
commit a240c296f3
2 files changed
+5 -2

No files matched your search

@@ -79,4 +79,5 @@ bearerTokenOrAnonymous = com.sap.sse.security.BearerTokenOrBasicOrFormOrAnonymou
/api/restsecurity/role/** = bearerToken
/api/restsecurity/ownership/** = bearerTokenOrAnonymous
/api/restsecurity/has_permission = bearerTokenOrAnonymous
/api/restsecurity/preferences/** = bearerToken
/api/restsecurity/preferences/** = bearerToken
/api/restsecurity/users_with_permission = bearerToken
@@ -63,7 +63,9 @@ public class SecurityResource extends AbstractSecurityResource {
final Iterable<User> usersWithPermission = getService().getUsersWithPermissions(wildcardPermission);
final JSONArray usernames = new JSONArray();
for (final User userWithPermission : usersWithPermission) {
usernames.add(userWithPermission.getName());
if (getService().hasCurrentUserReadPermission(userWithPermission)) {
usernames.add(userWithPermission.getName());
}
}
logger.info("Request for users with permission took "+start.until(TimePoint.now()));
return Response.status(Status.OK).entity(streamingOutput(usernames)).build();