Merge pull request #8995 from kenjis/fix-command-loads-rewrite.php

fix: `command()` may execute `rewrite.php`

Author: kenjis

system/Commands/Server/Serve.php

@@ -103,7 +103,7 @@ public function run(array $params)
         $docroot = escapeshellarg(FCPATH);
 
         // Mimic Apache's mod_rewrite functionality with user settings.
-        $rewrite = escapeshellarg(__DIR__ . '/rewrite.php');
+        $rewrite = escapeshellarg(SYSTEMPATH . 'rewrite.php');
 
         // Call PHP's built-in webserver, making sure to set our
         // base path to the public folder, and to use the rewrite file

system/Commands/Server/rewrite.php renamed to system/rewrite.php

@@ -21,11 +21,6 @@
  */
 
 // @codeCoverageIgnoreStart
-// Avoid this file run when listing commands
-if (PHP_SAPI === 'cli') {
-    return;
-}
-
 $uri = urldecode(
     parse_url('https://codeigniter.com' . $_SERVER['REQUEST_URI'], PHP_URL_PATH) ?? ''
 );