[ 
https://issues.apache.org/jira/browse/CXF-8193?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel
 ]

Freeman Yue Fang updated CXF-8193:
----------------------------------
    Description: 
9.4.25 will break JAXRSRequestDispatcherTest.testGetBookHTMLInclude, this is 
caused by the recent change in org.eclipse.jetty.server.Dispatcher.forward 
method.

However, I took a close look at the 
JAXRSRequestDispatcherTest.testGetBookHTMLInclude, the server bean has 
configuration like
{code}
<property name="includeResource" value="true"/>
{code}

So I think the test servlet configuration should also have parameter like
{code}
<init-param>
            <param-name>redirect-with-include</param-name>
            <param-value>true</param-value>
</init-param>
{code}
accordingly, and this can fix the failure test.

Seems the change in Jetty just expose a test configuration issue in CXF


  was:
9.4.25 will break JAXRSRequestDispatcherTest.testGetBookHTMLInclude, this is 
caused by the recent change in org.eclipse.jetty.server.Dispatcher.forward 
method.

However, I took a close look at the 
JAXRSRequestDispatcherTest.testGetBookHTMLInclude, the server bean has 
configuration like
{code}
<property name="includeResource" value="true"/>
{code}

So I think the servlet configuration should also have parameter like
{code}
<init-param>
            <param-name>redirect-with-include</param-name>
            <param-value>true</param-value>
</init-param>
{code}
accordingly, and this can fix the failure test



> upgrade to jetty 9.4.25.v20191220
> ---------------------------------
>
>                 Key: CXF-8193
>                 URL: https://issues.apache.org/jira/browse/CXF-8193
>             Project: CXF
>          Issue Type: Improvement
>            Reporter: Freeman Yue Fang
>            Assignee: Freeman Yue Fang
>            Priority: Major
>             Fix For: 3.4.0, 3.3.6
>
>
> 9.4.25 will break JAXRSRequestDispatcherTest.testGetBookHTMLInclude, this is 
> caused by the recent change in org.eclipse.jetty.server.Dispatcher.forward 
> method.
> However, I took a close look at the 
> JAXRSRequestDispatcherTest.testGetBookHTMLInclude, the server bean has 
> configuration like
> {code}
> <property name="includeResource" value="true"/>
> {code}
> So I think the test servlet configuration should also have parameter like
> {code}
> <init-param>
>             <param-name>redirect-with-include</param-name>
>             <param-value>true</param-value>
> </init-param>
> {code}
> accordingly, and this can fix the failure test.
> Seems the change in Jetty just expose a test configuration issue in CXF



--
This message was sent by Atlassian Jira
(v8.3.4#803005)

Reply via email to