Ravi Nori has posted comments on this change.

Change subject: aaa: Add support for rest api Basic auth
......................................................................


Patch Set 2:

(1 comment)

http://gerrit.ovirt.org/#/c/37786/2/backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-deployment-structure.xml
File 
backend/manager/modules/restapi/webapp/src/main/webapp/WEB-INF/jboss-deployment-structure.xml:

Line 34:       <module name="org.ovirt.engine.api.restapi-definition" 
annotations="true" services="import"/>
Line 35:       <module name="org.ovirt.engine.api.restapi-jaxrs" 
annotations="true" services="import"/>
Line 36:       <module name="org.ovirt.engine.api.restapi-types" 
annotations="true" services="import"/>
Line 37:       <module name="org.ovirt.engine.core.aaa"/>
Line 38:       <module name="org.ovirt.engine.api.ovirt-engine-extensions-api"/>
> but the deserialization should take the context classpath, it is done in th
Looks like some class loading issue with the way jackson is looking for classes

This works

      ClassLoader loader = Thread.currentThread().getContextClassLoader();
      Thread.currentThread().setContextClassLoader(getClass().getClassLoader());

      Map<String, Object> payload;
      try {
          payload = deserializer.deserialize(str, HashMap.class);
      } finally {
          Thread.currentThread().setContextClassLoader(loader);
      }
Line 39: 
Line 40:     </dependencies>
Line 41:   </deployment>
Line 42: 


-- 
To view, visit http://gerrit.ovirt.org/37786
To unsubscribe, visit http://gerrit.ovirt.org/settings

Gerrit-MessageType: comment
Gerrit-Change-Id: Ib5f6975f2d306a4dc2d81b795ab4905e5d3281a1
Gerrit-PatchSet: 2
Gerrit-Project: ovirt-engine
Gerrit-Branch: master
Gerrit-Owner: Ravi Nori <[email protected]>
Gerrit-Reviewer: Alon Bar-Lev <[email protected]>
Gerrit-Reviewer: Ravi Nori <[email protected]>
Gerrit-Reviewer: [email protected]
Gerrit-Reviewer: oVirt Jenkins CI Server
Gerrit-HasComments: Yes
_______________________________________________
Engine-patches mailing list
[email protected]
http://lists.ovirt.org/mailman/listinfo/engine-patches

Reply via email to