-
Notifications
You must be signed in to change notification settings - Fork 3
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
Collaborate with existing proposal? #14
Comments
Thanks, @domenic ! I'm looking at @patrickkettner 's specification and I like what I see - it seems like this specification exposes a lot more of the "why" of cookie changes than I proposed exposing my explainer, and that may be a good thing. Although this "why" information is not reliably available to web applications today, exposing it might be a useful thing. |
I just raised #32 to address cookie change access from Service Workers specifically. |
Hey fellers, At a core level, I think the goals of the two are pretty far apart. This spec is only intended to be a thin layer of sugar over the cookie (no pun intended) and a single change event. Whereas @bsittler's is more of a "what if we designed document.cookie from scratch". Both have a huge amount of value, and I think they can and should both coexist. We have seen at a number of sites, both large and small, the pattern of polling a cookie over and over again in order to ensure multi-tab/window users don't get into a bizarre state. This, along with very thin wrappers like js-cookie make up most of the interactions that I see folks make with From my point of view, the benefit that is gained from async-cookie is the fact you have access to cookies from outside of the I think that both specs have their own strengths, and implementing one will not take away from the value of the other. cool? |
I wanted to connect you and @bsittler who is working on https://github.com/WICG/async-cookies-api/blob/gh-pages/explainer.md . In particular his proposal includes an observer API at https://github.com/WICG/async-cookies-api/blob/gh-pages/explainer.md#monitoring .
The text was updated successfully, but these errors were encountered: