Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork9.7k
[Process] Fix finding executables independently of open_basedir#58291
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to ourterms of service andprivacy statement. We’ll occasionally send you account related emails.
Already on GitHub?Sign in to your account
Uh oh!
There was an error while loading.Please reload this page.
Conversation
nicolas-grekas commentedSep 17, 2024
Thank you@BlackbitDevs. |
fritzmg left a comment• edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others.Learn more.
But shouldn't theopen_basedir paths be checkedadditionally? After all,open_basedir could contain other paths with available executables thanPATH. This is why I did not consider just backporting the state of Symony 6+ to 5.4.
nicolas-grekas commentedSep 17, 2024
That'd be incorrect, open_basedir is not meant to replace PATH |
fritzmg commentedSep 17, 2024
That was not my point. However, since the usage of |
stof commentedSep 17, 2024
@fritzmg open_basedir is not about hinting about the location of executables at all. |
fritzmg commentedSep 17, 2024 • edited
Loading Uh oh!
There was an error while loading.Please reload this page.
edited
Uh oh!
There was an error while loading.Please reload this page.
Not directly, no. But if a hosting environment employs |
This backports#47422 to 5.4, which is a bugfix really.
Instead of#58008 and#57954 /cc@xabbuh@fritzmg