Change in ovirt-engine[master]: core: disable everyone for login
Oved Ourfali has submitted this change and it was merged.
Change subject: core: disable everyone for login
......................................................................
core: disable everyone for login
If the "everyone" has assignd a role which contains login,
from this point everyone is allowed to log in.
In this specific bug the NetworkUser has been assigned to
everyone which contained the login. This caused that everyone
was allowed to log in. This issue did not happened the first time
so this patch solves it in a more generic way - ignores the
everyone when checking if the user is allowed to log in.
Change-Id: I7bba37a450efdd9e3475470e4ed8f49347c51a2e
Bug-Url: https://bugzilla.redhat.com/916328
Signed-off-by: Tomas Jelinek <tjelinek(a)redhat.com>
---
M backend/manager/dbscripts/multi_level_administration_sp.sql
M backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/CommandBase.java
M
backend/manager/modules/bll/src/main/java/org/ovirt/engine/core/bll/LoginBaseCommand.java
M
backend/manager/modules/dal/src/main/java/org/ovirt/engine/core/dal/dbbroker/DbFacade.java
M
backend/manager/modules/dal/src/test/java/org/ovirt/engine/core/dao/DbFacadeDAOTest.java
5 files changed, 12 insertions(+), 9 deletions(-)
Approvals:
Tomas Jelinek: Verified
Oved Ourfali: Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/12830
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I7bba37a450efdd9e3475470e4ed8f49347c51a2e
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Tomas Jelinek <tjelinek(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Tomas Jelinek <tjelinek(a)redhat.com>