On Tue, 9 Apr 2002, Bill Barker wrote:

> >   +
> >   +        remoteAddrMB = coyoteRequest.remoteAddr();
> >   + remoteHostMB = coyoteRequest.remoteHost();
> >   + serverNameMB = coyoteRequest.serverName();
> >   +
> >   +
> 
> This forces the lookup to always be per-request, when it would be much nicer
> to do it per-connection.

This is done when the tomcat3 request is created  - and it only means 
that the buffers for storing the remoteHost and serverName are identical ( 
we override the tomcat.Request fields with the MessageBytes from 
coyote.Request ). That avoids copying them.

( the request is recycled, but the fields remain )

Unless I'm missing something.

> >   +    public void setAttribute(String name, Object value) {
> >   + coyoteRequest.setAttribute( name, value );
> >   +    }
> >   +
> >   +    public Object getAttribute(String name) {
> >   +        if (name == null) {
> >   +            return null;
> >   +        }
> >   +        return coyoteRequest.getAttribute( name );
> >   +    }
> >   +
> 
> Unless I'm missing something, this competely breaks SSL support, since
> CoyoteInterceptor2.getInfo is never called.

Yes, I'll fix it. Thanks ! 

Costin


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to