temporalio / cli

Command-line interface for running Temporal Server and interacting with Workflows, Activities, Namespaces, and other parts of Temporal
https://docs.temporal.io/cli
MIT License
247 stars 34 forks source link

Do not use payload shorthand on workflow show #505

Closed cretz closed 5 months ago

cretz commented 5 months ago

What was changed

We use "JSON payload shorthand" by default in our JSON protos throughout the CLI. Payloads are notoriously hard to use as base64 input/output, so this helps others, but for workflow show -o json specifically it makes the JSON incompatible with SDK replayers. So we have disabled the default (and there's no real way to put the default back at this time, though we can discuss in the future if needed).