]> granicus.if.org Git - php/commitdiff
Use ephemeral port in more server tests
authorNikita Popov <nikita.ppv@gmail.com>
Fri, 23 Oct 2020 09:29:44 +0000 (11:29 +0200)
committerNikita Popov <nikita.ppv@gmail.com>
Fri, 23 Oct 2020 12:26:34 +0000 (14:26 +0200)
Port the main php_cli_server.inc to use ephemeral ports, thus
allowing CLI server tests to be parallelized.

A complication here is that we also need to give each test a
separate doc root, to avoid index.php files writing over each
other.

Closes GH-6375.

15 files changed:
ext/soap/tests/bug73037.phpt
ext/soap/tests/custom_content_type.phpt
ext/standard/tests/streams/bug64433.phpt
ext/standard/tests/url/get_headers_error_003.phpt
sapi/cli/tests/CONFLICTS [deleted file]
sapi/cli/tests/bug61977.phpt
sapi/cli/tests/bug67429_1.phpt
sapi/cli/tests/bug67429_2.phpt
sapi/cli/tests/bug68291.phpt
sapi/cli/tests/bug69655.phpt
sapi/cli/tests/php_cli_server.inc
sapi/cli/tests/php_cli_server_002.phpt
sapi/cli/tests/php_cli_server_012.phpt
sapi/cli/tests/php_cli_server_015.phpt
tests/basic/bug67198.phpt

index 4e7293286244e7690df4b8570a72fa063eb57589..8060575bd88f0ade4e5d80a84ef075c614a0f01e 100644 (file)
@@ -1,7 +1,5 @@
 --TEST--
 Bug #73037 SoapServer reports Bad Request when gzipped, var 0
---CONFLICTS--
-server
 --SKIPIF--
 <?php
     require_once('skipif.inc');
index 17a074f102425a269ac6499e13bc425fa634d0b6..9172d3d6ca4c60f1b09cbb60e41f28d53f673a87 100644 (file)
@@ -8,8 +8,6 @@ SOAP customized Content-Type, eg. SwA use case
         echo "skip sapi/cli/tests/php_cli_server.inc required but not found";
     }
 ?>
---CONFLICTS--
-server
 --FILE--
 <?php
 
index 56af2418375a82cd8b2a281ebd5093e1f1a1afbf..53fecc57f9083a41e1c87ae355d856188c285433 100644 (file)
@@ -9,8 +9,6 @@ if(!$res) {
     die("skip could not open cli server script");
 }
 ?>
---CONFLICTS--
-server
 --FILE--
 <?php
 include __DIR__."/../../../../sapi/cli/tests/php_cli_server.inc";
index a8edc2cb21bf10dc17c82c083a98e02a9e77fb15..7ee0b9e210d6a7b0d6cc77070807badb896c5cf9 100644 (file)
@@ -1,7 +1,5 @@
 --TEST--
 Test get_headers() function : test with context
---CONFLICTS--
-server
 --FILE--
 <?php
 
diff --git a/sapi/cli/tests/CONFLICTS b/sapi/cli/tests/CONFLICTS
deleted file mode 100644 (file)
index 254defd..0000000
+++ /dev/null
@@ -1 +0,0 @@
-server
index b55c6a2e8ffb025852be821a4dbef57adb55caa1..e804e0dfce55bbb58dc88490c5905ddd0e2b9987 100644 (file)
@@ -7,7 +7,7 @@ include "skipif.inc";
 --FILE--
 <?php
 include "php_cli_server.inc";
-php_cli_server_start('<?php ?>', null);
+$doc_root = php_cli_server_start('<?php ?>', null)->docRoot;
 
 /*
  * If a Mime Type is added in php_cli_server.c, add it to this array and update
@@ -15,32 +15,29 @@ php_cli_server_start('<?php ?>', null);
  */
 $mimetypes = ['html', 'htm', 'svg', 'css', 'js', 'png', 'webm', 'ogv', 'ogg'];
 
-function test_mimetypes($mimetypes) {
-    foreach ($mimetypes as $mimetype) {
-        $host = PHP_CLI_SERVER_HOSTNAME;
-        $fp = php_cli_server_connect();
-        if (!$fp) die('Connect failed');
-        file_put_contents(__DIR__ . "/foo.{$mimetype}", '');
-        $header = <<<HEADER
+foreach ($mimetypes as $mimetype) {
+    $host = PHP_CLI_SERVER_HOSTNAME;
+    $fp = php_cli_server_connect();
+    if (!$fp) die('Connect failed');
+    file_put_contents($doc_root . "/foo.{$mimetype}", '');
+    $header = <<<HEADER
 GET /foo.{$mimetype} HTTP/1.1
 Host: {$host}
 
 
 HEADER;
-        if (fwrite($fp, $header)) {
-            while (!feof($fp)) {
-                $text = fgets($fp);
-                if (strncasecmp("Content-type:", $text, 13) == 0) {
-                    echo "foo.{$mimetype} => ", $text;
-                }
+    if (fwrite($fp, $header)) {
+        while (!feof($fp)) {
+            $text = fgets($fp);
+            if (strncasecmp("Content-type:", $text, 13) == 0) {
+                echo "foo.{$mimetype} => ", $text;
             }
-            @unlink(__DIR__ . "/foo.{$mimetype}");
-            fclose($fp);
         }
+        @unlink($doc_root . "/foo.{$mimetype}");
+        fclose($fp);
     }
 }
 
-test_mimetypes($mimetypes);
 ?>
 --EXPECT--
 foo.html => Content-Type: text/html; charset=UTF-8
index 5b23818bdb08892cfb036c73af519ca1e1e6d774..551946334036da1250e5fb9f6835b1c72ea30344 100644 (file)
@@ -8,7 +8,7 @@ include "skipif.inc";
 <?php
 include "php_cli_server.inc";
 
-$proc_handle = php_cli_server_start(<<<PHP
+php_cli_server_start(<<<PHP
 http_response_code(308);
 PHP
 );
index 381c7af8e5d8d74289c1cb216e2a19010a8e5097..5c4fffe410c86d5785f9477471104eda39a1270b 100644 (file)
@@ -8,7 +8,7 @@ include "skipif.inc";
 <?php
 include "php_cli_server.inc";
 
-$proc_handle = php_cli_server_start(<<<PHP
+php_cli_server_start(<<<PHP
 http_response_code(426);
 PHP
 );
index 62042307c58903a689f21f7c5df4d9e15addaf6b..1082194f5f8ff9beb338f69a56098f0fbdbc4909 100644 (file)
@@ -9,13 +9,10 @@ include "skipif.inc";
 --FILE--
 <?php
 include "php_cli_server.inc";
-file_put_contents(__DIR__ . '/bug68291+test.html', 'Found');
-php_cli_server_start(NULL, NULL);
+$docRoot = php_cli_server_start(NULL, NULL)->docRoot;
+file_put_contents($docRoot . '/bug68291+test.html', 'Found');
 echo file_get_contents('http://' . PHP_CLI_SERVER_ADDRESS . '/bug68291+test.html');
-?>
---CLEAN--
-<?php
-@unlink(__DIR__ . '/bug68291+test.html');
+@unlink($docRoot . '/bug68291+test.html');
 ?>
 --EXPECT--
 Found
index b5612357f9016fcf0e3115bd464f04ee14abe7fb..4ab53bcbaae0d182442ab213c6c136c6406aa619 100644 (file)
@@ -17,11 +17,11 @@ foreach (['MKCO', 'MKCOLL', 'M'] as $method) {
 }
 ?>
 --EXPECTF--
-Warning: file_get_contents(http://localhost:8964): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
+Warning: file_get_contents(http://localhost:%d): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
  in %s on line %d
 
-Warning: file_get_contents(http://localhost:8964): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
+Warning: file_get_contents(http://localhost:%d): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
  in %s on line %d
 
-Warning: file_get_contents(http://localhost:8964): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
+Warning: file_get_contents(http://localhost:%d): Failed to open stream: HTTP request failed! HTTP/1.1 501 Not Implemented
  in %s on line %d
index 4cf8705a333fd06b44580fe4d33b0899a65b9583..26bdd4c18f9f22ed7d8f922153720930e99f6bdf 100644 (file)
@@ -1,22 +1,29 @@
 <?php
-define("PHP_CLI_SERVER_HOSTNAME", "localhost");
-define("PHP_CLI_SERVER_PORT", 8964);
-define("PHP_CLI_SERVER_ADDRESS", PHP_CLI_SERVER_HOSTNAME.":".PHP_CLI_SERVER_PORT);
+
+// TODO: Move address/port info in here?
+class CliServerInfo {
+    public function __construct(
+        public string $docRoot,
+    ) {}
+}
 
 function php_cli_server_start(
     ?string $code = 'echo "Hello world";',
     ?string $router = 'index.php',
     array $cmd_args = []
-) {
+): CliServerInfo {
     $php_executable = getenv('TEST_PHP_EXECUTABLE');
-    $doc_root = __DIR__;
     $error = null;
 
+    // Create dedicated doc root to avoid index.php clashes between tests.
+    $doc_root = __DIR__ . '/' . basename($_SERVER['PHP_SELF'], '.php');
+    @mkdir($doc_root);
+
     if ($code) {
         file_put_contents($doc_root . '/' . ($router ?: 'index.php'), '<?php ' . $code . ' ?>');
     }
 
-    $cmd = [$php_executable, '-t', $doc_root, '-n', ...$cmd_args, '-S', PHP_CLI_SERVER_ADDRESS];
+    $cmd = [$php_executable, '-t', $doc_root, '-n', ...$cmd_args, '-S', 'localhost:0'];
     if (!is_null($router)) {
         $cmd[] = $router;
     }
@@ -24,62 +31,58 @@ function php_cli_server_start(
     $descriptorspec = array(
         0 => STDIN,
         1 => STDOUT,
-        2 => array("null"),
+        2 => ['pipe', 'w'],
     );
     $handle = proc_open($cmd, $descriptorspec, $pipes, $doc_root, null, array("suppress_errors" => true));
 
-    // note: here we check the process is running
-    for ($i=0; $i < 120; $i++) {
+    // First, wait for the dev server to declare itself ready.
+    $bound = null;
+    stream_set_blocking($pipes[2], false);
+    for ($i = 0; $i < 60; $i++) {
+        usleep(50000); // 50ms per try
         $status = proc_get_status($handle);
+        if (empty($status['running'])) {
+            echo "Server is not running\n";
+            proc_terminate($handle);
+            exit(1);
+        }
 
-        if (!$status || !$status['running']) {
-            if ($status &&
-               ($status['running'] == false && $status['exitcode'] != 0)) {
-                $error =
-                    "Server could not be started\n";
-                break;
+        while (($line = fgets($pipes[2])) !== false) {
+            if (preg_match('@PHP \S* Development Server \(https?://(.*?:\d+)\) started@', $line, $matches)) {
+                $bound = $matches[1];
+                // Now that we've identified the listen address, close STDERR.
+                // Otherwise the pipe may clog up with unread log messages.
+                fclose($pipes[2]);
+                break 2;
             }
-
-            usleep(50000); // 50ms per try
-            continue;
         }
+    }
+    if ($bound === null) {
+        echo "Server did not output startup message";
+        proc_terminate($handle);
+        exit(1);
+    }
 
-        if ($status['signaled']) {
-            $error =
-                "Server was terminated with {$status['termsig']}\n";
+    // Now wait for a connection to succeed.
+    // note: even when server prints 'Listening on localhost:8964...Press Ctrl-C to quit.'
+    //       it might not be listening yet...need to wait until fsockopen() call returns
+    $error = "Unable to connect to server\n";
+    for ($i=0; $i < 60; $i++) {
+        usleep(50000); // 50ms per try
+        $status = proc_get_status($handle);
+        $fp = @fsockopen("tcp://$bound");
+        // Failure, the server is no longer running
+        if (!($status && $status['running'])) {
+            $error = "Server is not running\n";
             break;
         }
-
-        if ($status['stopped']) {
-            $error =
-                "Server was stopped with {$status['stopsig']}\n";
+        // Success, Connected to servers
+        if ($fp) {
+            $error = '';
             break;
         }
-
-        // note: here we check the server is listening, even when the server prints
-        //          listening on %s:%d
-        //       it may not be ready to accept connections
-        $start = time();
-
-        for ($try = 0; $try < 120; $try++) {
-            $error = @fsockopen(
-                        PHP_CLI_SERVER_HOSTNAME, PHP_CLI_SERVER_PORT) ?
-                            null :
-                            sprintf(
-                                "Server is not accepting connections after %d seconds\n",
-                                time() - $start);
-
-            if (!$error) {
-                break 2;
-            }
-
-            usleep(50000);
-        }
-
-        break;
     }
 
-php_cli_server_start_error:
     if ($error) {
         echo $error;
         proc_terminate($handle);
@@ -87,14 +90,21 @@ php_cli_server_start_error:
     }
 
     register_shutdown_function(
-        function($handle) use($router) {
+        function($handle) use($router, $doc_root) {
             proc_terminate($handle);
             @unlink(__DIR__ . "/{$router}");
+            @rmdir($doc_root);
         },
         $handle
     );
 
-    return $handle;
+    // Define the same "constants" we previously did.
+    $port = (int) substr($bound, strrpos($bound, ':') + 1);
+    define("PHP_CLI_SERVER_HOSTNAME", "localhost");
+    define("PHP_CLI_SERVER_PORT", $port);
+    define("PHP_CLI_SERVER_ADDRESS", PHP_CLI_SERVER_HOSTNAME.":".PHP_CLI_SERVER_PORT);
+
+    return new CliServerInfo($doc_root);
 }
 
 function php_cli_server_connect() {
index e1fbd9039817b634611280226425f40ddf5934db..2daf6aad731073b33e13ca780d651fa843b487ec 100644 (file)
@@ -13,8 +13,8 @@ php_cli_server_start('var_dump($_SERVER["DOCUMENT_ROOT"], $_SERVER["SERVER_SOFTW
 var_dump(file_get_contents("http://" . PHP_CLI_SERVER_ADDRESS));
 ?>
 --EXPECTF--
-string(%d) "string(%d) "%stests"
+string(%d) "string(%d) "%sphp_cli_server_002"
 string(%d) "PHP %s Development Server"
 string(%d) "localhost"
-string(%d) "8964"
+string(%d) "%s"
 "
index 6262aa2c734e105e99700925f9e6e44517cfcb5f..9824c63c404085b6eb9060c6cfe0f6720783e20f 100644 (file)
@@ -7,8 +7,8 @@ include "skipif.inc";
 --FILE--
 <?php
 include "php_cli_server.inc";
-php_cli_server_start('print_r($_REQUEST); $_REQUEST["foo"] = "bar"; return FALSE;');
-$doc_root = __DIR__;
+$info = php_cli_server_start('print_r($_REQUEST); $_REQUEST["foo"] = "bar"; return FALSE;');
+$doc_root = $info->docRoot;
 file_put_contents($doc_root . '/request.php', '<?php print_r($_REQUEST); ?>');
 
 $host = PHP_CLI_SERVER_HOSTNAME;
index af48758306443ff56f9fa22cabce3d2fb1cacae9..5821fdbbc2e95fc2cff8d0ae80426fda7d20bf08 100644 (file)
@@ -9,10 +9,9 @@ display_errors=1
 --FILE--
 <?php
 include "php_cli_server.inc";
-php_cli_server_start('require("syntax_error.php");');
-$dir = realpath(__DIR__);
+$doc_root = php_cli_server_start('require("syntax_error.php");')->docRoot;
 
-file_put_contents($dir . "/syntax_error.php", "<?php non_exists_function(); ?>");
+file_put_contents($doc_root . "/syntax_error.php", "<?php non_exists_function(); ?>");
 
 $output = '';
 $host = PHP_CLI_SERVER_HOSTNAME;
@@ -30,7 +29,7 @@ HEADER
     }
 }
 echo $output;
-@unlink($dir . "/syntax_error.php");
+@unlink($doc_root . "/syntax_error.php");
 fclose($fp);
 ?>
 --EXPECTF--
index e5ffe3bf6cbe3a81b3a24fe539c3a63975ec2da6..c89dfd49da260dac2f953af75b10ee4c8559cbea 100644 (file)
@@ -2,8 +2,6 @@
 php://input is empty when enable_post_data_reading=Off
 --INI--
 allow_url_fopen=1
---CONFLICTS--
-server
 --SKIPIF--
 <?php
 include __DIR__."/../../sapi/cli/tests/skipif.inc";