[SYNCOPE-1808] fix location for group in ResourceTypes SCIMv2 service

apache · Feb 29, 2024 · e594496 · e594496
1 parent 94c778b
commit e594496
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 8 deletions.
diff --git a/ext/scimv2/logic/src/main/java/org/apache/syncope/core/logic/SCIMLogic.java b/ext/scimv2/logic/src/main/java/org/apache/syncope/core/logic/SCIMLogic.java
@@ -132,26 +132,25 @@ public ServiceProviderConfig serviceProviderConfig(final UriBuilder uriBuilder)
     }
 
     @PreAuthorize("isAuthenticated()")
-    public static List<ResourceType> resourceTypes(final UriBuilder uriBuilder) {
+    public List<ResourceType> resourceTypes(final UriBuilder uriBuilder) {
         synchronized (MONITOR) {
+            String uri = uriBuilder.build().toASCIIString();
             if (USER == null) {
                 USER = new ResourceType("User", "User", "/Users", "User Account", Resource.User.schema(),
-                        new Meta(Resource.ResourceType,
-                                null, null, null, uriBuilder.path("User").build().toASCIIString()));
+                        new Meta(Resource.ResourceType, null, null, null, uri + "User"));
                 USER.getSchemaExtensions().add(new SchemaExtension(Resource.EnterpriseUser.schema(), true));
             }
             if (GROUP == null) {
                 GROUP = new ResourceType("Group", "Group", "/Groups", "Group", Resource.Group.schema(),
-                        new Meta(Resource.ResourceType,
-                                null, null, null, uriBuilder.path("Group").build().toASCIIString()));
+                        new Meta(Resource.ResourceType, null, null, null, uri + "Group"));
             }
         }
 
         return List.of(USER, GROUP);
     }
 
     @PreAuthorize("isAuthenticated()")
-    public static ResourceType resourceType(final UriBuilder uriBuilder, final String type) {
+    public ResourceType resourceType(final UriBuilder uriBuilder, final String type) {
         if (Resource.User.name().equals(type)) {
             resourceTypes(uriBuilder);
             return USER;

diff --git a/...cim-rest-cxf/src/main/java/org/apache/syncope/ext/scimv2/cxf/service/SCIMServiceImpl.java b/...cim-rest-cxf/src/main/java/org/apache/syncope/ext/scimv2/cxf/service/SCIMServiceImpl.java
@@ -56,12 +56,12 @@ public ServiceProviderConfig serviceProviderConfig() {
 
     @Override
     public List<ResourceType> resourceTypes() {
-        return SCIMLogic.resourceTypes(uriInfo.getAbsolutePathBuilder());
+        return scimLogic.resourceTypes(uriInfo.getAbsolutePathBuilder());
     }
 
     @Override
     public ResourceType resourceType(final String type) {
-        return SCIMLogic.resourceType(uriInfo.getAbsolutePathBuilder(), type);
+        return scimLogic.resourceType(uriInfo.getAbsolutePathBuilder(), type);
     }
 
     @Override