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

Changing SameSite cookie to be 'Lax' #2

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

binchick-in
Copy link
Owner

Reference stuff -> https://www.chromium.org/updates/same-site

This needs to be in place for development purposes.

@koskorya can you see if this change impacts any services you are running?

@binchick-in binchick-in requested a review from koskorya September 1, 2020 19:01
@binchick-in binchick-in self-assigned this Sep 1, 2020
@binchick-in
Copy link
Owner Author

Reference -> https://imgur.com/a7QQeNS

@koskorya
Copy link
Collaborator

koskorya commented Sep 1, 2020

We will need this to be SameSite=None can we add Secure instead?

@binchick-in
Copy link
Owner Author

We will need this to be SameSite=None can we add Secure instead?

From what I've been testing. Secure hasn't worked for me.

Have you gotten it to work?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants