Skip to content
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 4 additions & 0 deletions NEWS
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@ PHP NEWS
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
?? ??? ????, PHP 8.6.0beta3

- CLI:
. Fixed bug GH-23242 (PHP development server does not support Expect
100-continue flow control). (Sjoerd Langkemper)


27 Aug 2026, PHP 8.6.0beta2

Expand Down
119 changes: 64 additions & 55 deletions UPGRADING

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions ext/readline/readline.c
Original file line number Diff line number Diff line change
Expand Up @@ -489,8 +489,8 @@ PHP_FUNCTION(readline_completion_function)

/* NOTE: The rl_attempted_completion_function variable (and others) are part of the readline library, not php */
rl_attempted_completion_function = php_readline_completion_cb;
RETURN_BOOL(rl_attempted_completion_function != NULL);

RETURN_TRUE;
}

/* }}} */
Expand Down
2 changes: 1 addition & 1 deletion ext/readline/readline.stub.php
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,7 @@ function readline_read_history(?string $filename = null): bool {}

function readline_write_history(?string $filename = null): bool {}

function readline_completion_function(callable $callback): bool {}
function readline_completion_function(callable $callback): true {}


#ifdef HAVE_RL_CALLBACK_READ_CHAR
Expand Down
4 changes: 2 additions & 2 deletions ext/readline/readline_arginfo.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

2 changes: 1 addition & 1 deletion ext/standard/basic_functions.stub.php
Original file line number Diff line number Diff line change
Expand Up @@ -2063,7 +2063,7 @@ function getprotobyname(string $protocol): int|false {}
function getprotobynumber(int $protocol): string|false {}
#endif

function register_tick_function(callable $callback, mixed ...$args): bool {}
function register_tick_function(callable $callback, mixed ...$args): true {}

function unregister_tick_function(callable $callback): void {}

Expand Down
4 changes: 2 additions & 2 deletions ext/standard/basic_functions_arginfo.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 4 additions & 4 deletions ext/standard/basic_functions_decl.h

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

26 changes: 26 additions & 0 deletions sapi/cli/php_cli_server.c
Original file line number Diff line number Diff line change
Expand Up @@ -176,6 +176,7 @@ typedef struct php_cli_server_client {
bool request_read;
bool too_large_post;
bool headers_written;
bool expect_continue;
zend_string *current_header_name;
zend_string *current_header_value;
enum { HEADER_NONE=0, HEADER_FIELD, HEADER_VALUE } last_header_element;
Expand Down Expand Up @@ -1794,6 +1795,13 @@ static int php_cli_server_client_read_request_on_headers_complete(php_http_parse
return 2;
}

zval *expect_val = zend_hash_str_find(&client->request.headers, "expect", sizeof("expect") - 1);
if (expect_val && Z_TYPE_P(expect_val) == IS_STRING
&& zend_string_equals_literal_ci(Z_STR_P(expect_val), "100-continue")
&& parser->http_major == 1 && parser->http_minor == 1) {
client->expect_continue = true;
}

return 0;
}

Expand Down Expand Up @@ -1901,6 +1909,23 @@ static int php_cli_server_client_read_request(php_cli_server_client *client, cha
return -1;
}

if (client->expect_continue && !client->request_read) {
/* Parser completed headers with Expect: 100-continue but hasn't
* finished reading the body. Send 100 Continue before the client
* sends the request body. Only supported in HTTP/1.1. */
static const char continue_response[] = "HTTP/1.1 100 Continue\r\n\r\n";
bool send_success = false;
client->expect_continue = false;
zend_try {
size_t sent = php_cli_server_client_send_through(client, continue_response, strlen(continue_response));
send_success = sent == strlen(continue_response);
} zend_end_try();
if (!send_success) {
*errstr = php_socket_strerror(php_socket_errno(), NULL, 0);
return -1;
}
}

return client->request_read ? 1: 0;
}
/* }}} */
Expand Down Expand Up @@ -1985,6 +2010,7 @@ static void php_cli_server_client_ctor(php_cli_server_client *client, php_cli_se
client->request_read = false;
client->too_large_post = false;
client->headers_written = false;
client->expect_continue = false;

client->last_header_element = HEADER_NONE;
client->current_header_name = NULL;
Expand Down
3 changes: 2 additions & 1 deletion sapi/cli/tests/php_cli_server.inc
Original file line number Diff line number Diff line change
Expand Up @@ -5,6 +5,7 @@ class CliServerInfo {
public function __construct(
public string $docRoot,
public $processHandle,
public $outputFile,
) {}
}

Expand Down Expand Up @@ -118,7 +119,7 @@ function php_cli_server_start(
define("PHP_CLI_SERVER_PORT", $port);
define("PHP_CLI_SERVER_ADDRESS", PHP_CLI_SERVER_HOSTNAME.":".PHP_CLI_SERVER_PORT);

return new CliServerInfo($doc_root, $handle);
return new CliServerInfo($doc_root, $handle, $output_file);
}

function php_cli_server_connect() {
Expand Down
35 changes: 35 additions & 0 deletions sapi/cli/tests/php_cli_server_expect_100_continue_curl.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,35 @@
--TEST--
Expect 100-continue behavior in PHP development server (curl)
--SKIPIF--
<?php
include "skipif.inc";
?>
--EXTENSIONS--
curl
--FILE--
<?php
include 'php_cli_server.inc';
$server = php_cli_server_start();

// Generate a POST body larger than 1MB to trigger Expect: 100-continue
$body = str_repeat('A', 1024 * 1024 + 1);

$ch = curl_init();
curl_setopt($ch, CURLOPT_URL, PHP_CLI_SERVER_ADDRESS);
curl_setopt($ch, CURLOPT_RETURNTRANSFER, 1);
curl_setopt($ch, CURLOPT_POSTFIELDS, $body);

// Set a high timeout for 100-continue response
curl_setopt($ch, CURLOPT_EXPECT_100_TIMEOUT_MS, 2000);

curl_exec($ch);
var_dump(curl_errno($ch));

echo "Did the PHP development server send a HTTP/1.1 100 Continue header?\n";
$start_transfer_time = curl_getinfo($ch, CURLINFO_STARTTRANSFER_TIME_T);
var_dump($start_transfer_time < 1_000_000);
?>
--EXPECT--
int(0)
Did the PHP development server send a HTTP/1.1 100 Continue header?
bool(true)
33 changes: 33 additions & 0 deletions sapi/cli/tests/php_cli_server_expect_100_continue_iua.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,33 @@
--TEST--
Failure to send "100 Continue" is reported with ignore_user_abort=1
--SKIPIF--
<?php
include "skipif.inc";
if (!extension_loaded("sockets")) die("skip sockets extension required");
if (PHP_OS_FAMILY === "Windows") die("skip SO_LINGER reset behaviour differs on Windows");
?>
--FILE--
<?php
include "php_cli_server.inc";
$server = php_cli_server_start('echo "Hello world";', 'index.php', ['-d', 'ignore_user_abort=1']);

$fp = fsockopen(PHP_CLI_SERVER_HOSTNAME, PHP_CLI_SERVER_PORT);
socket_set_option(socket_import_stream($fp), SOL_SOCKET, SO_LINGER, ['l_onoff' => 1, 'l_linger' => 0]);
fwrite($fp, "POST / HTTP/1.1\r\nExpect: 100-continue\r\nContent-Length: 4\r\n\r\n");
fclose($fp);

$output = '';
for ($i = 0; $i < 100 && !str_contains($output, 'Invalid request'); $i++) {
usleep(50000);
$output = file_get_contents($server->outputFile);
}

var_dump(str_contains($output, 'Invalid request'), str_contains($output, 'Unexpected EOF'));
?>
--CLEAN--
<?php
@unlink(__DIR__ . '/php_cli_server_expect_100_continue_iua.log')
?>
--EXPECT--
bool(true)
bool(false)
86 changes: 86 additions & 0 deletions sapi/cli/tests/php_cli_server_expect_100_continue_socket.phpt
Original file line number Diff line number Diff line change
@@ -0,0 +1,86 @@
--TEST--
Expect 100-continue behavior in PHP development server (sockets)
--SKIPIF--
<?php
include "skipif.inc";
?>
--FILE--
<?php
include "php_cli_server.inc";
php_cli_server_start();

echo "# Send Expect: 100-continue header, receive 100 Continue response.\n";
$fp = php_cli_server_connect();
fwrite($fp, "POST / HTTP/1.1\r\nExpect: 100-continue\r\nContent-Length: 4\r\nConnection: close\r\n\r\n");
echo fgets($fp);
echo fgets($fp);
fwrite($fp, "body");
echo fgets($fp);
fclose($fp);

echo "# Send Expect: 100-continue header on HTTP/1.0.\n";
$fp = php_cli_server_connect();
fwrite($fp, "POST / HTTP/1.0\r\nExpect: 100-continue\r\nContent-Length: 4\r\nConnection: close\r\n\r\n");
$read = [$fp];
var_dump(stream_select($read, $write, $except, 0, 1000));
fwrite($fp, "body");
echo fgets($fp);
fclose($fp);

echo "# Send Expect: 100-continue header and disconnect.\n";
$fp = php_cli_server_connect();
if (extension_loaded('sockets')) {
// Set SO_LINGER timeout to zero so that send fails on the server immediately
socket_set_option(
socket_import_stream($fp),
SOL_SOCKET,
SO_LINGER,
[
'l_onoff' => 1,
'l_linger' => 0,
]
);
}
stream_socket_shutdown($fp, STREAM_SHUT_RD);
fwrite($fp, "POST / HTTP/1.1\r\nExpect: 100-continue\r\nContent-Length: 4\r\nConnection: close\r\n\r\n");
fclose($fp);

$fp = php_cli_server_connect();
fwrite($fp, "GET / HTTP/1.1\r\nConnection: close\r\n\r\n");
echo fgets($fp);
fclose($fp);

echo "# GET with Expect header (no body).\n";
$fp = php_cli_server_connect();
fwrite($fp, "GET / HTTP/1.1\r\nExpect: 100-continue\r\nConnection: close\r\n\r\n");
echo fgets($fp);
fclose($fp);

echo "# POST with empty body.\n";
$fp = php_cli_server_connect();
fwrite($fp, "POST / HTTP/1.1\r\nExpect: 100-continue\r\nContent-Length: 0\r\nConnection: close\r\n\r\n");
echo fgets($fp);
fclose($fp);

echo "# Lower-case expect header.\n";
$fp = php_cli_server_connect();
fwrite($fp, "POST / HTTP/1.1\r\nexpect: 100-continue\r\nContent-Length: 4\r\nConnection: close\r\n\r\n");
echo fgets($fp);
fclose($fp);
?>
--EXPECT--
# Send Expect: 100-continue header, receive 100 Continue response.
HTTP/1.1 100 Continue

HTTP/1.1 200 OK
# Send Expect: 100-continue header on HTTP/1.0.
int(0)
HTTP/1.0 200 OK
# Send Expect: 100-continue header and disconnect.
HTTP/1.1 200 OK
# GET with Expect header (no body).
HTTP/1.1 200 OK
# POST with empty body.
HTTP/1.1 200 OK
# Lower-case expect header.
HTTP/1.1 100 Continue