(ranger) branch master updated: RANGER-5223: Audit logs not generated for Dataset modifications (Labels, Keywords, isEnabled)

[rmani]

This is an automated email from the ASF dual-hosted git repository.

rmani pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/master by this push:
     new 1e03bcaa7 RANGER-5223: Audit logs not generated for Dataset 
modifications (Labels, Keywords, isEnabled)
1e03bcaa7 is described below

commit 1e03bcaa774a0e5758c9a05cd30c5bdfba2a2cba
Author: Radhika Kundam <radhika.kun...@gmail.com>
AuthorDate: Tue Jun 10 17:16:20 2025 -0700

    RANGER-5223: Audit logs not generated for Dataset modifications (Labels, 
Keywords, isEnabled)
    
    Signed-off-by: Ramesh Mani <rm...@apache.org>
---
 .../java/org/apache/ranger/common/view/VTrxLogAttr.java     | 13 ++++++-------
 .../org/apache/ranger/service/RangerGdsDatasetService.java  |  7 +++++--
 2 files changed, 11 insertions(+), 9 deletions(-)

diff --git 
a/security-admin/src/main/java/org/apache/ranger/common/view/VTrxLogAttr.java 
b/security-admin/src/main/java/org/apache/ranger/common/view/VTrxLogAttr.java
index 5c637363f..4d3990aaf 100644
--- 
a/security-admin/src/main/java/org/apache/ranger/common/view/VTrxLogAttr.java
+++ 
b/security-admin/src/main/java/org/apache/ranger/common/view/VTrxLogAttr.java
@@ -114,15 +114,14 @@ public String toString() {
     }
 
     private Field getField(Object obj) {
-        Field field = null;
+        Field field    = null;
+        Class objClass = obj.getClass();
 
-        try {
-            field = obj.getClass().getDeclaredField(attribName);
-        } catch (NoSuchFieldException excp) {
+        while (field == null && objClass != null) {
             try {
-                field = 
obj.getClass().getSuperclass().getDeclaredField(attribName);
-            } catch (NoSuchFieldException excp1) {
-                // ignore
+                field = objClass.getDeclaredField(attribName);
+            } catch (NoSuchFieldException excp) {
+                objClass = objClass.getSuperclass();
             }
         }
 
diff --git 
a/security-admin/src/main/java/org/apache/ranger/service/RangerGdsDatasetService.java
 
b/security-admin/src/main/java/org/apache/ranger/service/RangerGdsDatasetService.java
index 7ded325d2..a66bfb7f6 100755
--- 
a/security-admin/src/main/java/org/apache/ranger/service/RangerGdsDatasetService.java
+++ 
b/security-admin/src/main/java/org/apache/ranger/service/RangerGdsDatasetService.java
@@ -83,9 +83,12 @@ public RangerGdsDatasetService() {
         sortFields.add(new SortField(SearchFilter.DATASET_ID, "obj.id", true, 
SortField.SORT_ORDER.ASC));
         sortFields.add(new SortField(SearchFilter.DATASET_NAME, "obj.name"));
 
-        trxLogAttrs.put("name", new VTrxLogAttr("name", "Name", false, true));
-        trxLogAttrs.put("acl", new VTrxLogAttr("acl", "ACL"));
+        trxLogAttrs.put("name",       new VTrxLogAttr("name", "Name", false, 
true));
+        trxLogAttrs.put("acl",        new VTrxLogAttr("acl", "ACL"));
         trxLogAttrs.put("termsOfUse", new VTrxLogAttr("termsOfUse", "Terms of 
use"));
+        trxLogAttrs.put("isEnabled",  new VTrxLogAttr("isEnabled", "Dataset 
Status"));
+        trxLogAttrs.put("labels",     new VTrxLogAttr("labels", "Labels"));
+        trxLogAttrs.put("keywords",   new VTrxLogAttr("keywords", "keywords"));
     }
 
     @Override