Hey Cesar, I have fixed a compilation issue after the MP work has been merged. Lemme know if you need some help with this and I'm happy to help -- Jean-Louis Monteiro http://twitter.com/jlouismonteiro http://www.tomitribe.com
On Thu, Dec 13, 2018 at 7:10 AM César Hernández Mendoza < [email protected]> wrote: > Hi @Jean-Louis, > > The fix you did solve the general build if the test is skipt, but today I > was starting to work on TOMEE-2357 [1] but when I pull the latest master, > now the mp-rest-jwt example[2] PR you merged today is broken: > > I've attached the entire log. But the first exception present is: > Dec 12, 2018 10:19:24 PM org.apache.webbeans.plugins.PluginLoader startUp > INFO: Adding OpenWebBeansPlugin : [CdiPlugin] > Dec 12, 2018 10:19:25 PM org.apache.webbeans.config.BeansDeployer > deploySingleAnnotatedType > WARNING: Skipping deployment of Class class > org.apache.tomee.microprofile.jwt.config.ConfigurableJWTAuthContextInfo due > to a NoClassDefFoundError: Lorg/eclipse/microprofile/config/Config; > Dec 12, 2018 10:19:25 PM org.apache.openejb.cdi.OpenEJBLifecycle > startApplication > SEVERE: CDI Beans module deployment failed > org.apache.webbeans.exception.WebBeansDeploymentException: > javax.enterprise.inject.UnsatisfiedResolutionException: Api type > [org.apache.tomee.microprofile.jwt.config.ConfigurableJWTAuthContextInfo] > is not found with the qualifiers > Qualifiers: [@javax.enterprise.inject.Default()] > for injection into Field Injection Point, field name : > configurableJWTAuthContextInfo, Bean Owner : [MPJWTFilter, > WebBeansType:MANAGED, Name:null, API > Types:[org.apache.tomee.microprofile.jwt.MPJWTFilter,javax.servlet.Filter,java.lang.Object], > Qualifiers:[javax.enterprise.inject.Default,javax.enterprise.inject.Any]] > at org.apache.webbeans.config.BeansDeployer.deploy(BeansDeployer.java:356) > at > org.apache.openejb.cdi.OpenEJBLifecycle.startApplication(OpenEJBLifecycle.java:196) > at > org.apache.openejb.cdi.ThreadSingletonServiceImpl.initialize(ThreadSingletonServiceImpl.java:229) > > I notice that the PR for TOMEE-2247 - MicroProfile JWT Propagation 1.1 [3] > was merged a couple days ago so I tried to upgrade in the mp-rest-jwt pom > file from <mp-jwt.version>1.0</mp-jwt.version> > to <mp-jwt.version>1.1.1</mp-jwt.version> but still the example test > failed. > > At this point, I'm blocked to move forward TOMEE-2357 since I don't know > the internals of the JWT implementation we currently have on master. > Any help or pointers are more than welcome :) > > [1] TOMEE-2357 - MicroProfile JWT @RolesAllowed is been applied with an > all or nothing policy > [2] https://github.com/apache/tomee/tree/master/examples/mp-rest-jwt > [3] https://github.com/apache/tomee/pull/173 > > El mié., 12 dic. 2018 a las 18:39, Jean-Louis Monteiro (< > [email protected]>) escribió: > >> Of course >> >> Le mer. 12 déc. 2018 à 23:28, Roberto Cortez <[email protected] >> > >> a écrit : >> >> > Yeah, sorry I got it. >> > >> > > On 12 Dec 2018, at 16:55, Jean-Louis Monteiro < >> [email protected]> >> > wrote: >> > > >> > > Yes, here is the fix >> > > >> > >> https://github.com/apache/tomee/commit/c17b93cb644bc7cbdd463d59ccfbf6a08998553f >> > > >> > > -- >> > > Jean-Louis Monteiro >> > > http://twitter.com/jlouismonteiro >> > > http://www.tomitribe.com >> > > >> > > >> > > On Wed, Dec 12, 2018 at 12:44 PM Roberto Cortez >> > <[email protected]> >> > > wrote: >> > > >> > >> Hey JL, >> > >> >> > >> Were you able to fix the issue? >> > >> >> > >>> On 11 Dec 2018, at 09:03, Jean-Louis Monteiro < >> > [email protected]> >> > >> wrote: >> > >>> >> > >>> I think I found the bug on this one. Let's open a PR and then if you >> > >> could >> > >>> give me your thoughts >> > >>> -- >> > >>> Jean-Louis Monteiro >> > >>> http://twitter.com/jlouismonteiro >> > >>> http://www.tomitribe.com >> > >>> >> > >>> >> > >>> On Mon, Dec 10, 2018 at 5:27 PM Jean-Louis Monteiro < >> > >>> [email protected]> wrote: >> > >>> >> > >>>> If you have some pointers, I'm happy to look at the issue. I'll >> > probably >> > >>>> learn something. >> > >>>> -- >> > >>>> Jean-Louis Monteiro >> > >>>> http://twitter.com/jlouismonteiro >> > >>>> http://www.tomitribe.com >> > >>>> >> > >>>> >> > >>>> On Mon, Dec 10, 2018 at 5:27 PM Jonathan Gallimore < >> > >>>> [email protected]> wrote: >> > >>>> >> > >>>>> Ok, sounds like something has gone wrong somewhere. Do you want >> to >> > >> look >> > >>>>> or >> > >>>>> do you want me to? >> > >>>>> >> > >>>>> On Mon, 10 Dec 2018, 16:21 Jean-Louis Monteiro < >> > >> [email protected] >> > >>>>> wrote: >> > >>>>> >> > >>>>>> In core yes for sure, but I've seen in the tomee-embedded module >> as >> > >> well >> > >>>>>> (need to check this one though) >> > >>>>>> -- >> > >>>>>> Jean-Louis Monteiro >> > >>>>>> http://twitter.com/jlouismonteiro >> > >>>>>> http://www.tomitribe.com >> > >>>>>> >> > >>>>>> >> > >>>>>> On Mon, Dec 10, 2018 at 5:05 PM Jonathan Gallimore < >> > >>>>>> [email protected]> wrote: >> > >>>>>> >> > >>>>>>> Was it in openejb-core or the jwt TCK? >> > >>>>>>> >> > >>>>>>> I'm sure the latter was working. >> > >>>>>>> >> > >>>>>>> On Mon, 10 Dec 2018, 15:56 Jean-Louis Monteiro < >> > >>>>> [email protected] >> > >>>>>>> wrote: >> > >>>>>>> >> > >>>>>>>> I'll pull from master and rebuild the whole thing here >> > >>>>>>>> Gimme 30 minutes >> > >>>>>>>> >> > >>>>>>>> -- >> > >>>>>>>> Jean-Louis Monteiro >> > >>>>>>>> http://twitter.com/jlouismonteiro >> > >>>>>>>> http://www.tomitribe.com >> > >>>>>>>> >> > >>>>>>>> >> > >>>>>>>> On Mon, Dec 10, 2018 at 4:55 PM Jonathan Gallimore < >> > >>>>>>>> [email protected]> wrote: >> > >>>>>>>> >> > >>>>>>>>> It _was_ working ok. Where are you seeing that exception? >> > >>>>>>>>> >> > >>>>>>>>> On Mon, 10 Dec 2018, 15:52 Jean-Louis Monteiro < >> > >>>>>>> [email protected] >> > >>>>>>>>> wrote: >> > >>>>>>>>> >> > >>>>>>>>>> This is after I merged the PR from Roberto on JWT yes. >> > >>>>>>>>>> >> > >>>>>>>>>> Caused by: java.lang.ClassCastException: Cannot cast >> > >>>>>>>>> com.sun.proxy.$Proxy55 >> > >>>>>>>>>> to java.security.Principal >> > >>>>>>>>>> at java.lang.Class.cast(Class.java:3369) >> > >>>>>>>>>> at >> > >>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>> >> > >>>>>>>> >> > >>>>>>> >> > >>>>>> >> > >>>>> >> > >> >> > >> org.apache.openejb.cdi.ManagedSecurityService.<init>(ManagedSecurityService.java:64) >> > >>>>>>>>>> at >> sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native >> > >>>>>>>> Method) >> > >>>>>>>>>> at >> > >>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>> >> > >>>>>>>> >> > >>>>>>> >> > >>>>>> >> > >>>>> >> > >> >> > >> sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) >> > >>>>>>>>>> at >> > >>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>> >> > >>>>>>>> >> > >>>>>>> >> > >>>>>> >> > >>>>> >> > >> >> > >> sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) >> > >>>>>>>>>> at >> > >>>>> java.lang.reflect.Constructor.newInstance(Constructor.java:423) >> > >>>>>>>>>> at >> > >>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>> >> > >>>>>>>> >> > >>>>>>> >> > >>>>>> >> > >>>>> >> > >> >> > >> org.apache.webbeans.config.WebBeansContext.createInstance(WebBeansContext.java:440) >> > >>>>>>>>>> -- >> > >>>>>>>>>> Jean-Louis Monteiro >> > >>>>>>>>>> http://twitter.com/jlouismonteiro >> > >>>>>>>>>> http://www.tomitribe.com >> > >>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>>> On Mon, Dec 10, 2018 at 4:51 PM Jonathan Gallimore < >> > >>>>>>>>>> [email protected]> wrote: >> > >>>>>>>>>> >> > >>>>>>>>>>> What's the context? JWT? >> > >>>>>>>>>>> >> > >>>>>>>>>>> Jon >> > >>>>>>>>>>> >> > >>>>>>>>>>> On Mon, 10 Dec 2018, 15:28 Jean-Louis Monteiro < >> > >>>>>>>>> [email protected] >> > >>>>>>>>>>> wrote: >> > >>>>>>>>>>> >> > >>>>>>>>>>>> Hey Jon, >> > >>>>>>>>>>>> >> > >>>>>>>>>>>> I think you worked on this issue. What's the status? >> > >>>>>>>>>>>> Do you have some inputs? >> > >>>>>>>>>>>> -- >> > >>>>>>>>>>>> Jean-Louis Monteiro >> > >>>>>>>>>>>> http://twitter.com/jlouismonteiro >> > >>>>>>>>>>>> http://www.tomitribe.com >> > >>>>>>>>>>>> >> > >>>>>>>>>>> >> > >>>>>>>>>> >> > >>>>>>>>> >> > >>>>>>>> >> > >>>>>>> >> > >>>>>> >> > >>>>> >> > >>>> >> > >> >> > >> >> > >> > >> > > > -- > Atentamente: > César Hernández Mendoza. >
