Allow for non-unique principal name (#251)

src/main/java/info/fingo/urlopia/user/ActiveDirectoryUserLeaderProvider.java

@@ -91,7 +91,7 @@ private Optional<User> getManagerDetails(String managerDN) throws NamingExceptio
             var principalNameAttribute = attributes.get(Attribute.PRINCIPAL_NAME.getKey());
             var principalName = principalNameAttribute != null ? (String) principalNameAttribute.get() : "";
             if (!principalName.isBlank()) {
-                return this.userRepository.findFirstByPrincipalName(principalName);
+                return this.userRepository.findFirstByPrincipalNameAndActiveTrue(principalName);
             }
         }
         return Optional.empty();

src/main/java/info/fingo/urlopia/user/User.java

@@ -19,7 +19,7 @@ public class User {
     @Column(nullable = false, unique = true)
     private String accountName;
 
-    @Column(nullable = false, unique = true)
+    @Column(nullable = false)
     private String principalName;
 
     @Column(nullable = false, unique = true)

src/main/java/info/fingo/urlopia/user/UserRepository.java

@@ -11,7 +11,7 @@ public interface UserRepository extends BaseRepository<User>, JpaRepository<User
 
     Optional<User> findFirstByMailAndActiveTrue(String mail);
 
-    Optional<User> findFirstByPrincipalName(String principalName);
+    Optional<User> findFirstByPrincipalNameAndActiveTrue(String principalName);
 
     Optional<User> findFirstByAccountName(String accountName);
 

src/main/resources/scripts/U3_5_0_4__drop_principal_name_unique_constraint.sql

@@ -0,0 +1,2 @@
+CREATE UNIQUE INDEX users_principal_name_index ON users (principal_name);
+CREATE UNIQUE INDEX users_principal_name_key ON users (principal_name);

src/main/resources/scripts/V3_5_0_4__drop_principal_name_unique_constraint.sql

@@ -0,0 +1,2 @@
+ALTER TABLE users DROP CONSTRAINT IF EXISTS users_principal_name_index;
+ALTER TABLE users DROP CONSTRAINT IF EXISTS users_principal_name_key;