Change in ovirt-engine[master]: core: aaa: Fix multiple misc.authn checking

[Code Review]

>From Moti Asayag <masayag at redhat.com>:

Moti Asayag has submitted this change and it was merged. ( https://gerrit.ovirt.org/79280 )

Change subject: core: aaa: Fix multiple misc.authn checking
......................................................................


core: aaa: Fix multiple misc.authn checking

When there is multiple misc.authn extension, with different
authorization providers, it may happen that the first authn choosen will
pass, but the authorization extension will fail. In this situation, we
should continue with other profile, rather then exit.

Change-Id: I65e19529df95354e44d1520b15cc049ffc64ce33
Bug-Url: https://bugzilla.redhat.com/show_bug.cgi?id=1468878
Signed-off-by: Ondra Machacek <omachace at redhat.com>
---
M backend/manager/modules/enginesso/src/main/java/org/ovirt/engine/core/sso/utils/NegotiateAuthUtils.java
1 file changed, 23 insertions(+), 18 deletions(-)

Approvals:
  Ondra Machacek: Verified
  Ravi Nori: Looks good to me, but someone else must approve
  Jenkins CI: Passed CI tests
  Moti Asayag: Looks good to me, approved



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

Gerrit-MessageType: merged
Gerrit-Change-Id: I65e19529df95354e44d1520b15cc049ffc64ce33
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: Moti Asayag <masayag 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>