No longer expose CheckTable

Signed-off-by: Nils Bandener <[email protected]>
nibix · Jun 28, 2024 · f653945 · f653945
1 parent 31c20ae
commit f653945
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 3 deletions.
diff --git a/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluator.java b/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluator.java
@@ -570,7 +570,7 @@ public PrivilegesEvaluatorResponse evaluate(PrivilegesEvaluationContext context)
                 action0,
                 mappedRoles
             );
-            log.info("Index to privilege matrix:\n{}", presponse.getCheckTable());
+            log.info("Index to privilege matrix:\n{}", presponse.getPrivilegeMatrix());
         }
 
         return presponse;

diff --git a/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluatorResponse.java b/src/main/java/org/opensearch/security/privileges/PrivilegesEvaluatorResponse.java
@@ -44,6 +44,7 @@ public class PrivilegesEvaluatorResponse {
     CreateIndexRequestBuilder createIndexRequestBuilder;
     private Set<String> onlyAllowedForIndices = ImmutableSet.of();
     private CheckTable<String, String> indexToActionCheckTable;
+    private String privilegeMatrix;
     private String reason;
 
     /**
@@ -77,8 +78,14 @@ public PrivilegesEvaluatorResponse reason(String reason) {
         return this;
     }
 
-    public CheckTable<String, String> getCheckTable() {
-        return indexToActionCheckTable;
+    public String getPrivilegeMatrix() {
+        String result = this.privilegeMatrix;
+
+        if (result == null) {
+            result = this.indexToActionCheckTable.toTableString("ok", "MISSING");
+            this.privilegeMatrix = result;
+        }
+        return result;
     }
 
     public Set<String> getMissingSecurityRoles() {