Fixes #458 #385 #276

Merged
merged 1 commit into from Aug 12, 2011

Conversation

Projects
None yet
3 participants
@3rd-Eden
Contributor

3rd-Eden commented Aug 11, 2011

They are reported in learnboost/socket.io

This causes a StackOverflow in IE8

@wayx

This comment has been minimized.

Show comment Hide comment
@wayx

wayx Aug 12, 2011

it did not help, still get stack overflow in IE8

wayx commented Aug 12, 2011

it did not help, still get stack overflow in IE8

@3rd-Eden

This comment has been minimized.

Show comment Hide comment
@3rd-Eden

3rd-Eden Aug 12, 2011

Contributor

@webstalk3r

Thats why I said a instead of the as there are multiple spots where this might ocurre

Contributor

3rd-Eden commented Aug 12, 2011

@webstalk3r

Thats why I said a instead of the as there are multiple spots where this might ocurre

@wayx

This comment has been minimized.

Show comment Hide comment
@wayx

wayx Aug 12, 2011

Ok, any advices?
Disable XHR/IE8 connection for a while?

wayx commented Aug 12, 2011

Ok, any advices?
Disable XHR/IE8 connection for a while?

rauchg added a commit that referenced this pull request Aug 12, 2011

@rauchg rauchg merged commit 1771590 into socketio:master Aug 12, 2011

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