Skip to content

chore: remove channel message relay timeout#1709

Merged
derklaro merged 1 commit intonightlyfrom
channel-message-relay-timeout
Jul 31, 2025
Merged

chore: remove channel message relay timeout#1709
derklaro merged 1 commit intonightlyfrom
channel-message-relay-timeout

Conversation

@derklaro
Copy link
Member

Motivation

Currently, a channel message query must respond within 20 seconds of being redirected into the cluster. This is not ideal as some handlers might need more time.

Modification

Remove forced 20 second channel message relay timeout. It is now up to the caller to apply a timeout to channel message queries, allowing for higher timeouts to be set.

Result

No more forced 20 second response window for channel message queries.

@derklaro derklaro added this to the 4.0.0-RC14 milestone Jul 28, 2025
@derklaro derklaro requested a review from 0utplay July 28, 2025 13:39
@derklaro derklaro self-assigned this Jul 28, 2025
@derklaro derklaro added v: 4.X This pull should be included in the 4.0 release in: node An issue/pull request releated to the node module code t: improvement The pull request improves existing code labels Jul 28, 2025
@github-actions
Copy link

Test Results

 56 files  ±0   56 suites  ±0   2m 10s ⏱️ -30s
570 tests ±0  570 ✅ ±0  0 💤 ±0  0 ❌ ±0 
935 runs  ±0  935 ✅ ±0  0 💤 ±0  0 ❌ ±0 

Results for commit 372404e. ± Comparison against base commit c7a789c.

@derklaro derklaro merged commit 9ca3f3c into nightly Jul 31, 2025
6 checks passed
@derklaro derklaro deleted the channel-message-relay-timeout branch July 31, 2025 12:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

in: node An issue/pull request releated to the node module code t: improvement The pull request improves existing code v: 4.X This pull should be included in the 4.0 release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants