Let me run a check with your PR. If it looks ok, I'll just merge it :) Jon
On Thu, Jul 9, 2020 at 3:12 PM Jean-Louis Monteiro <[email protected]> wrote: > Hi Jon, > > Yes I pushed the changes to TCK without the PR being merged. > As soon as this PR gets merged, it should all work again. > > Or I need to revert some stuff on the TCK. > -- > Jean-Louis Monteiro > http://twitter.com/jlouismonteiro > http://www.tomitribe.com > > > On Thu, Jul 9, 2020 at 3:33 PM Jonathan Gallimore < > [email protected]> wrote: > > > Hey JL > > > > I'm good for you to merge it. Quick question though - I'm running the > > servlet TCK tests at the moment, and nothing is passing. Looks like the > > issue is this: > > </pre><p><b>Root Cause</b></p><pre>java.lang.ClassNotFoundException: > > org.apache.tomee.security.provider.TomEESecurityAuthConfigProvider > > java.net.URLClassLoader.findClass(URLClassLoader.java:382) > > java.lang.ClassLoader.loadClass(ClassLoader.java:418) > > java.lang.ClassLoader.loadClass(ClassLoader.java:351) > > java.lang.Class.forName0(Native Method) > > java.lang.Class.forName(Class.java:264) > > > > > > > org.apache.catalina.authenticator.jaspic.AuthConfigFactoryImpl.createAuthConfigProvider(AuthConfigFactoryImpl.java:133) > > > > > > > org.apache.catalina.authenticator.jaspic.AuthConfigFactoryImpl.doRegisterConfigProvider(AuthConfigFactoryImpl.java:111) > > > > > > > org.apache.catalina.authenticator.jaspic.AuthConfigFactoryImpl.loadPersistentRegistrations(AuthConfigFactoryImpl.java:347) > > > > > > > org.apache.catalina.authenticator.jaspic.AuthConfigFactoryImpl.<init>(AuthConfigFactoryImpl.java:68) > > sun.reflect.NativeConstructorAccessorImpl.newInstance0(Native > > Method) > > > > > > > sun.reflect.NativeConstructorAccessorImpl.newInstance(NativeConstructorAccessorImpl.java:62) > > > > > > > sun.reflect.DelegatingConstructorAccessorImpl.newInstance(DelegatingConstructorAccessorImpl.java:45) > > java.lang.reflect.Constructor.newInstance(Constructor.java:423) > > > > > > > javax.security.auth.message.config.AuthConfigFactory$1.run(AuthConfigFactory.java:76) > > > > > > > javax.security.auth.message.config.AuthConfigFactory$1.run(AuthConfigFactory.java:67) > > java.security.AccessController.doPrivileged(Native Method) > > > > > > > javax.security.auth.message.config.AuthConfigFactory.getFactory(AuthConfigFactory.java:66) > > > > > > > org.apache.catalina.authenticator.AuthenticatorBase.findJaspicProvider(AuthenticatorBase.java:1382) > > > > > > > org.apache.catalina.authenticator.AuthenticatorBase.getJaspicProvider(AuthenticatorBase.java:1375) > > > > > > > org.apache.catalina.authenticator.AuthenticatorBase.invoke(AuthenticatorBase.java:532) > > > > > > > org.apache.catalina.valves.ErrorReportValve.invoke(ErrorReportValve.java:92) > > > > > > > org.apache.tomee.catalina.OpenEJBSecurityListener$RequestCapturer.invoke(OpenEJBSecurityListener.java:97) > > > > > > > org.apache.catalina.valves.AbstractAccessLogValve.invoke(AbstractAccessLogValve.java:690) > > > > > org.apache.catalina.connector.CoyoteAdapter.service(CoyoteAdapter.java:343) > > > > > org.apache.coyote.http11.Http11Processor.service(Http11Processor.java:373) > > > > > > > org.apache.coyote.AbstractProcessorLight.process(AbstractProcessorLight.java:65) > > > > > > > org.apache.coyote.AbstractProtocol$ConnectionHandler.process(AbstractProtocol.java:868) > > > > org.apache.tomcat.util.net > > .NioEndpoint$SocketProcessor.doRun(NioEndpoint.java:1589) > > > > org.apache.tomcat.util.net > > .SocketProcessorBase.run(SocketProcessorBase.java:49) > > > > > > > java.util.concurrent.ThreadPoolExecutor.runWorker(ThreadPoolExecutor.java:1149) > > > > > > > java.util.concurrent.ThreadPoolExecutor$Worker.run(ThreadPoolExecutor.java:624) > > > > > > > org.apache.tomcat.util.threads.TaskThread$WrappingRunnable.run(TaskThread.java:61) > > java.lang.Thread.run(Thread.java:748) > > > > I assume that this is overlapping on some of the work you've done. You > got > > any hints that might help me out? > > > > Cheers > > > > Jon > > > > On Thu, Jul 9, 2020 at 8:59 AM Jean-Louis Monteiro < > > [email protected]> > > wrote: > > > > > Hi all, > > > > > > Time to step back and give you some updates. > > > So far we are at > > > 82 PASSED > > > 4 FAILED > > > > > > Of the 4, 2 are under investigation on the Jakarta TCK side. > > > 1 is related to the security manager and another one to JACC > permissions. > > > > > > It may take some time to get the 4 fully completed, so I suggest we > leave > > > them until we hear back from Jakarta and we can fix them later. > > > > > > We are kinda in Pareto mode (20% of effort to cover 80% and the > remaining > > > may require too much effort for now). > > > > > > The PR is still up for review. There are a lot of todos in there and > > > requires a lot more testing (unit and integration). TCK coverage is not > > > enough. > > > > > > While we can dedicate more time to harden this part, any objection if > we > > > merge the PR? > > > > > > -- > > > Jean-Louis Monteiro > > > http://twitter.com/jlouismonteiro > > > http://www.tomitribe.com > > > > > >
