You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@directory.apache.org by sm...@apache.org on 2018/10/24 18:51:10 UTC
directory-fortress-core git commit: removeRoleConstraint contextId
not getting set on entity cause multitenancy bug.
Repository: directory-fortress-core
Updated Branches:
refs/heads/master fdc0662b6 -> 03dda314f
removeRoleConstraint contextId not getting set on entity cause multitenancy bug.
Project: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/repo
Commit: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/commit/03dda314
Tree: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/tree/03dda314
Diff: http://git-wip-us.apache.org/repos/asf/directory-fortress-core/diff/03dda314
Branch: refs/heads/master
Commit: 03dda314fcf278e8b7738d3471963a0a204cdaf9
Parents: fdc0662
Author: Shawn McKinney <sm...@apache.org>
Authored: Tue Oct 23 16:38:35 2018 -0500
Committer: Shawn McKinney <sm...@apache.org>
Committed: Tue Oct 23 16:38:35 2018 -0500
----------------------------------------------------------------------
.../org/apache/directory/fortress/core/impl/AdminMgrImpl.java | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
----------------------------------------------------------------------
http://git-wip-us.apache.org/repos/asf/directory-fortress-core/blob/03dda314/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
----------------------------------------------------------------------
diff --git a/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java b/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
index 968a5c2..1cf8df6 100755
--- a/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
+++ b/src/main/java/org/apache/directory/fortress/core/impl/AdminMgrImpl.java
@@ -504,8 +504,9 @@ public final class AdminMgrImpl extends Manageable implements AdminMgr, Serializ
//find role constraint that needs removed
boolean found = false;
-
- List<UserRole> userRoles = userP.read( new User(uRole.getUserId()), true ).getRoles();
+ User user = new User(uRole.getUserId());
+ user.setContextId( contextId );
+ List<UserRole> userRoles = userP.read( user, true ).getRoles();
for( UserRole ur : userRoles ){
// find matching name
if( ur.getName().equals( uRole.getName() ) ){