Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Optionally set crossOrigin = 'anonymous' #51

Merged
merged 1 commit into from Oct 11, 2013

Conversation

Projects
None yet
2 participants
Contributor

PhilHarnish commented Oct 11, 2013

crossOrigin = 'anonymous' will not send cookies with requests. This
can, for example, cause a login redirect which results in the cross
origin error the code tried to avoid in the first place.

Optionally set crossOrigin = 'anonymous'
crossOrigin = 'anonymous' will not send cookies with requests. This
can, for example, cause a login redirect which results in the cross
origin error the code tried to avoid in the first place.
Contributor

PhilHarnish commented Oct 11, 2013

Hey, I maintain Tinycon as it's used at Google and this fixes an issue which happens on our internal domains. I'd like to merge this diff upstream and update the Google code. Thanks!

Owner

tommoor commented Oct 11, 2013

Hey Phil, this makes a lot of sense - thanks!

tommoor added a commit that referenced this pull request Oct 11, 2013

Merge pull request #51 from PhilHarnish/optionalCrossOrigin
Optionally set crossOrigin = 'anonymous'

@tommoor tommoor merged commit 31d49da into tommoor:master Oct 11, 2013

@PhilHarnish PhilHarnish deleted the PhilHarnish:optionalCrossOrigin branch Nov 14, 2013

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