Change in ovirt-engine[master]: core: Fix DB storing for principal names as provided by Acti...

[yzaslavs at redhat.com]

Yair Zaslavsky has submitted this change and it was merged.

Change subject: core: Fix DB storing for principal names as provided by Active-Directory
......................................................................


core: Fix DB storing for principal names as provided by Active-Directory

IPA provides principal correctly (format of user at domain).
AD provides only the username.
We would like to present at UI in format of principal name.
Since this issue may occur with other directory services,
I prefered to handle this at the LdapBrokerCommandBase , regardless
of provider type.
The upgrade script is required to handle already existing users.

Bug-Url: https://bugzilla.redhat.com/888736
Change-Id: Ia9f8b60280d76b72a26f0d1feaf2650c4ef1ce3f
Signed-off-by: Yair Zaslavsky <yzaslavs at redhat.com>
---
A backend/manager/dbscripts/upgrade/03_02_0310_fix_principal_name_in_users_for_ad_users
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/adbroker/LdapBrokerCommandBase.java
2 files changed, 5 insertions(+), 1 deletion(-)

Approvals:
  Yair Zaslavsky: Verified; Looks good to me, approved


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

Gerrit-MessageType: merged
Gerrit-Change-Id: Ia9f8b60280d76b72a26f0d1feaf2650c4ef1ce3f
Gerrit-PatchSet: 5
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Eli Mesika <emesika at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>