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 Tue, 09 Apr 2002 18:43:05 GMT
> costin      02/04/09 10:23:49
>
>   Modified:    http11/src/java/org/apache/coyote/http11
>                         Http11Processor.java
>   Log:
>   If host header is not set, use the local socket to set the local port
>   and hostname.
>
>   Remy - I'm not sure if this is correct, but I see no other way to
>   pass this info ( I could add a callback - but this is needed in most
>   calls ).
>
>   If there is any particular reason to keep port and localhost unset
>   I'll rollback and add the callback.
>
>   ( this used to be in the 3.3-specific code, and is needed to construct
>   redirects, etc )
>   +            //The info from socket is usually acurate
>   +            // req.setServerPort(80);

That's not good. If no host is specified with HTTP/1.1, we must default to
the protocol default (as per the HTTP spec).
The if HostMB == null implies HTTP/1.0 (with HTTP/1.1, the request
processing would be mostly over without a Host header, with a 400 response).
I've modified the patch to reflect this.

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