-
Notifications
You must be signed in to change notification settings - Fork 7.9k
Implement proof-of-concept partitioned option for setcookie #12652
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
base: master
Are you sure you want to change the base?
Conversation
Partitioned cookies are marked as experimental by Mozilla (https://developer.mozilla.org/en-US/docs/Web/Privacy/Partitioned_cookies). Theoretically this means they are subject to change. It seems they're mostly pushed by Google. I'm also unsure about the premise of the original issue:
I haven't tested this, but I think what this means is that Maybe this warrants a short e-mail to the ML? I'm not a specialist in regards to cookies. Maybe other people have something to add. |
@nielsdos I am attempting to implement CHIPS for my app with an ugly workaround manually setting the cookie header so this PR would be very helpful but I think it should also include changes to the session cookie to truly allow developers to fix the problem. session_set_cookie_params(['partitioned' => true]) My use case is the most basic wherin my app (tool) is displayed in an iframe within a learning management system (LMS) and requires a session cookie for further interaction after the inital LTI launch. |
AFAIK, Mozilla and Safari have experimented with similar ideas, but right now only Chrome supports the
That's it indeed! Chrome just started an experiment, now active for 1% of users, in which it will block third-party cookies that lack the If this experiment results in a launch down the road, I think plenty of developers will want to use |
Yeah, I fear that the scope of this must be increased, and ini settings are probably necessary too. |
I think we should probably have RFC for this as the agreement is not clear (Derick raised some concerns on internals) so I don't think we can just merge this. |
No description provided.