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

Streaming unreliable on Opera in 1.0.12 #1044

Closed
ghost opened this issue Apr 24, 2013 · 12 comments
Closed

Streaming unreliable on Opera in 1.0.12 #1044

ghost opened this issue Apr 24, 2013 · 12 comments
Labels

Comments

@ghost
Copy link

ghost commented Apr 24, 2013

Streaming will sometimes deliver incorrect messages on Opera 12. Seems to me that the special case for Opera when streaming does not use _trackMessageSize at all, which would explain the problems.

@jfarcand
Copy link
Member

Have you try 1.0.13-SNAPSHOT by any chance?

@Artur-
Copy link
Contributor

Artur- commented Apr 25, 2013

Tried also with 1.0.13-SNAPSHOT javascript - still reproducible. It is not the same issue as #1027. The issue can be seen here: http://artur.virtuallypreinstalled.com/TablePush/?debug

jfarcand added a commit that referenced this issue Apr 25, 2013
@jfarcand
Copy link
Member

Heavily tested using [this][(https://gist.github.com/jfarcand/5374686). Let me know how it goes.

@Artur-
Copy link
Contributor

Artur- commented Apr 26, 2013

Awesome work. With the fixed version I no longer see any issues in my test.

@Legioth
Copy link
Contributor

Legioth commented Apr 26, 2013

Still seeing problems with this. It seems like the connection is always reconnected after calling push from the client.

@jfarcand
Copy link
Member

@Legoth: Share a test case, as I can't reproduce this issue. Are you using streaming or ling-polling?

@Artur-
Copy link
Contributor

Artur- commented Apr 29, 2013

Is there something in the fix that potentially cause severe regressions in streaming in at least IE10 and Opera 12? Testing with the lastest 1.0.13-SNAPSHOT js (1.0.12 server side) I see messages received multiple times on the server and possibly on the client also.

@Artur-
Copy link
Contributor

Artur- commented Apr 29, 2013

Downgrading to 1.0.12 fixes the issue. It's so bad that IE10 will crash after a few minutes of pushing back and forth at full speed.

@jfarcand
Copy link
Member

@artur: You said you tested above and nothing changed since then. So are you saying both opera and IE 10 are broken? Will take a look soon.

@jfarcand
Copy link
Member

Also, can you share your test case? I've tested with the gist above (simulate cache etc.) and it worked....

@Artur-
Copy link
Contributor

Artur- commented Apr 29, 2013

I think how I tested previously above did not show this regression.

The issue can be seen at http://artur.virtuallypreinstalled.com/PingPong/?transport=streaming&minsize=1000&maxsize=1000&symmetric=false&update=true&duration=1&go

in Chrome/Firefox this counts up 1,2,3... as expected
In Opera and IE10 there are duplicates every now and then.

@Artur-
Copy link
Contributor

Artur- commented Apr 30, 2013

http://artur.virtuallypreinstalled.com/PingPong-old/?transport=streaming&minsize=1000&maxsize=1000&symmetric=false&update=true&duration=1&go is identical to the test above except with an older js. In IE 10 (also in IE 9 mode) there is a huge difference as the newer one produces around 2 roundtrips per second with duplicates especially in IE10 mode whereas the old one produces > 40 roundtrips per second.

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

No branches or pull requests

3 participants