On 12/12/2007, Alin Dreghiciu <[EMAIL PROTECTED]> wrote: > > I committed the changes. It should work now :)
woohoo - TCK passes! On Dec 12, 2007 1:06 PM, Alin Dreghiciu <[EMAIL PROTECTED]> wrote: > > I was already working on this. For the exception bellow I already have > > a fix. Is a mater of couple of hours till I commit it. > > Pretty close now :) > > > > > > On Dec 12, 2007 12:47 PM, Stuart McCulloch <[EMAIL PROTECTED]> > wrote: > > > On 12/12/2007, Stuart McCulloch <[EMAIL PROTECTED]> wrote: > > > > > > > On 12/12/2007, Stuart McCulloch <[EMAIL PROTECTED] > > wrote: > > > > > > > > > On 12/12/2007, Alin Dreghiciu <[EMAIL PROTECTED]> wrote: > > > > > > > > > > > Thanx Stuart, > > > > > > > > > > > > Can you figure out what's the test about. From the exception I > do not > > > > > > know what they expect. > > > > > > > > > > > > > > > basically the test registers one servlet under a HttpContext, > > > unregisters it, > > > > > registers a different servlet, and then tries to access a resource > from > > > the > > > > > second servlet - it fails because it can't read the second > resource... > > > HTH > > > > > > > > > > > > FYI, the issue is with the reusing of contexts and registrations in > the > > > new code > > > > - there's already an HttpServiceContext in the map, but the new > > > registrations > > > > aren't merged in with the cached HttpServiceContext > > > > > > stopping the old context from being removed when there are no > registrations > > > (in StartedHttpService.unregister) works around this issue, but then > another > > > new exception appears, which may also be related: > > > > > > [java] Log: During execution of BundleStopping > > > org.osgi.framework.BundleException : Exception in > > > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start() of bundle 30. > > > [java] org.osgi.framework.BundleException: Exception in > > > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start() of bundle 30. > > > [java] at > > > > org.eclipse.osgi.framework.internal.core.BundleContextImpl.startActivator( > BundleContextImpl.java:1018) > > > [java] at > > > org.eclipse.osgi.framework.internal.core.BundleContextImpl.start( > BundleContextImpl.java > > > :974) > > > [java] at > > > org.eclipse.osgi.framework.internal.core.BundleHost.startWorker( > BundleHost.java:346) > > > [java] at > > > org.eclipse.osgi.framework.internal.core.AbstractBundle.start( > AbstractBundle.java:260) > > > [java] at > > > org.eclipse.osgi.framework.internal.core.AbstractBundle.start( > AbstractBundle.java:252) > > > [java] at > > > org.osgi.test.cases.http.tbc.HttpTestBundle1.BundleStopping( > HttpTestBundle1.java:284) > > > > > > [java] at sun.reflect.NativeMethodAccessorImpl.invoke0 > (Native > > > Method) > > > [java] at > > > sun.reflect.NativeMethodAccessorImpl.invoke( > NativeMethodAccessorImpl.java:64) > > > [java] at sun.reflect.DelegatingMethodAccessorImpl.invoke > > > (DelegatingMethodAccessorImpl.java:43) > > > [java] at java.lang.reflect.Method.invoke(Method.java:615) > > > [java] at > > > org.osgi.test.cases.util.DefaultTestBundleControl.run( > DefaultTestBundleControl.java:271) > > > [java] at java.lang.Thread.run(Thread.java:801) > > > [java] Caused by: org.osgi.service.http.NamespaceException: alias > is > > > already in use > > > [java] at > > > org.ops4j.pax.web.service.internal.RegistrationsImpl.validateAlias > > > (RegistrationsImpl.java:166) > > > [java] at > > > > org.ops4j.pax.web.service.internal.RegistrationsImpl.validateRegisterServletArguments > (RegistrationsImpl.java:125) > > > [java] at > > > org.ops4j.pax.web.service.internal.RegistrationsImpl.registerServlet > > > (RegistrationsImpl.java:67) > > > [java] at > > > org.ops4j.pax.web.service.internal.StartedHttpService.registerServlet( > StartedHttpService.java:101) > > > [java] at > > > org.ops4j.pax.web.service.internal.HttpServiceProxy.registerServlet > > > (HttpServiceProxy.java:50) > > > [java] at > > > org.osgi.test.cases.http.tbc.tb1.HttpTestBundle2.start( > HttpTestBundle2.java:30) > > > [java] at > > > org.eclipse.osgi.framework.internal.core.BundleContextImpl$2.run( > BundleContextImpl.java > > > :999) > > > [java] at > > > java.security.AccessController.doPrivileged(AccessController.java:242) > > > [java] at > > > > org.eclipse.osgi.framework.internal.core.BundleContextImpl.startActivator( > BundleContextImpl.java:993) > > > [java] ... 11 more > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > Alin > > > > > > > > > > > > On Dec 12, 2007 5:16 AM, Stuart McCulloch > > > <[EMAIL PROTECTED]> wrote: > > > > > > > On 12/12/2007, Alin Dreghiciu < [EMAIL PROTECTED]> wrote: > > > > > > > > > > > > > > > Hi, > > > > > > > > > > > > > > > > I committed quite a big change required by PAXWEB-21 and > > > PAXWEB-55, > > > > > > > > related to isolation of servlet contexts per http context. > > > > > > > > My tests were successful but I did not had a full blown app > to > > > test with. > > > > > > > > > > > > > > > > Stuart, can you run the TCK? > > > > > > > > > > > > > > I just fixed a NPE that was blocking the TCK, but now get a > new > > > > > > > FileNotFoundException: > > > > > > > > > > > > > > [java] Log: During execution of > SimpleServletUnregistration > > > > > > > java.io.FileNotFoundException : > > > http://localhost:8083/tc2servlet?TestCase=2 > > > > > > > [java] java.io.FileNotFoundException: > > > > > > > http://localhost:8083/tc2servlet?TestCase=2 > > > > > > > [java] at > > > > > > > sun.net.www.protocol.http.HttpURLConnection.getInputStream > > > (HttpURLConnection.java:1177) > > > > > > > [java] at java.net.URL.openStream(URL.java:1041) > > > > > > > [java] at > > > > > > > > > > > org.osgi.test.cases.http.tbc.HttpTestBundle1.SimpleServletUnregistration > > > > > > > (HttpTestBundle1.java :130) > > > > > > > [java] at > > > sun.reflect.NativeMethodAccessorImpl.invoke0(Native > > > > > > > Method) > > > > > > > [java] at > > > > > > > > > > sun.reflect.NativeMethodAccessorImpl.invoke( > NativeMethodAccessorImpl.java:64) > > > > > > > [java] at > > > > > > > > > > sun.reflect.DelegatingMethodAccessorImpl.invoke( > DelegatingMethodAccessorImpl.java:43) > > > > > > > [java] at java.lang.reflect.Method.invoke(Method.java > :615) > > > > > > > [java] at > > > org.osgi.test.cases.util.DefaultTestBundleControl.run > > > > > > > (DefaultTestBundleControl.java:271) > > > > > > > [java] at java.lang.Thread.run(Thread.java:801) > > > > > > > > > > > > > > this didn't happen before, so it's probably related to the > recent > > > context > > > > > > > isolation change... > > > > > > > > > > > > > > > > > > > > > > > There are still some issues related to stopping bundles but > I will > > > > > > > > address them asap. > > > > > > > > > > > > > > > > Looking at the current overall implementation of Pax > Web that I > > > do > > > > > > > > not like to much + some research I had done lately how could > be > > > done > > > > > > > > better I guess that I will son start to rewrite it from > scratch, > > > this > > > > > > > > after we will stabilize the current version. > > > > > > > > > > > > > > > > For those that used the Event Listeners I changed the > signature of > > > > > > > > registering a listener to add the http context the listener > will > > > > > > > > listen to, so please take a look. > > > > > > > > > > > > > > > > Alin > > > > > > > > > > > > > > > > _______________________________________________ > > > > > > > > general mailing list > > > > > > > > [email protected] > > > > > > > > http://lists.ops4j.org/mailman/listinfo/general > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > > > Cheers, Stuart > > > > > > > _______________________________________________ > > > > > > > general mailing list > > > > > > > [email protected] > > > > > > > http://lists.ops4j.org/mailman/listinfo/general > > > > > > > > > > > > > > > > > > > > > > > > > > _______________________________________________ > > > > > > general mailing list > > > > > > [email protected] > > > > > > http://lists.ops4j.org/mailman/listinfo/general > > > > > > > > > > > > > > > > > > > > > > > > > > -- > > > > > Cheers, Stuart > > > > > > > > > > > > > > > > -- > > > > Cheers, Stuart > > > > > > > > > > > > -- > > > Cheers, Stuart > > > _______________________________________________ > > > general mailing list > > > [email protected] > > > http://lists.ops4j.org/mailman/listinfo/general > > > > > > > > > > _______________________________________________ > general mailing list > [email protected] > http://lists.ops4j.org/mailman/listinfo/general > -- Cheers, Stuart
_______________________________________________ general mailing list [email protected] http://lists.ops4j.org/mailman/listinfo/general
