Change in ovirt-reports[master]: reports: open reports via webadmin - host or vm gives except...

[ydary at redhat.com]

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 at 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 at redhat.com>
Gerrit-Reviewer: Oved Ourfali <oourfali at redhat.com>
Gerrit-Reviewer: Yair Zaslavsky <yzaslavs at redhat.com>
Gerrit-Reviewer: Yaniv Dary <ydary at redhat.com>