Remy, Ok. I know I haven't tested it yet! :( I haven't gotten 4.1 to compile from CVS yet. Can we get the patch for bug 1 committed though. This would allow people to use RMI (I think) by adding the repository http://server/app to the ClassLoader. Were you thinking of trying to make 4.1.10 the stable release? If not couldn't we slip it in before.
Dave. > -----Original Message----- > From: Remy Maucherat [mailto:[EMAIL PROTECTED]] > Sent: 28 August 2002 08:19 > To: Tomcat Developers List > Subject: Re: Bug in WebappClassLoader > > David Oxley wrote: > > All, > > > > > > > > There is a couple of bugs in WebappClassLoader. > > > > 1. On line 953 there is a call to findClassInternal which could throw a > > ClassNotFoundException that gets caught on line 979. This means that > > lines 963 through 973 will never get executed. i.e. Repositories added > > with the method AddRepository will not get searched. > > That one is a minor bug, so it will not be fixed until after the first > TC 4.1 stable release. > > > 2. Also URL's returned from getURL are not encoded. > > Your fix is wrong (all the URL special characters would get encoded, > including ':', '/' and others). > > > I have submitted a patch for these two problems, but I have not got > > tomcat 4 to compile yet so haven't tested the patch. > > Please try to test patches a little bit before, it saves a few round > trips. > > > Comments or flames welcome. :) > > Better luck next time then ;-) > > Remy > > > -- > To unsubscribe, e-mail: <mailto:tomcat-dev- > [EMAIL PROTECTED]> > For additional commands, e-mail: <mailto:tomcat-dev- > [EMAIL PROTECTED]> -- To unsubscribe, e-mail: <mailto:[EMAIL PROTECTED]> For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>
