-
Notifications
You must be signed in to change notification settings - Fork 2.9k
Add cross-site ancestor flag to environment. #11133
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
base: main
Are you sure you want to change the base?
Conversation
This looks good to me, but talking to @cfredric we'd slightly prefer the term "ancestry" with "cross-site" or "same-site" as values. @bvandersloot-mozilla WDYT? |
I think that feedback makes a lot of sense- hanging it on the ESO is probably right. With respect to behavior when disconnected, I think the only corner case I worry about is if a disconnected document is able to perform fetches and then get reconnected later. In that case it should matter and we'd want some kind of latching to prevent disconnecting documents from being a way to escalate their privileges. If not, then this update should match your sketch for imlementation as a new algorithm on ESOs. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice, this looks good to me! Please also work on an update to the service worker spec, as the only other environment settings object on the platform.
I think the only corner case I worry about is if a disconnected document is able to perform fetches and then get reconnected later.
There's no way to reconnect disconnected documents, so I think we're safe!
This is commandeering #8036, and is meant to land with whatwg/fetch#1807 in its place.
(See WHATWG Working Mode: Changes for more details.)
/acknowledgements.html ( diff )
/nav-history-apis.html ( diff )
/webappapis.html ( diff )
/workers.html ( diff )
/worklets.html ( diff )