Skip to content

Comments

Add HTML5 crossorigin attribute.#524

Open
nlewycky wants to merge 2 commits intopicturepan2:masterfrom
nlewycky:master
Open

Add HTML5 crossorigin attribute.#524
nlewycky wants to merge 2 commits intopicturepan2:masterfrom
nlewycky:master

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.

@luantak

This comment has been minimized.

Repository owner deleted a comment from luantak 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