Skip to content

Add additional parameter shell to run/run_generic

Danny Willems requested to merge nomadic-labs/tezos:dannywillems@add-shell-param-in-run-python into master

Require when a parameter contains quotes. See https://docs.python.org/3.6/library/subprocess.html#security-considerations

A test in Sapling (see https://gitlab.com/cryptiumlabs/tezos/-/merge_requests/169 and https://gitlab.com/cryptiumlabs/tezos/-/blob/67fd51fb6b08837bb3f59754a61690629ad8c65f/tests_python/client/client.py#L906) does use this feature. Useful for commands like:

tezos-client sapling forge transaction using shielded-tez from alice sending \
  "3 to  zet13DPAZqRRtU9j6u9U35cZrrW4McQBQqun8cMciRzdcLjwNjFrPRnCLUX3CEiVugSQ5 with memo hell0" \
  "210 to zet13yz6m1RTCrRtqMyyPzyoJNzF42D3b2VpJ1uPeSyGXHKH7Bvj4akc2pSVvx3DuQyzC with memo hell1" \
  --json

This commit will be cherry-picked for 169.

Edited by Danny Willems

Merge request reports