Sergey Beryozkin created CXF-6587:
-------------------------------------

             Summary: NettyHttpDestination should support Netty-specific 
continuations
                 Key: CXF-6587
                 URL: https://issues.apache.org/jira/browse/CXF-6587
             Project: CXF
          Issue Type: Improvement
          Components: Transports
            Reporter: Sergey Beryozkin
            Priority: Minor


NettyHttpDestination does not try to wrap a Netty specific continuation 
provider:

https://github.com/apache/cxf/blob/master/rt/transports/http-netty/netty-server/src/main/java/org/apache/cxf/transport/http/netty/server/NettyHttpDestination.java#L202

I guess this should be implemented similar to the way JettyHttpDestination does 
it:

https://github.com/apache/cxf/blob/master/rt/transports/http-jetty/src/main/java/org/apache/cxf/transport/http_jetty/JettyHTTPDestination.java#L367

This test checks JAX-RS 2.0 AsyncResponse working on top of Jetty continuations:
https://github.com/apache/cxf/blob/master/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/JAXRSContinuationsTest.java

It uses 
https://github.com/apache/cxf/blob/master/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookContinuationServer.java

to set up the endpoint and this resource to test AsyncResponse:

https://github.com/apache/cxf/blob/master/systests/jaxrs/src/test/java/org/apache/cxf/systest/jaxrs/BookContinuationStore.java

This test should work if pom.xml in systests/jaxrs will have 
cxf-rt-transports-http-jetty replaced with cxf-rt-transports-http-netty-server. 
(Both Jetty and Netty endpoints will need to be tested though - but setting up 
both tests can be dealt with separately)  








--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to