tomcat-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Remy Maucherat" <r...@apache.org>
Subject Re: cvs commit: jakarta-tomcat-connectors/http11/src/java/org/apache/coyote/http11 Http11Processor.java
Date Thu, 11 Apr 2002 08:56:40 GMT
> billbarker    02/04/10 22:58:03
>
>   Modified:    http11/src/java/org/apache/coyote/http11
>                         Http11Processor.java
>   Log:
>   Multiple Issues.
>
>   -- Degrade to the socket port on HTTP/1.0 requests with a Host header
but no port number.
>
>   -- Default to port 443 if an HTTP/1.1 request with no port header, but
we are configured to be Secure.
>
>   -- Remember the RemoteHost/RemoteAddr per-connection so that only one
lookup is necessary (the 3.3.x Adapter recycles these, so memory is good).
>
>            if (colonPos < 0) {
>   -            req.setServerPort(80);
>   +            if( http11 ) {
>   +                if(sslSupport == null) // not configured Secure
>   +                    req.setServerPort(80);
>   +                else
>   +                    req.setServerPort(443); // if Secure, assume https
>   +            } else {
>   +                // Assume that non-HTTP/1.1 clients are broken
>   +                req.setServerPort(socket.getLocalPort());
>   +            }
>                req.serverName().setBytes( valueB, valueS, valueL);
>            } else {
>                req.serverName().setBytes( valueB, valueS, colonPos);

Yes, exactly what I wanted to do.
Thanks Bill.

Remy


--
To unsubscribe, e-mail:   <mailto:tomcat-dev-unsubscribe@jakarta.apache.org>
For additional commands, e-mail: <mailto:tomcat-dev-help@jakarta.apache.org>


Mime
View raw message