Skip to content

Commit 8fd4988

Browse files
authored
fix(commissioner): extend operational connect timeout to 4m15s (#3674)
* fix(commissioner): extend operational connect timeout to 4m15s Several commissioning steps allow up to ~2 minute server-side delays. The previous 4 minute CASE connect timeout left too little headroom in back-to-back delay scenarios. Bumping to 255 seconds keeps us safely inside the 5 minute failsafe re-arm and matches observed worst-case peer behavior. Failsafe reset on operational connect failure already fires through the existing CommissioningError catch path. * docs(commissioner): note rationale for 4m15s connect timeout Two ~2-minute server-side retry windows fit within 4m15s before we abort the operational connect.
1 parent 5cfede6 commit 8fd4988

1 file changed

Lines changed: 2 additions & 2 deletions

File tree

packages/protocol/src/peer/ControllerCommissioner.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -40,7 +40,6 @@ import {
4040
Logger,
4141
MaybePromise,
4242
Millis,
43-
Minutes,
4443
NoResponseTimeoutError,
4544
Seconds,
4645
ServerAddress,
@@ -583,7 +582,8 @@ export class ControllerCommissioner {
583582
const peer = this.#context.peers.for(address);
584583
peer.descriptor.discoveryData = discoveryData;
585584
await peer.connect({
586-
connectionTimeout: Minutes(4),
585+
// 4m15s allows two ~2-minute server-side retry windows to complete before we abort.
586+
connectionTimeout: Seconds(255),
587587
timing: caseConnectionTiming,
588588

589589
handleError: error => {

0 commit comments

Comments
 (0)