Changeset 60518

Timestamp:

07/28/2025 12:52:27 PM (4 months ago)

Author:

johnbillion

Message:

Build/Test Tools: Don't run WP-CLI commands as root in the local development environment and CI environment.

Props johnbillion, westonruter.

See #63167, #63564

Location:

trunk

Files:

• package.json (modified) (1 diff)
• tools/local-env/scripts/install.js (modified) (1 diff)

trunk/package.json

@@ -184 +184 @@
         "env:reset": "node ./tools/local-env/scripts/docker.js down --rmi all -v --remove-orphans",
         "env:install": "node ./tools/local-env/scripts/install.js",
-        "env:cli": "node ./tools/local-env/scripts/docker.js exec cli wp --allow-root",
+        "env:cli": "node ./tools/local-env/scripts/docker.js exec --user wp_php cli wp",
         "env:logs": "node ./tools/local-env/scripts/docker.js logs",
         "env:pull": "node ./tools/local-env/scripts/docker.js pull",

trunk/tools/local-env/scripts/install.js

@@ -15 +15 @@
 // Create wp-config.php.
 wp_cli( `config create --dbname=wordpress_develop --dbuser=root --dbpass=password --dbhost=mysql --force --config-file="wp-config.php"` );
-
-// Since WP-CLI runs as root, the wp-config.php created above will be read-only. This needs to be writable for the sake of E2E tests.
-execSync( 'node ./tools/local-env/scripts/docker.js exec cli chmod 666 wp-config.php' );
 
 // Add the debug settings to wp-config.php.