Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Transport timeout too short cause Transport close.. #403

Closed
binlaniua opened this issue Aug 24, 2017 · 4 comments · May be fixed by dyna-dot/create-react-app#9, anyulled/rhte-demo#3 or saurabharch/create-react-app#3

Comments

@binlaniua
Copy link

binlaniua commented Aug 24, 2017

my connect code

var socket;
if (that.isIE9()) {
	socket =  new SockJS(url)
} else {
	socket =  new SockJS(url, null, {transports:'websocket'})
}

i found code at "SockJS.prototype._connect"

var timeoutMs = (this._rto * Transport.roundTrips) || 5000;
this._transportTimeoutId = setTimeout(this._transportTimeout.bind(this), timeoutMs);

i log the timeoutMs is 698

image

in "websocket" my server reponse may be than this time... how to change it , did it has a setting ?

antonselukh pushed a commit to antonselukh/sockjs-client that referenced this issue Nov 7, 2017
@antonselukh
Copy link

Let's see if this pull request solves the issue. I had similar client issues

antonselukh added a commit to antonselukh/sockjs-client that referenced this issue Nov 7, 2017
sockjs#403 Provide SockJS config for minimum transport fallback timeout
lennertr added a commit to lennertr/sockjs-client that referenced this issue Oct 29, 2018
@dazraf
Copy link

dazraf commented Oct 29, 2018

Hi, are the PR 412 and 413 sufficient to close this issue and release this feature in the next release please? thanks.

@chrislhardin
Copy link

1.3.0 doesn't have the transport timeout configuration feature. Is there a beta release or do I just need to build my own until it gets released? The current calculated timeout is way too short for my app.

shengbeiniao pushed a commit to shengbeiniao/sockjs-client that referenced this issue Jan 21, 2019
shengbeiniao pushed a commit to shengbeiniao/sockjs-client that referenced this issue Jan 21, 2019
@brycekahle
Copy link
Contributor

@chrislhardin The referenced PR #413 has not been merged, thus it is not in master yet.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
5 participants