#33057 [Opn]: PHP returns Content-Type header for 304 responses

From: Date: Wed, 18 May 2005 14:23:52 +0000
Subject: #33057 [Opn]: PHP returns Content-Type header for 304 responses
References: 1  Groups: php.bugs 
Request: Send a blank email to php-bugs+get-79148@lists.php.net to get a copy of this message
ID: 33057 User updated by: cboitel at lfdj dot com Reported By: cboitel at lfdj dot com Status: Open Bug Type: Apache related Operating System: Solaris/Linux PHP Version: 4.3.11 New Comment: 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; } Previous Comments: ------------------------------------------------------------------------ [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

« previous php.bugs (#79148) next »