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

Cookie documentation #2241

Open
simon-friedberger opened this issue Oct 29, 2024 · 3 comments
Open

Cookie documentation #2241

simon-friedberger opened this issue Oct 29, 2024 · 3 comments
Labels
Volunteer Opportunity Tasks we're voluntelling you what would help us.

Comments

@simon-friedberger
Copy link
Contributor

The documentation claims that cookies cannot be set on public suffixes but that is simply not true.

Host cookies (__Host- or simply no domain=...) always work.

Domain cookies (with e.g. domain=example.com) automatically degrade to host cookies by having their domain string reset as specified in https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-rfc6265bis-15#section-5.7-3.9.1 and variations thereof.

@simon-friedberger simon-friedberger added the Volunteer Opportunity Tasks we're voluntelling you what would help us. label Oct 29, 2024
@simon-friedberger
Copy link
Contributor Author

@wdhdev @groundcat Does one of you maybe want to take this?

@wdhdev
Copy link
Contributor

wdhdev commented Nov 27, 2024

I'm not super aware of all the technical aspects around cookies, so I likely can't. I could give it a go though.

@groundcat
Copy link
Contributor

I have just opened publicsuffix/publicsuffix.org#46, but I am wondering if there are any other inaccuracies in other parts of the documentation. I did not find any explicit documentation stating that "cookies cannot be set on public suffixes" - though I might have missed it!

cc @wdhdev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Volunteer Opportunity Tasks we're voluntelling you what would help us.
Projects
None yet
Development

No branches or pull requests

3 participants