-
Notifications
You must be signed in to change notification settings - Fork 0
Closed
Labels
area:runtime-nodeArea: Node runtime bridgeArea: Node runtime bridgepriority:p2Priority P2 (medium)Priority P2 (medium)
Milestone
Description
Problem
timeoutMs is accepted without validation. If callers pass NaN, Infinity, or negative values, the per-request timeout timer may fire immediately or never fire, breaking timeout enforcement. NodeBridge has similar weak validation but timeoutMs is used more widely in OptimizedNodeBridge (queue wait, timeouts, TTL).
Evidence
src/runtime/optimized-node.ts:timeoutMsassigned directly to options with no normalization; used insetTimeoutand TTL calculations.
Acceptance criteria
- Normalize invalid
timeoutMsto a safe default (e.g., 30000) or clamp to a minimum > 0. - Add tests for
NaN,Infinity, negative values.
Metadata
Metadata
Assignees
Labels
area:runtime-nodeArea: Node runtime bridgeArea: Node runtime bridgepriority:p2Priority P2 (medium)Priority P2 (medium)