temporalio / sdk-core

Core Temporal SDK that can be used as a base for language specific Temporal SDKs
MIT License
262 stars 70 forks source link

[Bug] Core doesn't expose the `cause` from `WorkflowExecutionCancelRequestedEventAttributes` #708

Open bergundy opened 5 months ago

bergundy commented 5 months ago

Event has a reason:

https://github.com/temporalio/api/blob/6606687fb5ca8d899fe3e93313d53d9c7ed2a379/temporal/api/history/v1/message.proto#L403

Activation Job has details:

https://github.com/temporalio/sdk-core/blob/4840a9c570fa3388e190e20f01f61065d0b7e965/sdk-core-protos/protos/local/temporal/sdk/core/workflow_activation/workflow_activation.proto#L242

Must have been a mix up with WorkflowExecutionCanceledEventAttributes:

https://github.com/temporalio/api/blob/6606687fb5ca8d899fe3e93313d53d9c7ed2a379/temporal/api/history/v1/message.proto#L414