-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[3.0 patch] Re-implement SameSite for 2019 #13870
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
Conversation
Yeah definitely should go with a less specific namespace if we are using it in 3 places now |
973708d
to
e243b2b
Compare
This branch has been updated to be in sync with the others. Outstanding items:
|
This comment was made automatically. If there is a problem contact [email protected]. I've triaged the above build. I've created/commented on the following issue(s) |
e243b2b
to
cd258cf
Compare
Cherry pick of #13858 but expanded to account for the 3.0 defaults change from Lax to None. This preserves the old behavior of None for most components using a default value by witching them to -1 (Unspecified). E.g.
new CookieOptions()
excludes samesite like before unless you explicitly set it.