X-Git-Url: http://git.argeo.org/?a=blobdiff_plain;f=security%2Fruntime%2Forg.argeo.security.core%2Fsrc%2Fmain%2Fjava%2Forg%2Fargeo%2Fsecurity%2Fcore%2FDefaultSecurityService.java;h=e005e1ad4fac8bcb794417a56322c25381141078;hb=136dab5338b5f731b285d17c804861bd5e5a9b5c;hp=23e2372c8a5b1adb171c7e5a462d3ddad0e62be3;hpb=5bfc0d2e2e34f86b454a1ec209617a9fc0f306b1;p=lgpl%2Fargeo-commons.git

diff --git a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
index 23e2372c8..e005e1ad4 100644
--- a/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
+++ b/security/runtime/org.argeo.security.core/src/main/java/org/argeo/security/core/DefaultSecurityService.java
@@ -16,12 +16,17 @@
 
 package org.argeo.security.core;
 
+import java.util.Iterator;
+import java.util.Map;
+import java.util.Set;
+
 import org.argeo.ArgeoException;
 import org.argeo.security.ArgeoSecurity;
 import org.argeo.security.ArgeoSecurityDao;
 import org.argeo.security.ArgeoSecurityService;
 import org.argeo.security.ArgeoUser;
 import org.argeo.security.SimpleArgeoUser;
+import org.argeo.security.UserNature;
 import org.springframework.core.task.SimpleAsyncTaskExecutor;
 import org.springframework.core.task.TaskExecutor;
 import org.springframework.security.Authentication;
@@ -56,30 +61,59 @@ public class DefaultSecurityService implements ArgeoSecurityService {
 	public void updateUserPassword(String username, String password) {
 		SimpleArgeoUser user = new SimpleArgeoUser(
 				securityDao.getUser(username));
-		user.setPassword(password);
-		securityDao.update(user);
+		user.setPassword(securityDao.encodePassword(password));
+		securityDao.updateUser(user);
 	}
 
 	public void updateCurrentUserPassword(String oldPassword, String newPassword) {
 		SimpleArgeoUser user = new SimpleArgeoUser(getCurrentUser());
-		if (!user.getPassword().equals(oldPassword))
+		if (!securityDao.isPasswordValid(user.getPassword(), oldPassword))
 			throw new ArgeoException("Old password is not correct.");
-		user.setPassword(newPassword);
-		securityDao.update(user);
+		user.setPassword(securityDao.encodePassword(newPassword));
+		securityDao.updateUser(user);
 	}
 
 	public void newUser(ArgeoUser user) {
-		user.getUserNatures().clear();
 		argeoSecurity.beforeCreate(user);
-		securityDao.create(user);
+		// normalize password
+		if (user instanceof SimpleArgeoUser) {
+			if (user.getPassword() == null || user.getPassword().equals(""))
+				((SimpleArgeoUser) user).setPassword(securityDao
+						.encodePassword(user.getUsername()));
+			else if (!user.getPassword().startsWith("{"))
+				((SimpleArgeoUser) user).setPassword(securityDao
+						.encodePassword(user.getPassword()));
+		}
+		securityDao.createUser(user);
+	}
+
+	public ArgeoUser getUser(String username) {
+		return securityDao.getUser(username);
+	}
+
+	public Boolean userExists(String username) {
+		return securityDao.userExists(username);
 	}
 
 	public void updateUser(ArgeoUser user) {
-		String password = securityDao.getUserWithPassword(user.getUsername())
-				.getPassword();
+		String password = user.getPassword();
+		if (password == null)
+			password = securityDao.getUserWithPassword(user.getUsername())
+					.getPassword();
+		if (!password.startsWith("{"))
+			password = securityDao.encodePassword(user.getPassword());
 		SimpleArgeoUser simpleArgeoUser = new SimpleArgeoUser(user);
 		simpleArgeoUser.setPassword(password);
-		securityDao.update(simpleArgeoUser);
+		securityDao.updateUser(simpleArgeoUser);
+	}
+
+	public void deleteUser(String username) {
+		securityDao.deleteUser(username);
+
+	}
+
+	public void deleteRole(String role) {
+		securityDao.deleteRole(role);
 	}
 
 	public TaskExecutor createSystemAuthenticatedTaskExecutor() {
@@ -115,6 +149,33 @@ public class DefaultSecurityService implements ArgeoSecurityService {
 		};
 	}
 
+	public Set<ArgeoUser> listUsersInRole(String role) {
+		Set<ArgeoUser> lst = securityDao.listUsersInRole(role);
+		Iterator<ArgeoUser> it = lst.iterator();
+		while (it.hasNext()) {
+			if (it.next().getUsername()
+					.equals(argeoSecurity.getSuperUsername())) {
+				it.remove();
+				break;
+			}
+		}
+		return lst;
+	}
+
+	public void updateCurrentUserNatures(Map<String, UserNature> userNatures) {
+		// TODO Auto-generated method stub
+
+	}
+
+	public Set<ArgeoUser> listUsers() {
+		return securityDao.listUsers();
+	}
+
+	public Set<String> listEditableRoles() {
+		// TODO Auto-generated method stub
+		return securityDao.listEditableRoles();
+	}
+
 	public void setArgeoSecurity(ArgeoSecurity argeoSecurity) {
 		this.argeoSecurity = argeoSecurity;
 	}