You are viewing a plain text version of this content. The canonical link for it is here.
Posted to cvs@httpd.apache.org by dg...@hyperreal.org on 1997/11/22 23:24:53 UTC

cvs commit: apache/src http_request.c

dgaudet     97/11/22 14:24:53

  Modified:    src      Tag: APACHE_1_2_X http_request.c
  Log:
  Comment was a little bit inaccurate.  Also explicitly mention 2068 S9,
  S14.23 in the error message so that maybe we can avoid more and more
  people complaining about us rejecting HTTP/1.1 requests without a Host:
  header.
  
  PR:             1454
  
  Revision  Changes    Path
  No                   revision
  
  
  No                   revision
  
  
  1.50.2.8  +5 -2      apache/src/http_request.c
  
  Index: http_request.c
  ===================================================================
  RCS file: /export/home/cvs/apache/src/http_request.c,v
  retrieving revision 1.50.2.7
  retrieving revision 1.50.2.8
  diff -u -r1.50.2.7 -r1.50.2.8
  --- http_request.c	1997/08/11 14:20:03	1.50.2.7
  +++ http_request.c	1997/11/22 22:24:52	1.50.2.8
  @@ -906,9 +906,12 @@
   	((r->proto_num == 1001) && !table_get(r->headers_in, "Host"))) {
           /* Client sent us a HTTP/1.1 or later request without telling
   	 * us the hostname, either with a full URL or a Host: header.
  -	 * We therefore need to (as per the 1.1 spec) send an error
  +	 * We therefore need to (as per the 1.1 spec) send an error.
  +	 * As a special case, HTTP/1.1 mentions twice (S9, S14.23)
  +	 * that a request MUST contain a Host: header, and the server
  +	 * MUST respond with 400 if it doesn't.
   	 */
  -        log_reason ("client sent HTTP/1.1 request without hostname",
  +        log_reason ("client sent HTTP/1.1 request without hostname (see RFC2068 sections 9 and 14.23)",
   		    r->uri, r);
   	die (BAD_REQUEST, r);
   	return;