-
Notifications
You must be signed in to change notification settings - Fork 1.6k
Update API for SameSitemode #3569
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
Update API for SameSitemode #3569
Conversation
On framework systems, the behavior of SameSiteMode.None has changed. This PR documents the new behavior, and links to the KB articles.
Co-Authored-By: Chris Ross <[email protected]>
@StephenMolloy I update per your comments. The new text should be reviewed as well. |
@StephenMolloy I'm not sure what you mean by this:
|
closing and reopening for a fresh build. |
Co-Authored-By: Scott Addie <[email protected]>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if Lax and None should be in backticks as suggested. Also perhaps SameSite = Lax
.
Co-Authored-By: Genevieve Warren <[email protected]>
Co-Authored-By: Chris Ross <[email protected]>
On framework systems, the behavior of SameSiteMode.None has changed.
This PR documents the new behavior, and links to the KB articles.