Revert "Set $_SERVER['SCRIPT_NAME'] within proxy command"

[mvorisek]

reverts #11562

fixes #11615

why the PR should be reverted?

• $_SERVER['SCRIPT_NAME'] is expected to be the entry file, this is a php-src convenction and it should be safe to assume it no matter how the script is started
• the entry file path must match the first path in a backtrace, otherwise shortening backtraces might not work (as in Longer backtrace when installed using 2.6.0 #11615)
• original problem Change "outdated" color semantic #5447 mentioned https://github.com/sebastianbergmann/phpunit/blob/15a89f123d/build/templates/binary-phar-autoload.php.in#L58-L62 - so the fix was done to fix this - incorrect/not always valid assumption - and the proper fix should be to fix sebastianbergmann/phpunit instead (and possibly with some added global/API in composer like "entry target file" in https://github.com/mvorisek/composer/blob/6200985c2a/src/Composer/Installer/BinaryInstaller.php#L224)

[Seldaek]

Yeah I would still like to understand and look at the code producing the backtrace which causes your failure. Is there any chance you have a link to that?

[mvorisek]

here is a repo: https://github.com/atk4/core with CI/test failure starting with composer 2.6.0

the actual single usage should not matter much, as more projects can rely on native php-src properties

[Seldaek]

It matters to me to understand what kind of code causes the failure.. I know the repo is there but I didn't figure out from a quick look the source of the problem, as I am not familiar with it at all. I imagine it'd be easier for you to find out, so it'd be nice if you help.

[mvorisek]

Sure and thank you.

in phpunit printer the stack trace is redacted by
https://github.com/sebastianbergmann/phpunit/blob/899db92716/src/Framework/ExceptionWrapper.php#L53C58-L53C58
which calls
https://github.com/sebastianbergmann/phpunit/blob/899db92716/src/Util/Filter.php#L64
which redacts the stack trace lines in
https://github.com/sebastianbergmann/phpunit/blob/899db92716/src/Util/Filter.php#L90
and https://github.com/sebastianbergmann/phpunit/blob/899db92716/src/Util/Filter.php#L98

also realpath() is slow (requires many IO syscalls) and can even fail (silently, realpath returns false) if the $binPathExported is not on physical disk - repro https://3v4l.org/V9GMD (see last line, wrapper is copied example from https://www.php.net/manual/en/stream.streamwrapper.example-1.php)

[Seldaek]

Ok yeah I see, thanks.