Describe the bug
Currently, we do manual escaping in ProcessInvoker and its various overloads for arguments. We should move towards using the ArgumentListproperty which automatically handles escaping, as opposed to doing it by hand, which introduces opportunity of human error.
This is likely a large set of changes, we should put it behind a feature flag to ensure a safe rollout if we can.
See related prs in this area recently, and the source code for how we invoke process
Describe the bug
Currently, we do manual escaping in
ProcessInvokerand its various overloads forarguments. We should move towards using theArgumentListproperty which automatically handles escaping, as opposed to doing it by hand, which introduces opportunity of human error.This is likely a large set of changes, we should put it behind a feature flag to ensure a safe rollout if we can.
See related prs in this area recently, and the source code for how we invoke process