Gilad Chaplik has posted comments on this change.

Change subject: frontend: login: split user@domain
......................................................................


Patch Set 2:

@Einav, @Vojtech,

Please see https://bugzilla.redhat.com/show_bug.cgi?id=1112252#c7 , the issue 
raised in the comment can be solved pretty easily in the GUI and in the backend 
as well.

imo we need this fix because we cannot use browser's cookie to save user+domain 
anymore, means you'll need to select the domain as well -> imo really annoying 
(especially if you're used to it in previous versions).

I think it's much more serious that we no longer supports the above, than the 
chance administrator-user will use '@internal' suffix for his users 
configuration, he can always rename, sorry :)

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

Gerrit-MessageType: comment
Gerrit-Change-Id: Ie74a9b0d74b081c4966a26a5d9de1598c91fac51
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Einav Cohen <[email protected]>
Gerrit-Reviewer: Gilad Chaplik <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Vojtech Szocs <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: No
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to