Skip to content
This repository was archived by the owner on Jan 5, 2026. It is now read-only.

Update TimeoutException message - #6773

Merged
tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/update/timeout-exception
Apr 11, 2024
Merged

tracyboehrer (tracyboehrer) merged 1 commit into
mainfrom
southworks/update/timeout-exception

Conversation

@sw-joelmut

Copy link
Copy Markdown
Collaborator

#minor

Description

This PR detects when a timeout happens while streaming, it will throw a new OperationCanceledException explaining why the timeout happened.

Specific Changes

  • Adds 2 messages when timing out, when the streaming connection got disconnected, and just when the request didn't receive a response in the defined time frame.

Testing

The following image shows the bot capturing the new TimeoutException message.
image

@sw-joelmut
Joel Mut (sw-joelmut) requested a review from a team as a code owner April 10, 2024 20:01
@sw-joelmut Joel Mut (sw-joelmut) added the Automation: No parity PR does not need to be applied to other languages. label Apr 10, 2024
@tracyboehrer
tracyboehrer (tracyboehrer) deleted the southworks/update/timeout-exception branch April 11, 2024 13:17
This was referenced May 14, 2026
This was referenced May 14, 2026
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Automation: No parity PR does not need to be applied to other languages.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants