Give up a connect after connectTimeout instead of waiting the OS out #12

Merged

lilleman merged 7 commits from connect-timeout into main 2026-09-20 20:58:14 +02:00

Conversation 0 Commits 7 Files Changed 10

+260 -40

lilleman commented 2026-09-20 00:42:24 +02:00

Owner

Copy Link

Copy Source

A host that drops SYNs stalled every attempt for the OS TCP timeout, which reconnect could not retry. Every connect attempt is now bounded at 10 s, the TLS handshake included; connectTimeout retunes it, and false restores the operating system's wait.

A host that drops SYNs stalled every attempt for the OS TCP timeout, which reconnect could not retry. Every connect attempt is now bounded at 10 s, the TLS handshake included; connectTimeout retunes it, and false restores the operating system's wait.

lilleman added 1 commit 2026-09-20 00:42:24 +02:00

Give up a connect after connectTimeout instead of waiting the OS out 17447bbe3a

lilleman added 1 commit 2026-09-20 18:12:41 +02:00

Refuse a connectTimeout Node's timers cannot hold, and pin the refusals the tests missed 84174b4a5b

lilleman added 1 commit 2026-09-20 18:17:41 +02:00

Name the peer and the stalled phase on a connect timeout, and quote an untyped value 8f416c2bc4

lilleman added 1 commit 2026-09-20 20:34:55 +02:00

Bound every connect attempt at 10 s by default, with false the way to opt out 1416615ca3

lilleman added 1 commit 2026-09-20 20:41:48 +02:00

Prove the default bound is wired, and name the option on the failure a consumer reads a24f350b7a

lilleman added 1 commit 2026-09-20 20:46:45 +02:00

Bound the wait in the default test, and pin the boolean a refusal prints 45171c2697

lilleman added 1 commit 2026-09-20 20:52:33 +02:00

Wait the default out for real, since node:test mock timers postdate the Node 18 floor 92fbd6a9f2

lilleman marked the pull request as ready for review 2026-09-20 20:57:35 +02:00

lilleman merged commit 92fbd6a9f2 into main 2026-09-20 20:58:14 +02:00

lilleman deleted branch connect-timeout 2026-09-20 20:58:14 +02:00

Sign in to join this conversation.

Reviewers

No Reviewers

Labels

No items

No Label

Milestone

No items

No Milestone

Assignees

Clear assignees

claes.eismar.se lilleman (Lilleman auf Larv) martin.siden

No Assignees

1 Participants

Notifications

Subscribe

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: larvit/smpp-js#12