Skip to content
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.

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

Add HTML5 crossorigin attribute. #524

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

nlewycky
Copy link

@nlewycky nlewycky commented Nov 9, 2018

In suggested code to copy and paste, load spectre.css from CDNs without sending user-credentials.
Verified

In suggested code to copy and paste, load spectre.css from CDNs without sending user-credentials.
Add HTML5 crossorigin attribute.
@nlewycky
Copy link
Author

nlewycky commented Jul 1, 2020

@lu4p I don't think that's what the page is saying. The page says:

By default (that is, when the attribute is not specified), CORS is not used at all.

The default value of the crossorigin attribute is anonymous, which makes <link crossorigin> equivalent to <link crossorigin="anonymous">. However, that's not the same as leaving out crossorigin entirely which is to not use CORS at all.

The rationale is that crossorigin is new and if it's present at all then your HTML was written in an age after CORS was created. Leaving it out has to have behaviour compatible with a pre-CORS world, so the browser will send along other cookies, etc.

I think I actually checked the HTTP headers in Chrome developer tools before sending out this PR, but I'm not longer sure. I can check again and get back to you on this PR.

@lu4p

This comment has been minimized.

Repository owner deleted a comment from lu4p Jul 1, 2020
@davestewart davestewart mentioned this pull request Sep 20, 2023
29 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants