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

fix: Correct interpretation of SameSite=strict #32824

Closed
wants to merge 3 commits into from
Closed

Conversation

shimingw
Copy link

Description

In the Set-Cookie documentation, the explanation for SameSite=Strict is inaccurate.

image
The specification for SameSite=Strict adds two restrictions base on Lax restriction,but rel=prerender had deprecated,Therefore, there is only one restriction: top-level navigation scene.

Document address:https://datatracker.ietf.org/doc/html/draft-ietf-httpbis-cookie-same-site-00#section-4.1.1

Motivation

Additional details

Related issues and pull requests

@shimingw shimingw requested a review from a team as a code owner March 25, 2024 09:13
@shimingw shimingw requested review from teoli2003 and removed request for a team March 25, 2024 09:13
@github-actions github-actions bot added Content:HTTP HTTP docs size/xs [PR only] 0-5 LoC changed labels Mar 25, 2024
Copy link
Contributor

github-actions bot commented Mar 25, 2024

Preview URLs

(comment last updated: 2024-07-12 13:42:36)

@Josh-Cena Josh-Cena requested review from wbamberg and bsmth and removed request for teoli2003 and wbamberg July 12, 2024 04:24
@bsmth
Copy link
Member

bsmth commented Jul 12, 2024

Thanks for the suggestion! I'm inclined to keep the original here as I think it's more accurate. Even though rel=prerender is deprecated, it's used as an example (of a category of attack*) that SameSite=Lax won't prevent, so this doesn't change the definition of SameSite=Strict, IMO. I'm going to close this one, but if you think I've made a mistake, please let me know! Thank you :)

@bsmth bsmth closed this Jul 12, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content:HTTP HTTP docs size/xs [PR only] 0-5 LoC changed
3 participants