Interval between keep alive missed and notifying transport seems to small #1211

Closed
davidfowl opened this Issue Dec 29, 2012 · 1 comment

Comments

Projects
None yet
3 participants
Owner

davidfowl commented Dec 29, 2012

[11:43:58 GMT-0800 (Pacific Standard Time)] SignalR: Keep alive has been missed, connection may be dead/slow. jquery.signalR.js:54
[11:44:12 GMT-0800 (Pacific Standard Time)] SignalR: Keep alive timed out.  Notifying transport that connection has been lost. 

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jan 2, 2013

@NTaylorMullen NTaylorMullen Increased KeepAlive timeout threshold to be 3 keep alives
- This will also raise the timeout warn threshold
- I was unable to see scenarios when keep alive warn was being triggered
inaccurately but raising this threshold will ensure correctness

#1211
054119b

@NTaylorMullen NTaylorMullen added a commit that referenced this issue Jan 2, 2013

@NTaylorMullen NTaylorMullen Minified/compiled keep alive threshold increases
- Note, #1211 is unable to be JS unit tested at this time due to not
having control over the server keep alive configuration via the client.
- Reviewed by @shalter
a8cc480
Contributor

Xiaohongt commented Jan 3, 2013

verified

Xiaohongt closed this Jan 3, 2013

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