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

Andriy Redko commented on CXF-6132:
-----------------------------------

Hi Sergey,

Sure, there are two issues. One is related to 
CXFNonSpringJaxrsServlet::isIgnoreApplicationPath, which basically ignores the 
@ApplicationPath if it is not said explicitly (init parameter). I think, the 
logic should be inverted here:
{code}
return ignoreParam != null && MessageUtils.isTrue(ignoreParam)
{code}

Second issue is a bit more tricky and related to the 
CXFNonSpringJaxrsServlet::ApplicationImpl class (I know, it is private, should 
be fine) which you have introduced. I run into this issues while prototyping 
with Tomcat 8.0.17. For some reasons, container (Tomcat) classpath scanner 
detects it and sends it over to  ServletContextInitializer instance, along with 
other detected applications. The simplest solution I came up with is to ignore 
the application classes defined internally by Apache CXF. 

What do you think about new module? I would like to make initial commit and 
hear your feedback. The work we have done with respect to auto-discovery 
feature turned out to be very handy here. In general, we are able to detect 
resources and providers by scanning classpath, without application instance 
provided.

Thank you!

Best Regards,
    Andriy Redko



> Provide JAX-RS ServletContextInitializer 
> -----------------------------------------
>
>                 Key: CXF-6132
>                 URL: https://issues.apache.org/jira/browse/CXF-6132
>             Project: CXF
>          Issue Type: New Feature
>          Components: JAX-RS
>            Reporter: Sergey Beryozkin
>            Assignee: Andriy Redko
>             Fix For: 3.0.4, 3.1.0
>
>
> This will offer an advanced support for the auto-discovery of JAX-RS 
> Application, root resources and providers in OSGI in combination with 
> pax-web-jetty.
> Options:
> - dynamically register the implementation as OSGI service
> - ship a static implementation
> [1] 
> https://github.com/ops4j/org.ops4j.pax.web/blob/master/pax-web-jetty/src/main/java/org/ops4j/pax/web/service/jetty/internal/JettyServerWrapper.java#L303
> [2] 
> http://docs.oracle.com/javaee/7/api/javax/servlet/ServletContainerInitializer.html



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

Reply via email to