You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomcat.apache.org by cr...@locus.apache.org on 2000/03/28 05:40:01 UTC

cvs commit: jakarta-tomcat/src/share/org/apache/tomcat/service/http HttpRequestAdapter.java

craigmcc    00/03/27 19:40:00

  Modified:    src/share/org/apache/tomcat/service/http
                        HttpRequestAdapter.java
  Log:
  Patch HttpRequestAdapter to not hang on a POST when "HTTP/0.9" is included
  in the request.  From my reading of the specs, it seems that a client that
  sends HTTP/0.9 on the request line is broken (it should only be sending
  GET plus the request URI), but what the heck.
  PR:117
  Submitted by:	Larry.Isaacs@sas.com
  
  Revision  Changes    Path
  1.8       +5 -4      jakarta-tomcat/src/share/org/apache/tomcat/service/http/HttpRequestAdapter.java
  
  Index: HttpRequestAdapter.java
  ===================================================================
  RCS file: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/service/http/HttpRequestAdapter.java,v
  retrieving revision 1.7
  retrieving revision 1.8
  diff -u -r1.7 -r1.8
  --- HttpRequestAdapter.java	2000/02/18 00:17:49	1.7
  +++ HttpRequestAdapter.java	2000/03/28 03:40:00	1.8
  @@ -1,7 +1,7 @@
   /*
  - * $Header: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/service/http/HttpRequestAdapter.java,v 1.7 2000/02/18 00:17:49 costin Exp $
  - * $Revision: 1.7 $
  - * $Date: 2000/02/18 00:17:49 $
  + * $Header: /home/cvs/jakarta-tomcat/src/share/org/apache/tomcat/service/http/HttpRequestAdapter.java,v 1.8 2000/03/28 03:40:00 craigmcc Exp $
  + * $Revision: 1.8 $
  + * $Date: 2000/03/28 03:40:00 $
    *
    * ====================================================================
    *
  @@ -128,7 +128,8 @@
   	// 	}
   
   	// for 0.9, we don't have headers!
  -	if(protocol!=null)
  +	if ((protocol!=null) &&
  +            !protocol.toLowerCase().startsWith("http/0."))
   	    headers.read(in);
   	//	processCookies(); // called later