Re: TOMEE-2201 NPE when injecting Application

2018-07-17 Thread Thiago Veronezi
All good. Waiting for the build to pass (or not). Tx Romain. []s, Thiago. On Mon, Jul 16, 2018 at 1:37 PM, Romain Manni-Bucau wrote: > https://git-wip-us.apache.org/repos/asf?p=tomee.git;a=blob; > f=server/openejb-cxf-rs/src/main/java/org/apache/openejb/ > server/cxf/rs/Contexts.java;h=e1a541150

Re: TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Romain Manni-Bucau
https://git-wip-us.apache.org/repos/asf?p=tomee.git;a=blob;f=server/openejb-cxf-rs/src/main/java/org/apache/openejb/server/cxf/rs/Contexts.java;h=e1a54115070e660581d8e198ee6b7aea20ce64ce;hb=HEAD Ps: please also fix it on branch 8 if not already done Le lun. 16 juil. 2018 19:08, Thiago Veronezi

Re: TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Thiago Veronezi
"contexts.find"? What do you mean? I was thinking on custom context providers, but that goes to the " ThreadLocalContextManager.PROVIDERS" option. I'm wondering the "ThreadLocalContextManager.OTHERS" is ever (supposed to be) used. []s, Thiago. On Mon, Jul 16, 2018 at 12:30 PM, Romain Manni-Buc

Re: TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Romain Manni-Bucau
guess the contexts.find in cxf-rs module? Romain Manni-Bucau @rmannibucau | Blog | Old Blog | Github | LinkedIn | Boo

Re: TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Thiago Veronezi
Oh yeah. Tx Romain! Added a test case for the Application injection. Now I'm trying to add another case for new the thing being injected is not in the main if chain here... if (Request.class.equals(type)) { reference = new ObjectReference(ThreadLocalContextMan

Re: TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Romain Manni-Bucau
Hi Thiago, you can surely start from https://github.com/apache/tomee/blob/master/server/openejb-cxf-rs/src/test/java/org/apache/openejb/server/cxf/rs/JSonStreamingOutputTest.java and adapt the endpoint to match you use case? Romain Manni-Bucau @rmannibucau | Blo

TOMEE-2201 NPE when injecting Application

2018-07-16 Thread Thiago Veronezi
Hi guys, This is the cleaner version of the bug I mentioned last week. https://github.com/apache/tomee/pull/135 Can you guys help me finding a way to create a unit test for it? Thanks, Thiago.