-
Notifications
You must be signed in to change notification settings - Fork 23
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
Cookie Store API #36
Comments
There is also a TAG review request here: w3ctag/design-reviews#290 |
This comment by Ehsan on the Mozilla issue still seems applicable and relevant:
Based on that I'm adding |
Marking this as |
Is https://bugs.webkit.org/show_bug.cgi?id=258504 an indication that WebKit's position has changed? |
I don’t know if we’ve had a chance to communicate our latest analysis but it’s important for us that no cookie metadata is exposed through this API. The old document.cookie API provides key+value and that’s the only thing that should be exposed. User agents should be at liberty to honor, change, or not honor cookie directives, and webpages should not be able to check which decision the user agent made. This is important to be able to protect the user and enforce a cookie policy without websites pushing users to change settings. |
Yeah, couple of things to add to what John said above:
It would be interesting to know if @inexorabletash, @ayuishii, et al. could see eventual convergence on this set of ideas so we might all end up with the same API. |
Modulo compatibility with deployed content, converging on the above seems reasonable. Usage of the API per https://chromestatus.com/metrics/feature/timeline/popularity/2510 is higher than I would have expected, so research (as you're doing) would be necessary to ensure a subset of what Chrome currently ships is feasible. |
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
This comment was marked as resolved.
https://bugs.webkit.org/show_bug.cgi?id=278198 rdar://133994044 Reviewed by NOBODY (OOPS!). The document.cookie API exposes only the `name` and `value` attributes of the cookie. Our position is that we want to do the same for the new Cookie Store API. Since we do not want to expose any extra cookie metadata through this API, this patch removes those extra attributes and their feature flag. Discussion about WebKit's standards position on this: (WebKit/standards-positions#36) * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieListItem_attributes.https.any-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieListItem_attributes.https.any.serviceworker-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieStore_set_arguments.https.any-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieStore_set_arguments.https.any.serviceworker-expected.txt: * Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml: * Source/WebCore/Modules/cookie-store/CookieListItem.h: (WebCore::CookieListItem::CookieListItem): (): Deleted. * Source/WebCore/Modules/cookie-store/CookieListItem.idl:
https://bugs.webkit.org/show_bug.cgi?id=278198 rdar://133994044 Reviewed by NOBODY (OOPS!). The document.cookie API exposes only the `name` and `value` attributes of the cookie. Our position is that we want to do the same for the new Cookie Store API. Since we do not want to expose any extra cookie metadata through this API, this patch removes those extra attributes and their feature flag. See WebKit/standards-positions#36 * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieListItem_attributes.https.any-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieListItem_attributes.https.any.serviceworker-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieStore_set_arguments.https.any-expected.txt: * LayoutTests/imported/w3c/web-platform-tests/cookie-store/cookieStore_set_arguments.https.any.serviceworker-expected.txt: * Source/WTF/Scripts/Preferences/UnifiedWebPreferences.yaml: * Source/WebCore/Modules/cookie-store/CookieListItem.h: (WebCore::CookieListItem::CookieListItem): (): Deleted. * Source/WebCore/Modules/cookie-store/CookieListItem.idl:
Request for position on an emerging web specification
Information about the spec
Design reviews and vendor positions
The text was updated successfully, but these errors were encountered: