Reput everything

Author: RomuDeuxfois

openaev-api/src/main/java/io/openaev/config/security/SecurityService.java

@@ -6,10 +6,10 @@
 
 import io.openaev.database.model.Tenant;
 import io.openaev.database.model.User;
+import io.openaev.database.repository.TenantRepository;
 import io.openaev.database.repository.UserRepository;
 import io.openaev.service.UserMappingService;
 import io.openaev.service.UserService;
-import io.openaev.service.tenants.TenantService;
 import io.openaev.service.user_events.UserEventService;
 import jakarta.persistence.EntityNotFoundException;
 import jakarta.validation.constraints.NotBlank;
@@ -40,7 +40,7 @@ public class SecurityService {
   private final UserMappingService userMappingService;
   private final Environment env;
   private final UserEventService userEventService;
-  private final TenantService tenantService;
+  private final TenantRepository tenantRepository;
 
   public User userManagement(
       String emailAttribute,
@@ -114,7 +114,11 @@ private void attachTenant(String registrationId, User user) {
       return;
     }
     try {
-      Tenant tenant = tenantService.findById(tenantId);
+      Tenant tenant =
+          tenantRepository
+              .findById(tenantId)
+              .orElseThrow(() -> new EntityNotFoundException("Tenant not found: " + tenantId));
+      ;
       user.getTenants().add(tenant);
     } catch (EntityNotFoundException e) {
       log.warn("SSO tenant ID '{}' configured but not found in database", tenantId);

openaev-api/src/main/java/io/openaev/service/UserMappingService.java

@@ -5,12 +5,9 @@
 import com.fasterxml.jackson.core.type.TypeReference;
 import com.fasterxml.jackson.databind.ObjectMapper;
 import io.openaev.database.model.Group;
-import io.openaev.database.model.Tenant;
 import io.openaev.database.model.User;
 import io.openaev.database.repository.GroupRepository;
-import io.openaev.service.tenants.TenantService;
 import io.openaev.sso.GroupMapping;
-import jakarta.persistence.EntityNotFoundException;
 import jakarta.validation.constraints.NotBlank;
 import java.io.IOException;
 import java.util.ArrayList;
@@ -32,7 +29,6 @@
 public class UserMappingService {
 
   private final GroupRepository groupRepository;
-  private final TenantService tenantService;
   private final Environment env;
   public static final String ROLES_PATH_SUFFIX = "roles_path";
   public static final String GROUPS_PATH_SUFFIX = "groups_path";
@@ -68,7 +64,6 @@ public void mapCurrentUserWithGroup(String property, User user, List<String> gro
               log.error("Did not create new group");
             }
           }
-          attachTenantFromGroupMapping(mapping, user);
         } else {
           log.error(String.format("No corresponding group found for group %s", role));
         }
@@ -91,35 +86,14 @@ public void mapCurrentUserWithGroup(String property, User user, List<String> gro
   private static List<GroupMapping> safeParseMappings(String json) {
     ObjectMapper mapper = new ObjectMapper();
     try {
-      return mapper.readValue(json, new TypeReference<>() {});
+      return mapper.readValue(json, new TypeReference<List<GroupMapping>>() {});
     } catch (IOException e) {
       // Log and return empty list instead of throwing
-      log.error("Failed to parse group mappings: {}", e.getMessage(), e);
+      System.err.println("Failed to parse mappings: " + e.getMessage());
       return List.of();
     }
   }
 
-  /**
-   * Attaches the user to the tenant configured in the group mapping, if any. Skips if tenantId is
-   * not set, the user is already attached, or the tenant is not found.
-   */
-  private void attachTenantFromGroupMapping(GroupMapping mapping, User user) {
-    String tenantId = mapping.getTenantId();
-    if (tenantId == null || tenantId.isBlank()) {
-      return;
-    }
-    boolean alreadyAttached = user.getTenants().stream().anyMatch(t -> t.getId().equals(tenantId));
-    if (alreadyAttached) {
-      return;
-    }
-    try {
-      Tenant tenant = tenantService.findById(tenantId);
-      user.getTenants().add(tenant);
-    } catch (EntityNotFoundException e) {
-      log.warn("Group mapping tenant ID '{}' configured but not found in database", tenantId);
-    }
-  }
-
   /**
    * Extract the roles from a user
    *

openaev-api/src/main/java/io/openaev/sso/GroupMapping.java

@@ -14,7 +14,4 @@ public class GroupMapping {
 
   @JsonProperty("autoCreate")
   private boolean autoCreate;
-
-  @JsonProperty("tenantId")
-  private String tenantId;
 }

openaev-api/src/main/resources/application.properties

@@ -169,7 +169,6 @@ openaev.auth-saml2-enable=false
 # openaev.provider.{registrationId}.roles_admin=
 # openaev.provider.{registrationId}.audience=
 # openaev.provider.{registrationId}.groups_management=
-# openaev.provider.{registrationId}.tenant_id=
 
 ## Kerberos
 openaev.auth-kerberos-enable=false