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

docs: add 'none' as a possible value for sameSite #118

Merged
merged 1 commit into from Apr 19, 2020

Conversation

clauderic
Copy link
Contributor

@clauderic clauderic commented Nov 4, 2019

This PR follows up on #111 and updates the documentation to reflect the fact that 'none' was added as an option that can be passed to sameSite

@clauderic
Copy link
Contributor Author

@dougwilson any chance you can merge this PR?

@dougwilson
Copy link
Contributor

Appologies, for a while I was not getting all GitHub notifications and I must have missed this one. The change looks good, I will merge when I get home tonight 👍

@clauderic
Copy link
Contributor Author

🏓 @dougwilson

1 similar comment
@clauderic
Copy link
Contributor Author

🏓 @dougwilson

@dougwilson
Copy link
Contributor

This was on hold due to release process, sorry. It was unknown if the next release would be path (land on master) or minor (land on branch). I am putting together a few things today, so will get this landed on master and the duplicates closed.

@clauderic
Copy link
Contributor Author

Thank you! ❤️

@dougwilson dougwilson merged commit d211162 into pillarjs:master Apr 19, 2020
@clauderic clauderic deleted the patch-1 branch April 18, 2021 21:28
@pillarjs pillarjs deleted a comment from hermanskurichin Apr 18, 2021
@pillarjs pillarjs locked as resolved and limited conversation to collaborators Apr 18, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants