Alon Bar-Lev has uploaded a new change for review.

Change subject: aaa: populate display group earlier at login
......................................................................

aaa: populate display group earlier at login

we have a mess between groups that we display and group that we actually
use (uuid), populate display group earlier in login.

Topic: AAA
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1120720
Change-Id: I01da660c44899eabc85da5c23540ca6a9e8ec433
Signed-off-by: Alon Bar-Lev <[email protected]>
---
M 
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java
1 file changed, 1 insertion(+), 1 deletion(-)


  git pull ssh://gerrit.ovirt.org:29418/ovirt-engine refs/changes/41/30941/1

diff --git 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java
 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java
index 2aa7dfb..126c840 100644
--- 
a/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java
+++ 
b/backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/aaa/LoginBaseCommand.java
@@ -256,6 +256,7 @@
         }
 
         // Check that the user exists in the database, if it doesn't exist 
then we need to add it now:
+        DirectoryUtils.flatGroups(principalRecord);
         DbUser dbUser =
                 getDbUserDAO().getByExternalId(
                         AuthzUtils.getName(profile.getAuthz()),
@@ -264,7 +265,6 @@
             dbUser = 
DirectoryUtils.mapPrincipalRecordToDbUser(AuthzUtils.getName(profile.getAuthz()),
 principalRecord);
             dbUser.setId(Guid.newGuid());
         }
-        DirectoryUtils.flatGroups(principalRecord);
         
dbUser.setGroupIds(DirectoryUtils.getGroupIdsFromPrincipal(AuthzUtils.getName(profile.getAuthz()),
 principalRecord));
         if (!dbUser.isActive()) {
             dbUser.setActive(true);


-- 
To view, visit http://gerrit.ovirt.org/30941
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: newchange
Gerrit-Change-Id: I01da660c44899eabc85da5c23540ca6a9e8ec433
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-3.5
Gerrit-Owner: Alon Bar-Lev <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to