Skip to content

Commit 988283f

Browse files
committed
docs: improves connectionTimeout explanatory comment
1 parent 7fa1121 commit 988283f

File tree

1 file changed

+6
-2
lines changed

1 file changed

+6
-2
lines changed

__tests__/integration/configuration/config-fixture.js

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,12 @@ const config = {
2222
gapLimit: null,
2323

2424
/*
25-
* Usually the connectionTimeout option should be `null`, but to increase the stability of the
26-
* integration tests we are adding a very long timeout here.
25+
* When we leave this as null, we use the connectionTimeout default, which is 5000, i.e. 5s.
26+
*
27+
* This timeout is the maximum time that the websocket waits for a PONG message after sending
28+
* a PING message before assuming the full node has gone down. With the currently low performance
29+
* of tests, the full node is being fully loaded and we are increasing this timeout in order to
30+
* prevent any wrong websocket reload.
2731
*/
2832
connectionTimeout: 30000,
2933
allowPassphrase: false,

0 commit comments

Comments
 (0)