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

Improvements to cookie value validation #508

Merged
merged 3 commits into from Jul 29, 2020
Merged

Conversation

LukeTowers
Copy link
Contributor

No description provided.

@LukeTowers
Copy link
Contributor Author

I tested it and it invalidates all existing cookies except for the october_session, which then gets upgraded in the response to the request. This will ensure that users will remain logged in after they complete the update, although they will lose their persistence cookie if they have one and have to login again at some point.

@LukeTowers LukeTowers merged commit 28310d4 into develop Jul 29, 2020
@LukeTowers LukeTowers deleted the wip/fix-laravelcookies branch July 29, 2020 03:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
3 participants