Skip to content

Fix for COR #60

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.

Sign up for GitHub

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

Merged
4 commits merged into from
Oct 1, 2010
Merged

Fix for COR #60

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
c06de09
add package.json for npm
maccman Sep 23, 2010
fcba035
Fix COR headers/requests for different ports on Safari.
maccman Sep 27, 2010
607fe38
Fix COR headers/requests for different ports on Safari.
maccman Sep 27, 2010
4803d7b
Merge branch 'master' of github.com:maccman/Socket.IO-node
maccman Sep 27, 2010
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix COR headers/requests for different ports on Safari.
  • Loading branch information
@maccman
maccman committed Sep 27, 2010
commit fcba035330023802cbb391fc013c16338b0d3d16
2 changes: 1 addition & 1 deletion lib/socket.io/transports/xhr-multipart.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ Multipart.prototype._onConnect = function(req, res){
var self = this, body = '', headers = {};
// https://developer.mozilla.org/En/HTTP_Access_Control
if (req.headers.origin && this._verifyOrigin(req.headers.origin)){
headers['Access-Control-Allow-Origin'] = req.headers.origin;
headers['Access-Control-Allow-Origin'] = '*';
headers['Access-Control-Allow-Credentials'] = 'true';
}
if (typeof req.headers['access-control-request-method'] !== 'undefined'){
Expand Down