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

mperina at redhat.com mperina at redhat.com
Wed Sep 7 08:36:46 UTC 2016


Martin Peřina 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>
---
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: Verified
  Martin Peřina: Looks good to me, approved
  Ravi Nori: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I45bfabbea16bac2e47506744d8e5c3429bbabe90
Gerrit-PatchSet: 3
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ondra Machacek <omachace 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: Ravi Nori <rnori at redhat.com>
Gerrit-Reviewer: gerrit-hooks <automation at ovirt.org>



More information about the Engine-commits mailing list