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 <[email protected]>
---
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 <[email protected]>
Gerrit-Reviewer: Oved Ourfali <[email protected]>
Gerrit-Reviewer: Yair Zaslavsky <[email protected]>
Gerrit-Reviewer: Yaniv Dary <[email protected]>
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to