when set GlobalHost.Configuration.KeepAlive = null on server, client still call checkIfAlive recursively but it is not useful since keepAlive never can time out because keepAliveData.timeout is undefined #840

Closed
Xiaohongt opened this Issue Oct 22, 2012 · 1 comment

Comments

Projects
None yet
2 participants
Contributor

Xiaohongt commented Oct 22, 2012

when set GlobalHost.Configuration.KeepAlive = null on server, client still alwasy call checkIfAlive recursively but it is not useful since keepAlive never can time out.

Because in this case, keepAliveData.timeout is undefined, keepAliveData.timeoutWarning is undefined in this case,

inside function checkIfAlive(connection):
(timeElapsed >= keepAliveData.timeout) and (timeElapsed >= keepAliveData.timeoutWarning) always are false

@ghost ghost assigned NTaylorMullen Oct 22, 2012

NTaylorMullen pushed a commit that referenced this issue Oct 22, 2012

Contributor

Xiaohongt commented Oct 22, 2012

verified

@Xiaohongt Xiaohongt closed this Oct 22, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment