Use set for roles

Signed-off-by: Sergey Nuyanzin <snuyanzin@gmail.com>
Aiven-Open · Nov 20, 2023 · b75b830 · b75b830
1 parent 1b2bc6a
commit b75b830
Showing 1 changed file with 3 additions and 1 deletion.
diff --git a/core/src/main/java/io/aiven/klaw/service/MailUtils.java b/core/src/main/java/io/aiven/klaw/service/MailUtils.java
@@ -11,10 +11,12 @@
 import io.aiven.klaw.model.enums.MailType;
 import io.aiven.klaw.model.enums.PermissionType;
 import java.util.ArrayList;
+import java.util.HashSet;
 import java.util.List;
 import java.util.Map;
 import java.util.Objects;
 import java.util.Optional;
+import java.util.Set;
 import java.util.concurrent.CompletableFuture;
 import java.util.concurrent.ExecutionException;
 import lombok.extern.slf4j.Slf4j;
@@ -635,7 +637,7 @@ private List<String> getAllUsersWithPermissionToApproveRequest(
     Map<String, List<String>> rolesToPermissions =
         manageDatabase.getRolesPermissionsPerTenant(tenantId);
 
-    List<String> roles = new ArrayList<>();
+    Set<String> roles = new HashSet<>();
 
     rolesToPermissions.forEach(
         (k, v) -> {