-
-
Notifications
You must be signed in to change notification settings - Fork 754
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
Comments
Have you try 1.0.13-SNAPSHOT by any chance? |
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 |
Heavily tested using [this][(https://gist.github.com/jfarcand/5374686). Let me know how it goes. |
Awesome work. With the fixed version I no longer see any issues in my test. |
Still seeing problems with this. It seems like the connection is always reconnected after calling push from the client. |
@Legoth: Share a test case, as I can't reproduce this issue. Are you using streaming or ling-polling? |
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. |
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. |
@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. |
Also, can you share your test case? I've tested with the gist above (simulate cache etc.) and it worked.... |
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 |
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. |
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.
The text was updated successfully, but these errors were encountered: