Fix

Signed-off-by: Nils Bandener <[email protected]>
nibix · Jun 28, 2024 · d3fe0d4 · d3fe0d4
1 parent f93125f
commit d3fe0d4
Showing 1 changed file with 3 additions and 3 deletions.
diff --git a/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluatorResponse.java b/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluatorResponse.java
@@ -141,14 +141,14 @@ public static PrivilegesEvaluatorResponse partiallyOk(
         PrivilegesEvaluatorResponse response = new PrivilegesEvaluatorResponse();
         response.onlyAllowedForIndices = ImmutableSet.copyOf(availableIndices);
         response.indexToActionCheckTable = indexToActionCheckTable;
-        response.resolvedSecurityRoles.addAll(context.getMappedRoles());
+        response.resolvedSecurityRoles(context.getMappedRoles());
         return response;
     }
 
     public static PrivilegesEvaluatorResponse insufficient(String missingPrivilege, PrivilegesEvaluationContext context) {
         PrivilegesEvaluatorResponse response = new PrivilegesEvaluatorResponse();
         response.indexToActionCheckTable = CheckTable.create(ImmutableSet.of("_"), ImmutableSet.of(missingPrivilege));
-        response.resolvedSecurityRoles.addAll(context.getMappedRoles());
+        response.resolvedSecurityRoles(context.getMappedRoles());
         return response;
     }
 
@@ -158,7 +158,7 @@ public static PrivilegesEvaluatorResponse insufficient(
     ) {
         PrivilegesEvaluatorResponse response = new PrivilegesEvaluatorResponse();
         response.indexToActionCheckTable = indexToActionCheckTable;
-        response.resolvedSecurityRoles.addAll(context.getMappedRoles());
+        response.resolvedSecurityRoles(context.getMappedRoles());
         return response;
     }