The Wayback Machine - https://web.archive.org/web/20220203190725/https://github.com/dotnet/aspnetcore/issues/39968
Skip to content
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

Add general property bag to cookies to support potential new cookie "standards: #39968

Open
blowdart opened this issue Feb 3, 2022 · 1 comment
Open

Comments

@blowdart
Copy link
Contributor

@blowdart blowdart commented Feb 3, 2022

Problem

Google in their quest to remove third party cookies are proposing moving to yet another cookie standard.

It's both better than, and worse than the same site changes we had to go through before to keep oauth's lights on for IdPs that couldn't support more modern flows.

The CHIPS proposal adds new attributes to cookies, and as we've traditionally had strong properties on cookies any new cookie property needs a lot of work to support, and we end up having a slow reaction.

Potential solution

I propose a general property bag of names and values (with values allowing for NULL) for outbound cookie properties, which would allow customers to be more flexible in what gets added as a cookie property and not have to wait for us to push new code to test out a standard which may, or may not get ratified.

Of course the property bag would have to nicely sync with the existing "strong" properties.

@Tratcher
Copy link
Member

@Tratcher Tratcher commented Feb 3, 2022

Design point: both key= and key can be valid formats depending on the parameter. We might not know how to properly format entries from a key-value collection. A List<string> might be better, or even a flat string for people to append extra stuff to.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
2 participants