You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@ranger.apache.org by pr...@apache.org on 2021/06/14 14:18:11 UTC

[ranger] branch ranger-2.2 updated: RANGER-3312 : Role is getting removed from policy when user present in that policy is deleted

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

pradeep pushed a commit to branch ranger-2.2
in repository https://gitbox.apache.org/repos/asf/ranger.git


The following commit(s) were added to refs/heads/ranger-2.2 by this push:
     new accecd2  RANGER-3312 : Role is getting removed from policy when user present in that policy is deleted
accecd2 is described below

commit accecd25a3587bf3c6c513029f183d2632348e9b
Author: Dineshkumar Yadav <di...@outlook.com>
AuthorDate: Thu Jun 10 12:17:42 2021 +0530

    RANGER-3312 : Role is getting removed from policy when user present in that policy is deleted
    
    Signed-off-by: pradeep <pr...@apache.org>
---
 security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java b/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
index fad332a..6242326 100755
--- a/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
+++ b/security-admin/src/main/java/org/apache/ranger/biz/XUserMgr.java
@@ -2429,7 +2429,7 @@ public class XUserMgr extends XUserMgrBase {
 			if(StringUtils.isNotEmpty(group)) {
 				policyItem.getGroups().remove(group);
 			}
-			if(policyItem.getUsers().isEmpty() && policyItem.getGroups().isEmpty()) {
+			if(policyItem.getUsers().isEmpty() && policyItem.getGroups().isEmpty() && policyItem.getRoles().isEmpty()) {
 				if(itemsToRemove == null) {
 					itemsToRemove = new ArrayList<T>();
 				}