Hi Romain,
Well the exception being thrown is not the actual exception.
This was only happening in the MP binary due to the OpenAPI Geronimo
implementation. In the DefaultLoader the ServletContext is injected, but at the
time that the JAX-RS app is deployed, which is in the AfterApplicationCreated
event, the ServletContextHandler does not have a Context anymore so it throws a
IllegalStateException("Didnt find a web context for " + contextClassLoader).
The caller for this is the setApplication of the OpenAPIFilter when we try to
inject it, so that causes the exception we see in the logs.
We never say this in Arquillian testing, because Arquillian waits for the
server to start and then deploys the app. This means we are able to get a
ServletContext from the request in ServletContextHandler, so it works fine.
I believe this is also related with the fix you did here:
https://issues.apache.org/jira/browse/TOMEE-1687
<https://issues.apache.org/jira/browse/TOMEE-1687>
What do you think?
Cheers,
Roberto
> On 4 Oct 2018, at 08:23, Romain Manni-Bucau <rmannibu...@gmail.com> wrote:
>
> @Roberto: do we have a test to reproduce and prevent future regressions? If
> your fix is right the error message is quite unexpected so it would be
> better to ensure we don't break it unintentionnally (a side note is that
> setting a single webapp context and firing an event for all webapps can
> have the same pitfall so it can need to be reworked to ensure we don't hit
> it for ears to not leak context between webapp which would be another nasty
> bug).
>
> Romain Manni-Bucau
> @rmannibucau <https://twitter.com/rmannibucau> | Blog
> <https://rmannibucau.metawerx.net/> | Old Blog
> <http://rmannibucau.wordpress.com> | Github <https://github.com/rmannibucau> |
> LinkedIn <https://www.linkedin.com/in/rmannibucau> | Book
> <https://www.packtpub.com/application-development/java-ee-8-high-performance>
>
>
> Le jeu. 4 oct. 2018 à 01:43, Roberto Cortez <radcor...@yahoo.com.invalid> a
> écrit :
>
>> Hi Cesar,
>>
>> I think I’ve found the issue. Just pushed a fix and now waiting to the
>> build bot to check if everything is ok. I’ll let you know when it is done.
>>
>> Cheers,
>> Roberto
>>
>>> On 3 Oct 2018, at 22:50, Roberto Cortez <radcor...@yahoo.com.INVALID>
>> wrote:
>>>
>>>
>>> Not sure. Let me have a look. On Wednesday, October 3, 2018, 8:56:13
>> PM GMT+1, César Hernández Mendoza <cesargu...@gmail.com> wrote:
>>>
>>> Hi everyone,
>>> I have a simple REST app that is working fine with tomee.version 7.1.0,
>> but tomee:run it's getting issues after updating it to 8.0.0-RC1-SNAPSHOT
>> and also update the javaee-api to 8.0-SNAPSHOT. mvn test runs fine but
>> when I try to run mvn clean install tomee:run I got a cxf error (I attached
>> the full log file to this email).
>>> 03-Oct-2018 11:54:26.368 SEVERE [main]
>> org.apache.cxf.jaxrs.utils.InjectionUtils.reportServerError Method
>> setApplication can not be accessed due to security manager
>> restrictions03-Oct-2018 11:54:26.378 SEVERE [main]
>> org.apache.openejb.observer.ObserverManager$MethodInvocation.invoke error
>> invoking org.apache.tomee.webservices.TomeeJaxRsService@7ea37dbf
>>> Any idea why this is happening with the RC1?
>>> El mar., 2 oct. 2018 a las 14:04, Roberto Cortez
>> (<radcor...@yahoo.com.invalid>) escribió:
>>>
>>> Hi Matthew,
>>>
>>> Feel free to try it. Yes, no planned changes, unless something required
>> fixing.
>>>
>>> Cheers,
>>> Roberto
>>>
>>>> On 2 Oct 2018, at 16:37, Matthew Broadhead
>>>> <matthew.broadh...@nbmlaw.co.uk.INVALID>
>> wrote:
>>>>
>>>> should i try out the RC1 or wait for release candidate? i guess they
>> will be the same unless something needs upgrading?
>>>>
>>>> On 02/10/18 12:18, Roberto Cortez wrote:
>>>>> I guess it would be ASAP :)
>>>>>
>>>>> So if anyone wants to have anything in, now its the time. Speak now or
>> forever hold your peace :)
>>>>>
>>>>> Cheers,
>>>>> Roberto
>>>>>
>>>>>> On 2 Oct 2018, at 00:16, David Blevins <david.blev...@gmail.com>
>> wrote:
>>>>>>
>>>>>> Awesome. If you have a date in mind for the first release attempt,
>> definitely let people know so they can complete anything they may want to
>> get done.
>>>>>>
>>>>>>
>>>>>> --
>>>>>> David Blevins
>>>>>> http://twitter.com/dblevins
>>>>>> http://www.tomitribe.com
>>>>>>
>>>>>>> On Oct 1, 2018, at 2:41 PM, Roberto Cortez
>> <radcor...@yahoo.com.INVALID> wrote:
>>>>>>>
>>>>>>> Hi David,
>>>>>>>
>>>>>>> Thank you.
>>>>>>>
>>>>>>> I do intent to keep pushing with the release. Just doing “this”
>> already had me going through some stuff, which Jon helped me out.
>>>>>>>
>>>>>>> So consider it a first step :)
>>>>>>>
>>>>>>> Cheers,
>>>>>>> Roberto
>>>>>>>
>>>>>>>> On 1 Oct 2018, at 19:28, David Blevins <david.blev...@gmail.com>
>> wrote:
>>>>>>>>
>>>>>>>> I think the spirit behind this is great. However, I don't think it
>> actually helps as the hardest part which will be for you to learn the
>> release process itself. Time wise you should budget 3 weeks. When you do
>> know what you're doing the process usually takes 2 weeks minimum as there's
>> almost always at least one aborted vote and reroll. Given that, the
>> chances of a release by CodeOne are already slim.
>>>>>>>>
>>>>>>>> My advice would be to create a release branch and start cutting
>> actual binaries in nexus.
>>>>>>>>
>>>>>>>> --
>>>>>>>> David Blevins
>>>>>>>> http://twitter.com/dblevins
>>>>>>>> http://www.tomitribe.com
>>>>>>>>
>>>>>>>>> On Oct 1, 2018, at 10:57 AM, Roberto Cortez
>> <radcor...@yahoo.com.INVALID> wrote:
>>>>>>>>>
>>>>>>>>> Hi everyone,
>>>>>>>>>
>>>>>>>>> Following up some of our recent emails, I did push a snapshot
>> version marked as RC of the upcoming TomEE 8 Release.
>>>>>>>>>
>>>>>>>>> Please, find the binaries here:
>>>>>>>>>
>> https://repository.apache.org/content/repositories/snapshots/org/apache/tomee/apache-tomee/8.0.0-RC1-SNAPSHOT/
>> <
>> https://repository.apache.org/content/repositories/snapshots/org/apache/tomee/apache-tomee/8.0.0-RC1-SNAPSHOT/
>>>
>>>>>>>>>
>>>>>>>>> And remember to add the following repository to your project:
>>>>>>>>> <repositories>
>>>>>>>>> <repository>
>>>>>>>>> <id>apache.snapshots.https</id>
>>>>>>>>> <name>Apache Development Snapshot Repository</name>
>>>>>>>>> <url>https://repository.apache.org/content/repositories/snapshots
>> </url>
>>>>>>>>> </repository>
>>>>>>>>> </repositories>
>>>>>>>>>
>>>>>>>>> It should be very close to the actual release. Please, help up
>> take it for a spin and try it out :) Thank you!
>>>>>>>>>
>>>>>>>>> Cheers,
>>>>>>>>> Roberto
>>>>
>>>
>>>
>>>
>>>
>>> --
>>> Atentamente:
>>> César Hernández Mendoza.
>>
>>
