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