-
Notifications
You must be signed in to change notification settings - Fork 697
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
TOMEE-4424 - improved JAX-RS request matching if a welcome-file is defined #1600
Conversation
...t/java/org/apache/openejb/arquillian/tests/jaxrs/servlets/welcomefile/MyFiletypeServlet.java
Show resolved
Hide resolved
.../src/test/java/org/apache/openejb/arquillian/tests/jaxrs/servlets/welcomefile/MyRestApi.java
Show resolved
Hide resolved
...t/java/org/apache/openejb/arquillian/tests/jaxrs/servlets/welcomefile/MyRestApplication.java
Show resolved
Hide resolved
.../apache/openejb/arquillian/tests/jaxrs/servlets/welcomefile/ServletsWithWelcomeFileTest.java
Show resolved
Hide resolved
I triggered a rebuild (but the failing tests might be actual regression, so just to be sure). Can you do a JAX-RS TCK run as well? |
Ran the JAX-RS TCK now:
This did definitely break something, I'll look at this |
Looks like the same tests are failing on main now too (see full build and tck), I suspect that a change in CXF is causing these failures. Will keep this as a draft until these are resolved |
Let's see if the adaptions in https://github.com/apache/tomee/tree/cxf-4.1.x-jsonp-fixes will fix the TCK/build on the manual full build, so we get it sorted out. |
So let's see how it goes https://ci-builds.apache.org/job/Tomee/job/pull-request-manual/153/ |
the 2 failing tests are unrelated, see 9a68d9f |
See https://issues.apache.org/jira/browse/TOMEE-4424
Fulll build is running: https://ci-builds.apache.org/job/Tomee/view/tomee-10.x/job/pull-request-manual/146/