Costin,
The code you are referring to in 3.3 makes it easy to abstract out
jdk 1.2 dependencies at build and runtime? If so, I have no
problems with you back porting that into 3.2.
If these classes are generic enough, perhaps they could be put
into the commons.
Regards,
Glenn
[EMAIL PROTECTED] wrote:
>
> On Tue, 1 May 2001, Marc Saegesser wrote:
>
> > Costin,
> >
> > The toLowerCase() patch look OK.
> >
> > If Glenn can fix the session id stuff easily then I think thats the best
> > approach right now.
>
> Adding the jdkcompat package can't hurt - it'll be a minimal change in
> SessionUtil plus the addition of 3 classes.
>
> I'll wait for Glenn - it's probably a 15 minutes fix if we use the stuff
> from 33.
>
> Costin
>
> >
> > > -----Original Message-----
> > > From: [EMAIL PROTECTED] [mailto:[EMAIL PROTECTED]]
> > > Sent: Tuesday, May 01, 2001 2:58 PM
> > > To: [EMAIL PROTECTED]
> > > Cc: [EMAIL PROTECTED]
> > > Subject: RE: Tomcat 3.2.2 beta 4
> > >
> > >
> > > Marc,
> > >
> > > I can quickly backport the jdkcompat package from 3.3 - it would be
> > > cleaner than hacking generateSessionId.
> > >
> > > The package is "standalone", doesn't depend on any 3.3 feature - and
> > > it's quite simple and effective.
> > >
> > > Also, if you could review the following patch that will fix "case
> > > sensitive match for host names " ( the patch is ported from 3.3 ).
> > >
> > > Costin
> > >
> > > Index: src/share/org/apache/tomcat/util/PrefixMapper.java
> > > =================================================================== RCS
> > > file:
> > > /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/util/PrefixMa
> > > pper.java,v
> > > retrieving revision 1.3.2.2 diff -r1.3.2.2 PrefixMapper.java
> > > 125a126
> > > > host=host.toLowerCase();
> > > 165a167
> > > > host=host.toLowerCasse();
> > > 181a184
> > > > host=host.toLowerCase();
> > > 201c204
> > > < if( host!=null )
> > > ---
> > > > if( host!=null ) {
> > > 202a206,209
> > > > if( myMap==null ) {
> > > > myMap=(PrefixMapper)vhostMaps.get( host.toLowerCase() );
> > > > }
> > > > }
> > >
> > >
> > >
> >
--
----------------------------------------------------------------------
Glenn Nielsen [EMAIL PROTECTED] | /* Spelin donut madder |
MOREnet System Programming | * if iz ina coment. |
Missouri Research and Education Network | */ |
----------------------------------------------------------------------
