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

secure cookie error crash #2685

Merged
merged 30 commits into from
Nov 1, 2018
Merged

secure cookie error crash #2685

merged 30 commits into from
Nov 1, 2018

Conversation

brian-mann
Copy link
Member

@brian-mann brian-mann commented Oct 31, 2018

- for special use in vscode, although useful outside it too
- this fixes the edge case where a secure cookie flag is set and then
that cookie is then expired from an https URL *without* the subsequent
secure flag
- due to the way we were constructing the URL, this would fail to
actually remove the cookie
@brian-mann brian-mann merged commit 2333d04 into develop Nov 1, 2018
kuceb added a commit that referenced this pull request Nov 13, 2018
kuceb pushed a commit that referenced this pull request Nov 14, 2018
- fixes #1264
- fixes #1321
- fixes #1799
- fixes #2689
- fixes #2688
- fixes #2687
- fixes #2686
@kuceb kuceb mentioned this pull request Nov 19, 2018
kuceb added a commit to kuceb/cypress that referenced this pull request Apr 23, 2019
kuceb pushed a commit to kuceb/cypress that referenced this pull request Apr 23, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment