You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
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
The text was updated successfully, but these errors were encountered:
Describe the bug
Currently, we do manual escaping in
ProcessInvoker
and its various overloads forarguments
. We should move towards using theArgumentList
property 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
The text was updated successfully, but these errors were encountered: