Skip to content

Commit 7c41509

Browse files
committed
Merge branch 'PHP-8.2' into PHP-8.3
* PHP-8.2: php_cli_server: ensure single date header is present
2 parents a2d25af + a184594 commit 7c41509

File tree

3 files changed

+62
-5
lines changed

3 files changed

+62
-5
lines changed

NEWS

+3
Original file line numberDiff line numberDiff line change
@@ -6,6 +6,9 @@ PHP NEWS
66
. Fixed bug #80092 (ZTS + preload = segfault on shutdown). (nielsdos)
77
. Fixed buffer underflow when compiling memoized expression. (ilutov)
88

9+
- CLI:
10+
. Ensure a single Date header is present. (coppolafab)
11+
912
- CType:
1013
. Fixed bug GH-11997 (ctype_alnum 5 times slower in PHP 8.1 or greater).
1114
(nielsdos)

sapi/cli/php_cli_server.c

+20-5
Original file line numberDiff line numberDiff line change
@@ -356,18 +356,33 @@ static void append_http_status_line(smart_str *buffer, int protocol_version, int
356356
smart_str_appendl_ex(buffer, "\r\n", 2, persistent);
357357
} /* }}} */
358358

359-
static void append_essential_headers(smart_str* buffer, php_cli_server_client *client, bool persistent) /* {{{ */
359+
static void append_essential_headers(smart_str* buffer, php_cli_server_client *client, bool persistent, sapi_headers_struct *sapi_headers) /* {{{ */
360360
{
361361
zval *val;
362362
struct timeval tv = {0};
363+
bool append_date_header = true;
364+
365+
if (sapi_headers != NULL) {
366+
zend_llist_position pos;
367+
sapi_header_struct *h = (sapi_header_struct*)zend_llist_get_first_ex(&sapi_headers->headers, &pos);
368+
while (h) {
369+
if (h->header_len > strlen("Date:")) {
370+
if (strncasecmp(h->header, "Date:", strlen("Date:")) == 0) {
371+
append_date_header = false;
372+
break;
373+
}
374+
}
375+
h = (sapi_header_struct*)zend_llist_get_next_ex(&sapi_headers->headers, &pos);
376+
}
377+
}
363378

364379
if (NULL != (val = zend_hash_find(&client->request.headers, ZSTR_KNOWN(ZEND_STR_HOST)))) {
365380
smart_str_appends_ex(buffer, "Host: ", persistent);
366381
smart_str_append_ex(buffer, Z_STR_P(val), persistent);
367382
smart_str_appends_ex(buffer, "\r\n", persistent);
368383
}
369384

370-
if (!gettimeofday(&tv, NULL)) {
385+
if (append_date_header && !gettimeofday(&tv, NULL)) {
371386
zend_string *dt = php_format_date("D, d M Y H:i:s", sizeof("D, d M Y H:i:s") - 1, tv.tv_sec, 0);
372387
smart_str_appends_ex(buffer, "Date: ", persistent);
373388
smart_str_append_ex(buffer, dt, persistent);
@@ -551,7 +566,7 @@ static int sapi_cli_server_send_headers(sapi_headers_struct *sapi_headers) /* {{
551566
append_http_status_line(&buffer, client->request.protocol_version, SG(sapi_headers).http_response_code, 0);
552567
}
553568

554-
append_essential_headers(&buffer, client, 0);
569+
append_essential_headers(&buffer, client, 0, sapi_headers);
555570

556571
h = (sapi_header_struct*)zend_llist_get_first_ex(&sapi_headers->headers, &pos);
557572
while (h) {
@@ -2049,7 +2064,7 @@ static zend_result php_cli_server_send_error_page(php_cli_server *server, php_cl
20492064
/* out of memory */
20502065
goto fail;
20512066
}
2052-
append_essential_headers(&buffer, client, 1);
2067+
append_essential_headers(&buffer, client, 1, NULL);
20532068
smart_str_appends_ex(&buffer, "Content-Type: text/html; charset=UTF-8\r\n", 1);
20542069
smart_str_appends_ex(&buffer, "Content-Length: ", 1);
20552070
smart_str_append_unsigned_ex(&buffer, php_cli_server_buffer_size(&client->content_sender.buffer), 1);
@@ -2165,7 +2180,7 @@ static zend_result php_cli_server_begin_send_static(php_cli_server *server, php_
21652180
php_cli_server_log_response(client, 500, NULL);
21662181
return FAILURE;
21672182
}
2168-
append_essential_headers(&buffer, client, 1);
2183+
append_essential_headers(&buffer, client, 1, NULL);
21692184
if (mime_type) {
21702185
smart_str_appendl_ex(&buffer, "Content-Type: ", sizeof("Content-Type: ") - 1, 1);
21712186
smart_str_appends_ex(&buffer, mime_type, 1);

sapi/cli/tests/gh12363.phpt

+39
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,39 @@
1+
--TEST--
2+
Ensure a single Date header is present
3+
--SKIPIF--
4+
<?php
5+
include "skipif.inc";
6+
?>
7+
--FILE--
8+
<?php
9+
include "php_cli_server.inc";
10+
php_cli_server_start(<<<'PHP'
11+
header('Date: Mon, 25 Mar 1985 00:20:00 GMT');
12+
PHP
13+
);
14+
15+
$host = PHP_CLI_SERVER_HOSTNAME;
16+
$fp = php_cli_server_connect();
17+
18+
if(fwrite($fp, <<<HEADER
19+
GET / HTTP/1.1
20+
Host: {$host}
21+
22+
23+
HEADER
24+
)) {
25+
while (!feof($fp)) {
26+
echo fgets($fp);
27+
}
28+
}
29+
30+
fclose($fp);
31+
?>
32+
--EXPECTF--
33+
HTTP/1.1 200 OK
34+
Host: %s
35+
Connection: close
36+
X-Powered-By: %s
37+
Date: Mon, 25 Mar 1985 00:20:00 GMT
38+
Content-type: text/html; charset=UTF-8
39+

0 commit comments

Comments
 (0)