[jira] [Commented] (CXF-7510) SSE integration in CDI abruptly fails with no indication why

[John D. Ament (JIRA)]

    [ 
https://issues.apache.org/jira/browse/CXF-7510?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16196402#comment-16196402
 ] 

John D. Ament commented on CXF-7510:
------------------------------------

I was thinking a little more comprehensive patch:

{code}
diff --git a/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/SseFeature.java 
b/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/SseFeature.java
index 36ea9ed..9193426 100644
--- a/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/SseFeature.java
+++ b/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/SseFeature.java
@@ -30,7 +30,10 @@ import org.apache.cxf.feature.AbstractFeature;
 import org.apache.cxf.jaxrs.provider.ServerProviderFactory;
 import 
org.apache.cxf.jaxrs.sse.atmosphere.SseAtmosphereEventSinkContextProvider;
 
+import javax.enterprise.context.Dependent;
+
 @Provider(value = Type.Feature, scope = Scope.Server)
+@Dependent
 public class SseFeature extends AbstractFeature {
     @Override
     public void initialize(Server server, Bus bus) {
diff --git 
a/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/cdi/SseTransportCustomizationExtension.java
 
b/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/cdi/SseTransportCustomizationExtension.java
index 68af13c..66cb166 100644
--- 
a/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/cdi/SseTransportCustomizationExtension.java
+++ 
b/rt/rs/sse/src/main/java/org/apache/cxf/jaxrs/sse/cdi/SseTransportCustomizationExtension.java
@@ -22,6 +22,9 @@ import 
org.apache.cxf.cdi.extension.JAXRSServerFactoryCustomizationExtension;
 import org.apache.cxf.jaxrs.JAXRSServerFactoryBean;
 import org.apache.cxf.transport.sse.SseHttpTransportFactory;
 
+import javax.enterprise.context.Dependent;
+
+@Dependent
 public class SseTransportCustomizationExtension implements 
JAXRSServerFactoryCustomizationExtension {
     @Override
     public void customize(final JAXRSServerFactoryBean bean) {
diff --git a/rt/rs/sse/src/main/resources/META-INF/beans.xml 
b/rt/rs/sse/src/main/resources/META-INF/beans.xml
index 8d1007d..6b50728 100644
--- a/rt/rs/sse/src/main/resources/META-INF/beans.xml
+++ b/rt/rs/sse/src/main/resources/META-INF/beans.xml
@@ -1,5 +1,6 @@
 <?xml version="1.0"?>
 <beans xmlns="http://java.sun.com/xml/ns/javaee";
-       xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"; 
-       xsi:schemaLocation="http://java.sun.com/xml/ns/javaee 
http://xmlns.jcp.org/xml/ns/javaee/beans_1_1.xsd";>
+   xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance";
+   xsi:schemaLocation="http://java.sun.com/xml/ns/javaee 
http://xmlns.jcp.org/xml/ns/javaee/beans_1_1.xsd";
+    bean-discovery-mode="annotated">
 </beans>
{code}

Also RE the transport customization, I'm not sure that fixes it consistently.  
I do add this servlet param, which I was told on list sets it as well

{code}
if(enableSseTransport) {
            params.add(new WebParam(CXFNonSpringJaxrsServlet.TRANSPORT_ID, 
SseHttpTransportFactory.TRANSPORT_ID));
        }
{code}

> SSE integration in CDI abruptly fails with no indication why
> ------------------------------------------------------------
>
>                 Key: CXF-7510
>                 URL: https://issues.apache.org/jira/browse/CXF-7510
>             Project: CXF
>          Issue Type: Bug
>            Reporter: John D. Ament
>            Assignee: Andriy Redko
>
> https://lists.apache.org/thread.html/10d99c0d013a3d23361a3898171e7bd2b311a509349433db3b6cacaa@%3Cusers.cxf.apache.org%3E
>  has more details
> Intermittently, when bootstrapping CXF + CDI, the integration for SSE will 
> fail.  When it fails, there's no log messages indicating the issue, however 
> attempts to invoke any rest endpoint will give a warning like:
> {code}
> Sep 17, 2017 7:50:33 PM org.apache.cxf.transport.servlet.ServletController
> invoke
> WARNING: Can't find the request for http://my-hostname:4403/rest's Observer
> {code}



--
This message was sent by Atlassian JIRA
(v6.4.14#64029)