Change in ovirt-reports[master]: reports: open reports via webadmin - host or vm gives except...
Yaniv Dary has submitted this change and it was merged.
Change subject: reports: open reports via webadmin - host or vm gives exception (#855865)
......................................................................
reports: open reports via webadmin - host or vm gives exception (#855865)
https://bugzilla.redhat.com/855865
Exception tends to occur in the reports integration.
The details of this exception is:
org.ovirt.authentication.EngineUserDetails
cannot be cast to
com.jaspersoft.jasperserver.api.metadata.user.domain.User
This shouldn't happen, as jasper is supposed to convert the
EngineUserDetails from the token to its own user details, and leave the
original details in originalAuthentication field.
However, in this patch, when re-authentication is needed, we leave the
current authentication object, setting only our user details. That way,
jasper doesn't need to recreate their own user details every time.
Change-Id: Ie9c2ce53018e2b9d08edb5d0162c584c81ac03d5
Signed-off-by: Oved Ourfali <oourfali(a)redhat.com>
---
M
reports/jars/EngineAuthentication/src/main/java/org/ovirt/authentication/EngineSimplePreAuthFilter.java
1 file changed, 12 insertions(+), 2 deletions(-)
Approvals:
Yaniv Dary: Verified; Looks good to me, approved
--
To view, visit http://gerrit.ovirt.org/7945
To unsubscribe, visit http://gerrit.ovirt.org/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: Ie9c2ce53018e2b9d08edb5d0162c584c81ac03d5
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-reports
Gerrit-Branch: master
Gerrit-Owner: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali(a)redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs(a)redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary(a)redhat.com>