-
Notifications
You must be signed in to change notification settings - Fork 35
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
Should the API be (at least initially) read-only? #13
Comments
I like this plan. It seems better to start out small. Also, if you want client-side state, there's a ton of storage APIs already. |
I may still revisit this proposal, but for now I've gone ahead and proposed a full read/write/monitor API; see the description in the explainer for details and please file issues as you find them (pull requests happily accepted too!) Rationale: it's harder to make usability improvements to the cookies API (for instance, around default path, predictable character encoding, write-time enforcement of Any and all feedback and opinions are certainly welcome! There's also an open pull request to make further changes to the explainer and I also intend to start another soon to bring the polyfill in line with the current proposal. |
Read/write is explained and polyfilled now, many of the problems that lead me to open this issue are instead addressed head-on by proposing to support only a sensible subset of cookie behavior and using saner-than- |
If we don't allow writes we side-step a bunch of the issues around not being able to use the results from the read API in the write API due to cookie name per se being an insufficient key
The text was updated successfully, but these errors were encountered: