X-Git-Url: https://git.argeo.org/?a=blobdiff_plain;f=org.argeo.cms%2Fsrc%2Forg%2Fargeo%2Fcms%2Finternal%2Fkernel%2FHomeRepository.java;h=1546a0b6786be387df8e4311c7225f86f77cafdd;hb=36c14000ab482f558a9679f75ac7bfb0953171f4;hp=bfcb940c1d8336c1d74f8b7fb8c12f85609f1fed;hpb=1c25d6856f45fae563a7b6085bf7aa57c5b74793;p=lgpl%2Fargeo-commons.git

diff --git a/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java b/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
index bfcb940c1..1546a0b67 100644
--- a/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
+++ b/org.argeo.cms/src/org/argeo/cms/internal/kernel/HomeRepository.java
@@ -165,12 +165,13 @@ class HomeRepository extends JcrRepositoryWrapper implements KernelConstants {
 		}
 		try {
 			// TODO enhance transformation of cn to a valid node name
-			String relPath = cn.replaceAll("[^a-zA-Z0-9]", "_");
+			// String relPath = cn.replaceAll("[^a-zA-Z0-9]", "_");
+			String relPath = JcrUtils.replaceInvalidChars(cn);
 			newWorkgroup = JcrUtils.mkdirs(adminSession.getNode(groupsBasePath), relPath, NodeType.NT_UNSTRUCTURED);
 			newWorkgroup.addMixin(NodeTypes.NODE_GROUP_HOME);
 			newWorkgroup.setProperty(NodeNames.LDAP_CN, cn);
 			adminSession.save();
-			JcrUtils.addPrivilege(adminSession, newWorkgroup.getPath(), dn.toString(), Privilege.JCR_ALL);
+			JcrUtils.addPrivilege(adminSession, newWorkgroup.getPath(), dn.toString(), Privilege.JCR_READ);
 			adminSession.save();
 		} catch (RepositoryException e) {
 			throw new CmsException("Cannot create workgroup", e);