diff --git a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserMapper.java b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserMapper.java index 0563196de..5ef96cc2e 100644 --- a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserMapper.java +++ b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserMapper.java @@ -32,23 +32,6 @@ import org.apache.ibatis.annotations.Param; */ public interface UserMapper extends DirectoryObjectMapper { - /** - * Returns the user having the given username and password, if any. If no - * such user exists, null is returned. - * - * @param username - * The username of the user to return. - * - * @param password - * The password of the user to return. - * - * @return - * The user having the given username and password, or null if no such - * user exists. - */ - UserModel selectOneByCredentials(@Param("username") String username, - @Param("password") String password); - /** * Returns the user having the given username, if any. If no such user * exists, null is returned. diff --git a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserService.java b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserService.java index 1fd24dc54..98114434d 100644 --- a/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserService.java +++ b/extensions/guacamole-auth-jdbc/modules/guacamole-auth-jdbc-base/src/main/java/org/glyptodon/guacamole/auth/jdbc/user/UserService.java @@ -24,6 +24,7 @@ package org.glyptodon.guacamole.auth.jdbc.user; import com.google.inject.Inject; import com.google.inject.Provider; +import java.util.Arrays; import java.util.Collection; import java.util.Collections; import org.glyptodon.guacamole.net.auth.Credentials; @@ -33,6 +34,7 @@ import org.glyptodon.guacamole.GuacamoleClientException; import org.glyptodon.guacamole.GuacamoleException; import org.glyptodon.guacamole.auth.jdbc.permission.ObjectPermissionMapper; import org.glyptodon.guacamole.auth.jdbc.permission.UserPermissionMapper; +import org.glyptodon.guacamole.auth.jdbc.security.PasswordEncryptionService; import org.glyptodon.guacamole.net.auth.User; import org.glyptodon.guacamole.net.auth.permission.ObjectPermissionSet; import org.glyptodon.guacamole.net.auth.permission.SystemPermission; @@ -64,6 +66,12 @@ public class UserService extends DirectoryObjectService userProvider; + /** + * Service for hashing passwords. + */ + @Inject + private PasswordEncryptionService encryptionService; + @Override protected DirectoryObjectMapper getObjectMapper() { return userMapper; @@ -169,15 +177,24 @@ public class UserService extends DirectoryObjectService - - -