Add top-level site to storage key. (#127)

whatwg / storage

Storage Standard

https://storage.spec.whatwg.org/

Other

126 stars 55 forks source link

Add top-level site to storage key. (#127) #144

Open wanderview opened 2 years ago

wanderview commented 2 years ago

[ ] At least two implementers are interested (and none opposed):
- …
- …
[ ] Tests are written and can be reviewed and commented upon at:
- …
[ ] Implementation bugs are filed:
- Chrome: …
- Firefox: …
- Safari: …

(See WHATWG Working Mode: Changes for more details.)

Preview | Diff

wanderview commented 2 years ago

Its unclear to me if this PR is something we are comfortable merging into the spec now or not. Do we have adequate consensus on using top-level site for partitioning? My sense is we have at least two engines between chromium and gecko, but there are still some differences to webkit (ephemeral 3rd party storage).

annevk commented 2 years ago

I think that only affects how long the storage is kept, not how it's keyed, right?

It seems to me that all browsers want to align on this keying.

cc @johnwilander @cdumez