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

0.7.7 Causes a "Stack Overflow" in IE8 when using xhr-polling #385

Closed
xtassin opened this issue Jul 16, 2011 · 7 comments
Closed

0.7.7 Causes a "Stack Overflow" in IE8 when using xhr-polling #385

xtassin opened this issue Jul 16, 2011 · 7 comments
Labels
bug Something isn't working

Comments

@xtassin
Copy link

xtassin commented Jul 16, 2011

Using a minimal test implementation (client and server), IE8 shows a popup "stack overflow in line 0", on receiving the GET response.

The problem appears to occur in a random fashion and more often when the frequency of requests increases.

The problem does not occur when using "flashsocket" transport.

Tested on WinXP, IE8

@3rd-Eden
Copy link
Contributor

It would be nice if you could share your minimal test implementation so we can reproduce it.

@xtassin
Copy link
Author

xtassin commented Jul 16, 2011

Server:

var io = require('socket.io').listen(8080);
io.sockets.on('connection', function (socket) {
    socket.on('update', function(sMsg) {
        socket.emit('test', 'bar');
    });
});

Client:

  <script src="http://localhost:8080/socket.io/socket.io.js"></script>
  <script>
        var socket = io.connect('http://localhost:8080');
        setInterval(function() {
            socket.emit('update', 'foo');
        }, 500);
  </script>

See? That was minimal!

@rauchg
Copy link
Contributor

rauchg commented Aug 2, 2011

just came across this.

@denisu
Copy link

denisu commented Aug 10, 2011

About 1 out of 20 IE 8.0 users get this error on my site, also with 0.7.8.

Relevant: http://stackoverflow.com/questions/226102/stack-overflow-in-line-0-on-internet-explorer

@3rd-Eden
Copy link
Contributor

Do you have a reproducible test case?

@denisu
Copy link

denisu commented Aug 10, 2011

No, sorry. I can not reproduce it on my machine, but its permanent at some of my users. Is there a simple way to debug them remotely?

@denisu
Copy link

denisu commented Aug 11, 2011

I was now able to reproduce it, just disable Flash :).

firejune fixed it, and it works: #458

darrachequesne pushed a commit that referenced this issue Jul 4, 2024
darrachequesne pushed a commit that referenced this issue Jul 4, 2024
darrachequesne pushed a commit that referenced this issue Jul 4, 2024
Fix XHR.check, which was throwing an error and causing non-IE browsers to fall back to JSONP
darrachequesne pushed a commit that referenced this issue Jul 8, 2024
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

4 participants