Change in ovirt-engine[ovirt-engine-4.0]: aaa: sso: print only error message on login screen if fail

tnisan at redhat.com tnisan at redhat.com
Tue Sep 20 12:52:42 UTC 2016


Tal Nisan has submitted this change and it was merged.

Change subject: aaa: sso: print only error message on login screen if fail
......................................................................


aaa: sso: print only error message on login screen if fail

Before we printed full cause of the error to login screen, when login
failed. This error was unreadable to user, as user is interested only
to error message not full Java stack trace. This patch changes it, so
we now only show the error.

Change-Id: I45bfabbea16bac2e47506744d8e5c3429bbabe90
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1373092
Signed-off-by: Ondra Machacek <omachace at redhat.com>
(cherry picked from commit fa6bd301ff12f3558550e17adcd7f97c907b32e0)
---
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/servlets/InteractiveAuthServlet.java
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  Ondra Machacek: Looks good to me, but someone else must approve
  Martin Peřina: Verified; Looks good to me, approved
  Jenkins CI: Passed CI tests



-- 
To view, visit https://gerrit.ovirt.org/64190
To unsubscribe, visit https://gerrit.ovirt.org/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I45bfabbea16bac2e47506744d8e5c3429bbabe90
Gerrit-PatchSet: 1
Gerrit-Project: ovirt-engine
Gerrit-Branch: ovirt-engine-4.0
Gerrit-Owner: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Jenkins CI
Gerrit-Reviewer: Martin Peřina <mperina at redhat.com>
Gerrit-Reviewer: Ondra Machacek <omachace at redhat.com>
Gerrit-Reviewer: Piotr Kliczewski <piotr.kliczewski at gmail.com>
Gerrit-Reviewer: Tal Nisan <tnisan at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list