Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork5.3k
[Process] allow writing "prepared" command line.#9295
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
…nes (Simperfit)This PR was merged into the 4.1-dev branch.Discussion----------[Process] Allow writing portable "prepared" command lines| Q | A| ------------- | ---| Branch? | 4.1| Bug fix? | no| New feature? | yes| BC breaks? | no| Deprecations? | no| Tests pass? | yes| Fixed tickets |#23778| License | MIT| Doc PR |symfony/symfony-docs#9295This give the opportunity to create process commands that allow to changes only the values instead of changing the code.Commits-------d1e4f48 [Process] Allow writing portable "prepared" command lines
| $process = new Process('/usr/bin/php {{ phpFile }}'); | ||
| $process->run(null, array('phpFile' => 'worker.php')); | ||
| Which means that you can use placeholder in order to have a process that can be changed only with the values and without changing the php code. |
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.
it should be noted whether the user of this api is still responsible to escape the passed in data or whether the console component will take care of escaping (e.g.> character or similar)
Simperfit commentedMar 6, 2018
We need to rework the feature, so this is not relevant anymore. |
symfony/symfony#24763