[GitHub] [tomee] jeanouii opened a new pull request #800: See if we can find some other way to fix around construct

2021-05-26 Thread GitBox


jeanouii opened a new pull request #800:
URL: https://github.com/apache/tomee/pull/800


   This reverts commit 8f589318


-- 
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




TOMEE-3752 - Injection of WebServiceContext

2021-05-26 Thread Jonathan Gallimore
Hi All,

I noticed an issue with injecting a WebServiceContext into a field
using @Resource when the endpoint uses CDI interceptors. I've created a PR
with a test and fix. Essentially, the field is being injected into the OWB
proxy, but being accessed from the underlying instance (resulting in a
NullPointerException).

The PR is here: https://github.com/apache/tomee/pull/799 - please do let me
know if you have any feedback.

Thanks

Jon


[GitHub] [tomee] jgallimore opened a new pull request #799: TOMEE-3752 Fix issue injecting WebServiceContext into field when usig…

2021-05-26 Thread GitBox


jgallimore opened a new pull request #799:
URL: https://github.com/apache/tomee/pull/799


   …n CDI interceptor


-- 
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.

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org




TOMEE-3740 - The CompBean - What is it?

2021-05-26 Thread Zowalla, Richard
Hi all,

I took a quick look at our failing openejb-core tests. Most of them
seem to fail due to this change [1] (TOMEE-3740,[2]), in which we
decided to always deploy the CompBean.

AFAIK, we did this on purpose as the TCK expected the CompBean to be
present in its test.

Can someone explain me in short words (or point me to some
documentation which explains it):

- What is the CompBean and why is it necessary to always deploy it? 

- Wondering, if itwouldn't be an option to configure this behaviour via
a system property to avoid deploying the CompBean, if it isn't needed?

Thanks & Gruss
Richard


[1] 
https://github.com/apache/tomee/commit/8f5893189dbc52668354bb0ffd9d8da008a588fb
[2] https://issues.apache.org/jira/browse/TOMEE-3740


smime.p7s
Description: S/MIME cryptographic signature