Skip to content

feat: handle CanceledException as TimeoutException#709

Open
xepozz wants to merge 1 commit intomasterfrom
timeout-exc
Open

feat: handle CanceledException as TimeoutException#709
xepozz wants to merge 1 commit intomasterfrom
timeout-exc

Conversation

@xepozz
Copy link
Collaborator

@xepozz xepozz commented Feb 9, 2026

What was changed

Why?

Synced with another usage:

try {
$response = $this->client->PollWorkflowExecutionUpdate($request, $context);
} catch (TimeoutException|CanceledException $e) {
throw WorkflowUpdateRPCTimeoutOrCanceledException::fromTimeoutOrCanceledException($e);
}

As we can see the description in the \Temporal\Exception\Client\WorkflowUpdateRPCTimeoutOrCanceledException:

/**
* Occurs when an update call times out or is cancelled.
*
* @note this is not related to any general concept of timing out or cancelling a running update,
* this is only related to the client call itself.
*/

Checklist

  1. Closes

  2. How was this tested:

  1. Any docs updates needed?

@xepozz xepozz requested a review from wolfy-j as a code owner February 9, 2026 14:37
@vercel
Copy link

vercel bot commented Feb 9, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
php Ready Ready Preview, Comment Feb 9, 2026 2:38pm

Request Review

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments