IE throws SyntaxError for every test when Yeti is used on port 80 or 443 #46

Closed
reid opened this Issue May 22, 2013 · 1 comment

Projects

None yet

1 participant

@reid

The infamous "SyntaxError on line 30" that happens on IE 9 & 10 happens only when Yeti is used on port 80. This doesn't happen during testing, since Yeti is tested on port 9000 for real browsers and random ports in automation.

Stopping early on this error makes it difficult to resolve a few YUI issues:

The error is caused by code in SockJS that essentially does:

var ws = new WebSocket("ws://yeti.yuilibrary.com:/tower/930/rnd0m/websocket");

It seems to include the colon : when it's not needed. IE doesn't like that.

Error in F12 debugger context

@reid reid was assigned May 22, 2013
@reid reid added a commit to reid/yeti that referenced this issue May 23, 2013
@reid reid Omit port from SockJS URL if not available. GH-46.
The document.location.port may be the empty string on IE
when a default port is being used. In that case, avoid
putting a port, or colon, in the URL to prevent a SyntaxError
during WebSocket construction.
8453218
@reid

Fixed in Yeti 0.2.23.

@reid reid closed this May 23, 2013
@reid reid was unassigned by jlecomte Mar 13, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment