diff --git a/src/main/java/sh/libre/scim/core/AbstractScimService.java b/src/main/java/sh/libre/scim/core/AbstractScimService.java index 8f92a2268147689f42cba210a4407b67fae3c328..a0939adb460c5871528f1b14561327532dcbd1a2 100644 --- a/src/main/java/sh/libre/scim/core/AbstractScimService.java +++ b/src/main/java/sh/libre/scim/core/AbstractScimService.java @@ -97,7 +97,7 @@ public abstract class AbstractScimService<RMM extends RoleMapperModel, S extends scimClient.delete(externalId); getScimResourceDao().delete(resource); } catch (NoSuchElementException e) { - throw new ScimPropagationException("Failed to delete resource %s, scim mapping not found : " + id, e); + throw new ScimPropagationException("Failed to delete resource %s, scim mapping not found: ".formatted(id), e); } } diff --git a/src/main/java/sh/libre/scim/core/ScimDispatcher.java b/src/main/java/sh/libre/scim/core/ScimDispatcher.java index 8154bb393024c3fe3c7d28b87b8848dc69b2b324..62a8bfbda67c404099cddbe2ad67091fd00f9a84 100644 --- a/src/main/java/sh/libre/scim/core/ScimDispatcher.java +++ b/src/main/java/sh/libre/scim/core/ScimDispatcher.java @@ -60,7 +60,7 @@ public class ScimDispatcher { userScimServices.add(userScimService); } } catch (Exception e) { - logger.warnf("[SCIM] Invalid Endpoint configuration %s : %s", scimEndpointConfigurationRaw.getId(), e.getMessage()); + logger.warnf("[SCIM] Invalid Endpoint configuration %s: %s", scimEndpointConfigurationRaw.getId(), e.getMessage()); // TODO is it ok to log and try to create the other clients ? } }); diff --git a/src/main/java/sh/libre/scim/event/ScimEventListenerProvider.java b/src/main/java/sh/libre/scim/event/ScimEventListenerProvider.java index e9635d455a26df0f20287f908eb743913fafff05..06a01db52b4ab99e4089bc7daf9d0e7b04ae8f73 100644 --- a/src/main/java/sh/libre/scim/event/ScimEventListenerProvider.java +++ b/src/main/java/sh/libre/scim/event/ScimEventListenerProvider.java @@ -107,7 +107,7 @@ public class ScimEventListenerProvider implements EventListenerProvider { ScimResourceType type = switch (rawResourceType) { case "users" -> ScimResourceType.USER; case "groups" -> ScimResourceType.GROUP; - default -> throw new IllegalArgumentException("Unsuported resource type : " + rawResourceType); + default -> throw new IllegalArgumentException("Unsuported resource type: " + rawResourceType); }; KeycloakId id = new KeycloakId(matcher.group(2)); handleRoleMappingEvent(event, type, id); diff --git a/src/main/java/sh/libre/scim/storage/ScimEndpointConfigurationStorageProviderFactory.java b/src/main/java/sh/libre/scim/storage/ScimEndpointConfigurationStorageProviderFactory.java index 748e18b5cac615d017dc58af41a8f18a2c850945..00a527fe3ecb22e62e2ebd469dd72061b8fdfd8b 100644 --- a/src/main/java/sh/libre/scim/storage/ScimEndpointConfigurationStorageProviderFactory.java +++ b/src/main/java/sh/libre/scim/storage/ScimEndpointConfigurationStorageProviderFactory.java @@ -78,7 +78,7 @@ public class ScimEndpointConfigurationStorageProviderFactory ScimDispatcher dispatcher = new ScimDispatcher(session); for (GroupModel group : session.groups().getGroupsStream(realm) .filter(x -> BooleanUtils.TRUE.equals(x.getFirstAttribute("scim-dirty"))).toList()) { - logger.infof("[SCIM] Dirty group : %s", group.getName()); + logger.infof("[SCIM] Dirty group: %s", group.getName()); dispatcher.dispatchGroupModificationToAll(client -> client.replace(group)); group.removeAttribute("scim-dirty"); }