Oved Ourfali has submitted this change and it was merged.
Change subject: aaa: Pass password across filters so it will be known during
LoginUserCommand
......................................................................
aaa: Pass password across filters so it will be known during LoginUserCommand
As basic authentication passed auth record to Login filter, the passsword
used for authentication was not passed, hence was not set in the session data container.
Change-Id: Ie48f8f1408b17882337f4fd0a8c38f7829472cf2
Bug-Url:
https://bugzilla.redhat.com/1141541
Topic: AAA
Signed-off-by: Yair Zaslavsky <yzaslavs(a)redhat.com>
---
M
backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/BasicAuthenticationFilter.java
M
backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/FiltersHelper.java
M
backend/manager/modules/aaa/src/main/java/org/ovirt/engine/core/aaa/filters/LoginFilter.java
M
backend/manager/modules/common/src/main/java/org/ovirt/engine/core/common/action/LoginUserParameters.java
4 files changed, 7 insertions(+), 2 deletions(-)
Approvals:
Alon Bar-Lev: Looks good to me, but someone else must approve
Oved Ourfali: Looks good to me, approved
Yair Zaslavsky: Verified
--
To view, visit
http://gerrit.ovirt.org/34818
To unsubscribe, visit
http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie48f8f1408b17882337f4fd0a8c38f7829472cf2
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Alon Bar-Lev <alonbl(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: automation(a)ovirt.org