php-cli (v1.13.0): $argv[0] is "frankenphp" and the script path is shifted to $argv[1] — breaks Symfony Console
I maintainer di solito rispondono entro 1 giorno
Valutazione
- Difficoltà
- 3/5
- Tempo stimato
- 1-2 giorni
- Idoneità per principianti
- 30/100
Direzione di ricerca
Start by reading caddy/php-cli.go and trace how its argument vector reaches frankenphp_execute_script_cli and do_php_cli. Compare the resulting $argv and $argc with native PHP and FrankenPHP 1.12.7 using the reproduction in the issue. Done means php-cli exposes the script path as $argv[0] and the remaining arguments in the same positions as native PHP.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
What happened?
Since v1.13.0, frankenphp php-cli script.php args... exposes the binary name as $argv[0] and shifts the script path to $argv[1] ($argc grows by one). Native php and FrankenPHP ≤ 1.12.7 expose the script path as $argv[0], as documented for the CLI SAPI.
Minimal reproduction (same PHP 8.5.11, only FrankenPHP differs):
<?php printf("PHP %s SAPI=%s argc=%d argv=%s\n", PHP_VERSION, PHP_SAPI, $argc, json_encode($argv));
# dunglas/frankenphp:1.12.7-php8.5
php /t/argv.php foo bar → PHP 8.5.11 SAPI=cli argc=3 argv=["/t/argv.php","foo","bar"]
frankenphp php-cli /t/argv.php foo bar → PHP 8.5.11 SAPI=cli argc=3 argv=["/t/argv.php","foo","bar"]
# dunglas/frankenphp:1.13.0-php8.5
php /t/argv.php foo bar → PHP 8.5.11 SAPI=cli argc=3 argv=["/t/argv.php","foo","bar"]
frankenphp php-cli /t/argv.php foo bar → PHP 8.5.11 SAPI=cli argc=4 argv=["frankenphp","/t/argv.php","foo","bar"]
$_SERVER['argv'] shows the same shift. SCRIPT_NAME, PHP_SELF and SCRIPT_FILENAME are correct in both versions. Passing -- does not help (-- then shows up inside $argv).
Impact: anything that treats $argv[0] as the script and $argv[1] as the first argument breaks. The most visible case is Symfony Console: ArgvInput drops argv[0] and takes the next element as the command name, so every command fails with
Command "/app/bin/console" is not defined.
This affects frankenphp php-cli bin/console doctrine:migrations:migrate, messenger:consume, messenger:stop-workers, etc. The HTTP server and worker mode are unaffected.
Likely cause: #1757 ("simply use the built-in php-cli"). caddy/php-cli.go now builds the argument vector as append([]string{os.Args[0]}, os.Args[2:]...) (the binary name is prepended so option parsing works like php -i) and passes it unchanged through frankenphp_execute_script_cli to do_php_cli(argc, argv). The real php binary shifts the vector after option parsing so that the script becomes $argv[0]; that shift does not happen on this code path, so user scripts see the raw vector including the binary name. The pre-1.13 emulation built the vector starting at the script path, which is why it was correct before.
Expected: frankenphp php-cli script.php a b gives $argv === ["script.php","a","b"], exactly like php script.php a b.
Build Type
Official Docker image (dunglas/frankenphp:1.13.0-php8.5); also reproduced with a custom build of v1.13.0.
Worker Mode
No — CLI only (php-cli), unrelated to worker mode.
Operating System
GNU/Linux (Debian-based official image; host: Linux x86_64, podman 5.8)
CPU Architecture
x86_64
PHP configuration
phpinfo() output
PHP 8.5.11 (bundled in the official image), default php.ini, no custom extensions needed to reproduce. The repro script above has no dependencies.
Relevant log output
Relevant log output
$ podman run --rm -v $PWD/argv.php:/t/argv.php:ro dunglas/frankenphp:1.13.0-php8.5 frankenphp version
FrankenPHP v1.13.0 PHP 8.5.11 Caddy v2.11.7 h1:yj0Y4fYZGPkSvibBJ1sTWE33xC0fxztVyXEW5iIdUT4=
$ podman run --rm -v $PWD/argv.php:/t/argv.php:ro dunglas/frankenphp:1.13.0-php8.5 frankenphp php-cli /t/argv.php foo bar
PHP 8.5.11 SAPI=cli argc=4 argv=["frankenphp","\/t\/argv.php","foo","bar"]
$ podman run --rm -v $PWD/argv.php:/t/argv.php:ro dunglas/frankenphp:1.12.7-php8.5 frankenphp php-cli /t/argv.php foo bar
PHP 8.5.11 SAPI=cli argc=3 argv=["\/t\/argv.php","foo","bar"]
# Symfony 7.4 app, same container, v1.13.0:
$ frankenphp php-cli /app/bin/console doctrine:migrations:migrate -n
Command "/app/bin/console" is not defined.
- Lingua principale
- Go
- Stelle
- 11.4k
- Fork
- 487
- Merge medio
- 2g 19h
- PR unite (30g)
- 20
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Nessun modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di php/frankenphp
-
Segfault of the whole process when restarting an unhealthy thread: SG(server_context) is read after ts_free_thread()Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 76/100
php/frankenphp#2688 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 90/100
php/frankenphp#2671 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
Include frankenphp version in logsForse già presa @ousamabenyounes l’ha presa 78 giorni fa. Apertaenhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
php/frankenphp#2483 · 3 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
suggestion about compile.mdForse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. Apertaenhancement
Difficoltà 1/5 1-3 ore Idoneità per principianti 68/100
php/frankenphp#601 · 5 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
bug
Difficoltà 4/5 3-5 giorni Idoneità per principianti 35/100
php/frankenphp#2695 ·
I maintainer di solito rispondono entro 1 giorno
Tutte le issue di php/frankenphp
Issue simili
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 88/100
-
Difficoltà 2/5 1-3 ore Idoneità per principianti 67/100
vanderheijden86/b9s#20 ·
-
go-battery needs an ndsctl on PATH: TestPurchaseSessionGuardHoldsThroughTheOutcomeUnknownWindow fails on bare hosts (passes with stub)Forse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 78/100
OpenTollGate/tollgate-module-basic-go#726 ·
I maintainer di solito rispondono entro 1 giorno
-
ux waiting for feedback
Difficoltà 2/5 1-3 ore Idoneità per principianti 63/100
evcc-io/evcc#34527 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
phase:v3 type:harness
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
I maintainer di solito rispondono entro 1 giorno