LearnBoost/socket.io — pull request #1471 by liorsion (2014-03-28 12:20:06+00:00)
Title: support cors

socket.io supports cors internally today. The problem with this approach is that sometimes socket.io is used with platforms such as express (ie, serving other content) - this content often has it's own way of supporting CORS

For example, a popular setup would be to have the nginx or apache in front of the node.js handle the cors (saving the need to upgrade and offloading the prefetch requests).

In those cases, the socket.io part returns double the CORS headers, and this is illegal in CORS - it fails in Chrome for example.

This change request enables optionally to turn off cors support. By default it's still on