In the ConnectionManager library, the following code is used in the constructor:
// Set the timeout policy + disconnect if required server.setsendtimeoutpolicy(RETURN_ON_ERROR, WAIT_TIL_SENT, 0);
is the use of zero for timeout a wise choice ? The documentation of the server.setsendtimeoutpolicy ImpApi function is actually warning against the use of zero
For now I override it by calling
server.setsendtimeoutpolicy(RETURN_ON_ERROR, WAIT_TIL_SENT, 30);
again after the instantiation of the class.
Can I suggest to make it nonzero by default, or adding a setting in the class constructor (or use the already provided watchdog timeout setting) ?