[GitHub] [tomee] rzo1 opened a new pull request, #989: TOMEE-4014 - Porting the patch from 8.x to 9.x

2022-12-15 Thread GitBox
rzo1 opened a new pull request, #989: URL: https://github.com/apache/tomee/pull/989 TOMEE-4014 alternative approach that does not touch ServerInfo.properties in catalina.jar, and works around the reflection issue with the final modifier Cherry-picked from

[GitHub] [tomee] dblevins merged pull request #987: Make sure providers aren't scanned if Application returns classes or singletons.

2022-12-15 Thread GitBox
dblevins merged PR #987: URL: https://github.com/apache/tomee/pull/987 -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: dev-unsubscr...@tomee.apache.org

[GitHub] [tomee] rzo1 commented on pull request #988: TOMEE-4127 - CXF 3.5.5

2022-12-15 Thread GitBox
rzo1 commented on PR #988: URL: https://github.com/apache/tomee/pull/988#issuecomment-1353612061 No intention to merge this eagerly in. If we want to be save, I can change it to "draft", so it does not accidently get merged in. Currently, I am waiting for the normal PR CI results

[GitHub] [tomee] dblevins commented on pull request #988: TOMEE-4127 - CXF 3.5.5

2022-12-15 Thread GitBox
dblevins commented on PR #988: URL: https://github.com/apache/tomee/pull/988#issuecomment-1353594469 We're currently waiting on TCK results for latest changes against CXF 3.5.4. The expectation is that the TCK should now be passing and once Jean-Louis' AroundConstruct PR is merged, we are

[GitHub] [tomee] rzo1 opened a new pull request, #988: TOMEE-4127 - CXF 3.5.5

2022-12-15 Thread GitBox
rzo1 opened a new pull request, #988: URL: https://github.com/apache/tomee/pull/988 as the title says -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe,

Re: JAX-RS and Reworking Open Tracing integration

2022-12-15 Thread Jean-Louis Monteiro
Let's see what this one looks like https://github.com/apache/tomee/pull/987 I reverted the revert in there lol -- Jean-Louis Monteiro http://twitter.com/jlouismonteiro http://www.tomitribe.com On Thu, Dec 15, 2022 at 2:01 PM Jean-Louis Monteiro < jlmonte...@tomitribe.com> wrote: > Hi David, >

[GitHub] [tomee] jeanouii opened a new pull request, #987: Make sure providers aren't scanned if Application returns classes or singletons.

2022-12-15 Thread GitBox
jeanouii opened a new pull request, #987: URL: https://github.com/apache/tomee/pull/987 Register OpenTracing providers during MP initialization -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to

Re: JAX-RS and Reworking Open Tracing integration

2022-12-15 Thread Jean-Louis Monteiro
Hi David, Sorry for the bad revert and introducing a regression. I can't recall why I did it at that time. I'll pause the @AroundConstruct implementation. The PR exists and is mostly green but one test https://ci-builds.apache.org/job/Tomee/job/master-pull-request/72/ -->