eclipse / microprofile-conference

Microprofile.io Demo Code - Web Services Conference Application
http://microprofile.io/
Apache License 2.0
117 stars 116 forks source link

Updates for when all microservices are running on Liberty #162

Closed aguibert closed 7 years ago

aguibert commented 7 years ago

Fixes 2 issues when running all microservices with Liberty.

Issue 1

Speakers UI does not load because CORS filter did not apply Access-Control-Allow-Origin header. Fixed by including CORS filter in the jax-rs app (i.e. let jax-rs figure out app packaging on its own)

Issue 2

NPE from JAX-RS. Fixed by upgrading to Liberty 17.0.0.2)

Caused by: java.lang.NullPointerException
    at io.microprofile.showcase.speaker.rest.ResourceSpeaker.getUri(ResourceSpeaker.java:115)
    at io.microprofile.showcase.speaker.rest.ResourceSpeaker.addHyperMedia(ResourceSpeaker.java:100)
    at io.microprofile.showcase.speaker.rest.ResourceSpeaker$$Lambda$8.0000000003BD3D70.accept(Unknown Source)
    at java.util.ArrayList.forEach(ArrayList.java:1260)
    at io.microprofile.showcase.speaker.rest.ResourceSpeaker.retrieveAll(ResourceSpeaker.java:56)
    at io.microprofile.showcase.speaker.rest.ResourceSpeaker$Proxy$_$$_WeldClientProxy.retrieveAll(Unknown Source)
    at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
    at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:95)
    at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:55)
    at java.lang.reflect.Method.invoke(Method.java:507)
    at com.ibm.ws.jaxrs20.server.LibertyJaxRsServerFactoryBean.performInvocation(LibertyJaxRsServerFactoryBean.java:651)
    at com.ibm.ws.jaxrs20.server.LibertyJaxRsInvoker.performInvocation(LibertyJaxRsInvoker.java:115)
    at org.apache.cxf.service.invoker.AbstractInvoker.invoke(AbstractInvoker.java:97)
    ... 35 more