ID: 33057
Updated by: [EMAIL PROTECTED]
Reported By: cboitel at lfdj dot com
-Status: Open
+Status: Feedback
Bug Type: Apache related
Operating System: Solaris/Linux
PHP Version: 4.3.11
New Comment:
Please generate unified diff (`diff -u`), put it somewhere and give us
it's URL.
Previous Comments:
------------------------------------------------------------------------
[2005-05-18 16:23:49] cboitel at lfdj dot com
I have patched the mod_php4.c file to use send_error_response apache
function instead of send_http_headers for 304 responses. You should
also check if send_error_response shall also be used for non-200
responses and if its second param (recursive call) shall be set to 0 or
1.
static int sapi_apache_send_headers(sapi_headers_struct *sapi_headers
TSRMLS_DC)
{
request_rec *r;
if(SG(server_context) == NULL) { /* server_context is not here
anymore */
return SAPI_HEADER_SEND_FAILED;
}
r = (request_rec *) SG(server_context);
r->status = SG(sapi_headers).http_response_code;
if( r-> status==304 )
{
send_error_response( r, 0 );
}
else
{
send_http_header((request_rec *) SG(server_context));
}
return SAPI_HEADER_SENT_SUCCESSFULLY;
}
------------------------------------------------------------------------
[2005-05-18 16:12:27] cboitel at lfdj dot com
Description:
------------
When setting response code to 304 from within PHP code, a
"Content-Type:" header is always added when PHP is compiled within
Apache 1.3x whereas it is not for Apache 2.x
This makes Apache 1.3x/PHP non compliant with RFC HTTP/1.1 for 304
responses in the case where a weak validator is used (Content-Type MUST
not be returned).
Reproduce code:
---------------
mytest1.php
<?
header( "HTTP/1.1 304 Not Modified" );
?>
Expected result:
----------------
telnet localhost 80
GET /mytest1.php HTTP/1.0
HTTP/1.1 304 Not Modified
Date: Wed, 18 May 2005 14:10:06 GMT
Server: Apache
Connection: close
Actual result:
--------------
telnet localhost 80
GET /mytest1.php HTTP/1.0
HTTP/1.1 304 Not Modified
Date: Wed, 18 May 2005 14:10:06 GMT
Server: Apache
Content-Type: text/html
Connection: close
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=33057&edit=1
