This is an automated email from the ASF dual-hosted git repository.
enorman pushed a commit to branch master
in repository
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-jcr-jackrabbit-usermanager.git
The following commit(s) were added to refs/heads/master by this push:
new e92e8a8 SLING-12203 Remove dependency on org.apache.sling.jcr.base
(#25)
e92e8a8 is described below
commit e92e8a8d89fafbd1f873c32b4851bf878a3457d3
Author: Eric Norman <[email protected]>
AuthorDate: Mon Dec 18 17:21:40 2023 -0800
SLING-12203 Remove dependency on org.apache.sling.jcr.base (#25)
---
pom.xml | 6 ------
.../usermanager/impl/AuthorizablePrivilegesInfoImpl.java | 10 +++++-----
.../impl/post/AbstractAuthorizablePostServlet.java | 4 ++--
.../usermanager/impl/post/AbstractGroupPostServlet.java | 4 ++--
.../usermanager/impl/post/ChangeUserPasswordServlet.java | 8 ++++----
.../jackrabbit/usermanager/impl/post/CreateGroupServlet.java | 4 ++--
.../jackrabbit/usermanager/impl/post/CreateUserServlet.java | 8 ++++----
.../usermanager/impl/post/DeleteAuthorizableServlet.java | 6 +++---
.../jackrabbit/usermanager/impl/post/UpdateGroupServlet.java | 4 ++--
.../jackrabbit/usermanager/impl/post/UpdateUserServlet.java | 4 ++--
.../impl/resource/AuthorizableResourceProvider.java | 12 ++++++------
.../usermanager/it/AuthorizablePrivilegesInfoIT.java | 3 +--
12 files changed, 33 insertions(+), 40 deletions(-)
diff --git a/pom.xml b/pom.xml
index 11a26d6..3fdbf6b 100644
--- a/pom.xml
+++ b/pom.xml
@@ -150,12 +150,6 @@
<version>2.3.0</version>
<scope>provided</scope>
</dependency>
- <dependency>
- <groupId>org.apache.sling</groupId>
- <artifactId>org.apache.sling.jcr.base</artifactId>
- <version>3.1.12</version>
- <scope>provided</scope>
- </dependency>
<dependency>
<groupId>javax.jcr</groupId>
<artifactId>jcr</artifactId>
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/AuthorizablePrivilegesInfoImpl.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/AuthorizablePrivilegesInfoImpl.java
index 0b82fc8..947e2d4 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/AuthorizablePrivilegesInfoImpl.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/AuthorizablePrivilegesInfoImpl.java
@@ -25,6 +25,7 @@ import javax.jcr.Session;
import javax.jcr.security.AccessControlManager;
import javax.jcr.security.Privilege;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.User;
@@ -36,7 +37,6 @@ import org.apache.sling.commons.osgi.OsgiUtil;
import org.apache.sling.jackrabbit.usermanager.AuthorizablePrivilegesInfo;
import org.apache.sling.jackrabbit.usermanager.ChangeUserPassword;
import org.apache.sling.jackrabbit.usermanager.CreateUser;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.jetbrains.annotations.NotNull;
import org.osgi.framework.BundleContext;
import org.osgi.service.component.annotations.Activate;
@@ -144,7 +144,7 @@ public class AuthorizablePrivilegesInfoImpl implements
AuthorizablePrivilegesInf
public boolean canAddGroup(Session jcrSession) {
boolean hasRights = false;
try {
- UserManager userManager =
AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable currentUser =
userManager.getAuthorizable(jcrSession.getUserID());
if (currentUser instanceof User && ((User)currentUser).isAdmin()) {
@@ -178,7 +178,7 @@ public class AuthorizablePrivilegesInfoImpl implements
AuthorizablePrivilegesInf
if (selfRegistrationEnabled) {
hasRights = true;
} else {
- UserManager userManager =
AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable currentUser =
userManager.getAuthorizable(jcrSession.getUserID());
if (currentUser instanceof User &&
((User)currentUser).isAdmin()) {
hasRights = true; //admin user has full control
@@ -205,7 +205,7 @@ public class AuthorizablePrivilegesInfoImpl implements
AuthorizablePrivilegesInf
protected boolean checkAuthorizablePath(Session jcrSession, String
principalId,
AuthorizableChecker authorizableChecker, AccessChecker
accessChecker) throws RepositoryException {
boolean hasRights = false;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable currentUser =
userManager.getAuthorizable(jcrSession.getUserID());
Authorizable authorizable = userManager.getAuthorizable(principalId);
@@ -394,7 +394,7 @@ public class AuthorizablePrivilegesInfoImpl implements
AuthorizablePrivilegesInf
try {
// can't change your own password without the old password
if (!jcrSession.getUserID().equals(userId)) {
- UserManager um = AccessControlUtil.getUserManager(jcrSession);
+ UserManager um =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable currentUser =
um.getAuthorizable(jcrSession.getUserID());
if (currentUser instanceof User) {
Authorizable targetUser = um.getAuthorizable(userId);
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractAuthorizablePostServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractAuthorizablePostServlet.java
index 59debea..37b7cc3 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractAuthorizablePostServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractAuthorizablePostServlet.java
@@ -39,6 +39,7 @@ import javax.jcr.nodetype.NodeType;
import javax.jcr.nodetype.PropertyDefinition;
import org.apache.jackrabbit.JcrConstants;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.UserManager;
import org.apache.jackrabbit.oak.spi.security.user.AuthorizableType;
@@ -49,7 +50,6 @@ import
org.apache.sling.jackrabbit.usermanager.PrincipalNameFilter;
import org.apache.sling.jackrabbit.usermanager.PrincipalNameGenerator;
import org.apache.sling.jackrabbit.usermanager.PrincipalNameGenerator.NameInfo;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.SlingPostConstants;
import org.apache.sling.servlets.post.impl.helper.DateParser;
@@ -168,7 +168,7 @@ public abstract class AbstractAuthorizablePostServlet
extends
principalName = nameInfo.getPrincipalName();
if (principalName != null && nameInfo.isMakeUnique()) {
// make sure the name is not already used
- UserManager um =
AccessControlUtil.getUserManager(jcrSession);
+ UserManager um =
((JackrabbitSession)jcrSession).getUserManager();
// if resulting authorizable exists, add a random suffix
until it's not the case
// anymore
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractGroupPostServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractGroupPostServlet.java
index d7d1ffd..e599a34 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractGroupPostServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/AbstractGroupPostServlet.java
@@ -22,12 +22,12 @@ import java.util.Map;
import javax.jcr.RepositoryException;
import javax.jcr.Session;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.UserManager;
import org.apache.sling.api.resource.Resource;
import org.apache.sling.api.resource.ResourceResolver;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.SlingPostConstants;
@@ -62,7 +62,7 @@ public abstract class AbstractGroupPostServlet extends
ResourceResolver resolver = baseResource.getResourceResolver();
boolean changed = false;
- UserManager userManager =
AccessControlUtil.getUserManager(resolver.adaptTo(Session.class));
+ UserManager userManager =
((JackrabbitSession)resolver.adaptTo(Session.class)).getUserManager();
// first remove any members posted as ":member@Delete"
String[] membersToDelete =
convertToStringArray(properties.get(SlingPostConstants.RP_PREFIX
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/ChangeUserPasswordServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/ChangeUserPasswordServlet.java
index 743b161..026d76f 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/ChangeUserPasswordServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/ChangeUserPasswordServlet.java
@@ -25,6 +25,7 @@ import javax.jcr.security.AccessControlManager;
import javax.jcr.security.Privilege;
import javax.servlet.Servlet;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.User;
@@ -37,7 +38,6 @@ import org.apache.sling.commons.osgi.OsgiUtil;
import org.apache.sling.jackrabbit.usermanager.ChangeUserPassword;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
import org.apache.sling.jcr.api.SlingRepository;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.serviceusermapping.ServiceUserMapped;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.PostResponse;
@@ -258,7 +258,7 @@ public class ChangeUserPasswordServlet extends
AbstractAuthorizablePostServlet i
}
User user;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(name);
if (authorizable instanceof User) {
user = (User)authorizable;
@@ -274,7 +274,7 @@ public class ChangeUserPasswordServlet extends
AbstractAuthorizablePostServlet i
// check that the submitted parameter values have valid values.
if (oldPassword == null || oldPassword.length() == 0) {
try {
- UserManager um = AccessControlUtil.getUserManager(jcrSession);
+ UserManager um =
((JackrabbitSession)jcrSession).getUserManager();
User currentUser = (User)
um.getAuthorizable(jcrSession.getUserID());
administrator = currentUser.isAdmin();
@@ -324,7 +324,7 @@ public class ChangeUserPasswordServlet extends
AbstractAuthorizablePostServlet i
Session svcSession = null;
try {
svcSession = repository.loginService(null, null);
- UserManager um =
AccessControlUtil.getUserManager(svcSession);
+ UserManager um =
((JackrabbitSession)svcSession).getUserManager();
User user2 = (User) um.getAuthorizable(name);
user2.changePassword(newPassword, oldPassword);
if (svcSession.hasPendingChanges()) {
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateGroupServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateGroupServlet.java
index 2689328..0f7853a 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateGroupServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateGroupServlet.java
@@ -25,6 +25,7 @@ import javax.jcr.RepositoryException;
import javax.jcr.Session;
import javax.servlet.Servlet;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.UserManager;
@@ -38,7 +39,6 @@ import org.apache.sling.jackrabbit.usermanager.CreateGroup;
import org.apache.sling.jackrabbit.usermanager.PrincipalNameFilter;
import org.apache.sling.jackrabbit.usermanager.PrincipalNameGenerator;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.PostResponse;
import org.apache.sling.servlets.post.PostResponseCreator;
@@ -235,7 +235,7 @@ public class CreateGroupServlet extends
AbstractGroupPostServlet implements Crea
throw new IllegalArgumentException("Group name was not supplied");
}
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(principalName);
Group group = null;
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateUserServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateUserServlet.java
index d90de67..a5b4f97 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateUserServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/CreateUserServlet.java
@@ -26,6 +26,7 @@ import javax.jcr.security.AccessControlManager;
import javax.jcr.security.Privilege;
import javax.servlet.Servlet;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.User;
import org.apache.jackrabbit.api.security.user.UserManager;
@@ -39,7 +40,6 @@ import
org.apache.sling.jackrabbit.usermanager.PrincipalNameFilter;
import org.apache.sling.jackrabbit.usermanager.PrincipalNameGenerator;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
import org.apache.sling.jcr.api.SlingRepository;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.serviceusermapping.ServiceUserMapped;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.ModificationType;
@@ -321,7 +321,7 @@ public class CreateUserServlet extends
AbstractAuthorizablePostServlet implement
// check for an administrator
boolean administrator = false;
try {
- UserManager um = AccessControlUtil.getUserManager(jcrSession);
+ UserManager um = ((JackrabbitSession)jcrSession).getUserManager();
User currentUser = (User)
um.getAuthorizable(jcrSession.getUserID());
administrator = currentUser.isAdmin();
@@ -372,7 +372,7 @@ public class CreateUserServlet extends
AbstractAuthorizablePostServlet implement
selfRegSession = getSession();
}
- UserManager userManager =
AccessControlUtil.getUserManager(selfRegSession);
+ UserManager userManager =
((JackrabbitSession)selfRegSession).getUserManager();
Authorizable authorizable =
userManager.getAuthorizable(principalName);
if (authorizable != null) {
@@ -402,7 +402,7 @@ public class CreateUserServlet extends
AbstractAuthorizablePostServlet implement
if (useAdminSession) {
//lookup the user from the user session so we can return a
live object
- UserManager userManager2 =
AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager2 =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable2 =
userManager2.getAuthorizable(user.getID());
if (authorizable2 instanceof User) {
user = (User)authorizable2;
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/DeleteAuthorizableServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/DeleteAuthorizableServlet.java
index baf390c..990f4a4 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/DeleteAuthorizableServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/DeleteAuthorizableServlet.java
@@ -26,6 +26,7 @@ import javax.jcr.Session;
import javax.servlet.Servlet;
import javax.servlet.http.HttpServletResponse;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.User;
@@ -38,7 +39,6 @@ import
org.apache.sling.jackrabbit.usermanager.DeleteAuthorizables;
import org.apache.sling.jackrabbit.usermanager.DeleteGroup;
import org.apache.sling.jackrabbit.usermanager.DeleteUser;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.PostResponse;
import org.apache.sling.servlets.post.PostResponseCreator;
@@ -163,7 +163,7 @@ public class DeleteAuthorizableServlet extends
AbstractPostServlet
List<Modification> changes) throws RepositoryException {
User user;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(name);
if (authorizable instanceof User) {
user = (User)authorizable;
@@ -186,7 +186,7 @@ public class DeleteAuthorizableServlet extends
AbstractPostServlet
List<Modification> changes) throws
RepositoryException {
Group group;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(name);
if (authorizable instanceof Group) {
group = (Group)authorizable;
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateGroupServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateGroupServlet.java
index 59b6f3e..df2bd41 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateGroupServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateGroupServlet.java
@@ -25,6 +25,7 @@ import javax.jcr.RepositoryException;
import javax.jcr.Session;
import javax.servlet.Servlet;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.Group;
import org.apache.jackrabbit.api.security.user.UserManager;
@@ -36,7 +37,6 @@ import org.apache.sling.api.resource.ResourceResolver;
import org.apache.sling.api.resource.ResourceResolverFactory;
import org.apache.sling.jackrabbit.usermanager.UpdateGroup;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.jcr.resource.api.JcrResourceConstants;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.PostResponse;
@@ -180,7 +180,7 @@ public class UpdateGroupServlet extends
AbstractGroupPostServlet
throws RepositoryException {
Group group = null;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(name);
if (authorizable instanceof Group) {
group = (Group)authorizable;
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateUserServlet.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateUserServlet.java
index 1233b9f..8cc3cd1 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateUserServlet.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/post/UpdateUserServlet.java
@@ -24,6 +24,7 @@ import javax.jcr.RepositoryException;
import javax.jcr.Session;
import javax.servlet.Servlet;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.user.Authorizable;
import org.apache.jackrabbit.api.security.user.User;
import org.apache.jackrabbit.api.security.user.UserManager;
@@ -32,7 +33,6 @@ import org.apache.sling.api.resource.Resource;
import org.apache.sling.api.resource.ResourceNotFoundException;
import org.apache.sling.jackrabbit.usermanager.UpdateUser;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.servlets.post.Modification;
import org.apache.sling.servlets.post.PostResponse;
import org.apache.sling.servlets.post.PostResponseCreator;
@@ -170,7 +170,7 @@ public class UpdateUserServlet extends
AbstractAuthorizablePostServlet
throws RepositoryException {
User user;
- UserManager userManager = AccessControlUtil.getUserManager(jcrSession);
+ UserManager userManager =
((JackrabbitSession)jcrSession).getUserManager();
Authorizable authorizable = userManager.getAuthorizable(name);
if (authorizable instanceof User) {
user = (User)authorizable;
diff --git
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
index 13207ae..7b95951 100644
---
a/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
+++
b/src/main/java/org/apache/sling/jackrabbit/usermanager/impl/resource/AuthorizableResourceProvider.java
@@ -26,6 +26,7 @@ import java.util.NoSuchElementException;
import javax.jcr.RepositoryException;
import javax.jcr.Session;
+import org.apache.jackrabbit.api.JackrabbitSession;
import org.apache.jackrabbit.api.security.principal.GroupPrincipal;
import org.apache.jackrabbit.api.security.principal.PrincipalIterator;
import org.apache.jackrabbit.api.security.principal.PrincipalManager;
@@ -39,7 +40,6 @@ import org.apache.sling.api.resource.ResourceResolver;
import org.apache.sling.api.resource.SyntheticResource;
import org.apache.sling.commons.osgi.OsgiUtil;
import org.apache.sling.jackrabbit.usermanager.resource.SystemUserManagerPaths;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.spi.resource.provider.ResolveContext;
import org.apache.sling.spi.resource.provider.ResourceContext;
import org.apache.sling.spi.resource.provider.ResourceProvider;
@@ -253,7 +253,7 @@ public class AuthorizableResourceProvider extends
ResourceProvider<Object> imple
Session session = ctx.getResourceResolver().adaptTo(Session.class);
if (session != null) {
try {
- UserManager userManager =
AccessControlUtil.getUserManager(session);
+ UserManager userManager =
((JackrabbitSession)session).getUserManager();
if (userManager != null) {
Authorizable authorizable =
userManager.getAuthorizable(pid);
if (authorizable != null) {
@@ -262,7 +262,7 @@ public class AuthorizableResourceProvider extends
ResourceProvider<Object> imple
}
} else if (principalWorker != null && relPath == null){
// SLING-11098 check for a principal that is not
an authorizable like the everyone group
- PrincipalManager principalManager =
AccessControlUtil.getPrincipalManager(session);
+ PrincipalManager principalManager =
((JackrabbitSession)session).getPrincipalManager();
if (principalManager != null) {
@Nullable
Principal principal =
principalManager.getPrincipal(pid);
@@ -324,7 +324,7 @@ public class AuthorizableResourceProvider extends
ResourceProvider<Object> imple
ResourceResolver resourceResolver =
parent.getResourceResolver();
Session session = resourceResolver.adaptTo(Session.class);
if (session != null) {
- PrincipalManager principalManager =
AccessControlUtil.getPrincipalManager(session);
+ PrincipalManager principalManager =
((JackrabbitSession)session).getPrincipalManager();
principals = principalManager.getPrincipals(searchType);
}
@@ -428,7 +428,7 @@ public class AuthorizableResourceProvider extends
ResourceProvider<Object> imple
protected @Nullable Resource createNext(Object child, String
principalName,
ResourceResolver resourceResolver, Session session) throws
RepositoryException {
Resource next = null;
- UserManager userManager =
AccessControlUtil.getUserManager(session);
+ UserManager userManager =
((JackrabbitSession)session).getUserManager();
if (userManager != null) {
Authorizable authorizable =
userManager.getAuthorizable(principalName);
if (authorizable != null) {
@@ -512,7 +512,7 @@ public class AuthorizableResourceProvider extends
ResourceProvider<Object> imple
Resource next = super.createNext(child, principalName,
resourceResolver, session);
if (next == null) {
// SLING-11098 check for principal that is not authorizable
- PrincipalManager principalManager =
AccessControlUtil.getPrincipalManager(session);
+ PrincipalManager principalManager =
((JackrabbitSession)session).getPrincipalManager();
if (principalManager != null) {
@Nullable
Principal principal =
principalManager.getPrincipal(principalName);
diff --git
a/src/test/java/org/apache/sling/jcr/jackrabbit/usermanager/it/AuthorizablePrivilegesInfoIT.java
b/src/test/java/org/apache/sling/jcr/jackrabbit/usermanager/it/AuthorizablePrivilegesInfoIT.java
index 29c2437..a7f3ad3 100644
---
a/src/test/java/org/apache/sling/jcr/jackrabbit/usermanager/it/AuthorizablePrivilegesInfoIT.java
+++
b/src/test/java/org/apache/sling/jcr/jackrabbit/usermanager/it/AuthorizablePrivilegesInfoIT.java
@@ -55,7 +55,6 @@ import org.apache.sling.jackrabbit.usermanager.DeleteUser;
import org.apache.sling.jackrabbit.usermanager.UpdateGroup;
import org.apache.sling.jackrabbit.usermanager.UpdateUser;
import org.apache.sling.jcr.api.SlingRepository;
-import org.apache.sling.jcr.base.util.AccessControlUtil;
import org.apache.sling.jcr.jackrabbit.accessmanager.DeleteAces;
import org.apache.sling.jcr.jackrabbit.accessmanager.ModifyAce;
import org.junit.After;
@@ -786,7 +785,7 @@ public class AuthorizablePrivilegesInfoIT extends
UserManagerTestSupport {
deleteUser.deleteUser(user1Session, user2Id, new ArrayList<>());
user2 = null;
// verify the user is no longer there
- UserManager um = AccessControlUtil.getUserManager(user1Session);
+ UserManager um =
((JackrabbitSession)user1Session).getUserManager();
assertNull("Expected user to be gone: " + user2Id,
um.getAuthorizable(user2Id));
// verify that the user can actually delete the group