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

Adds `setCookieSameSite` security option. #13

Merged
merged 4 commits into from Apr 20, 2016

Conversation

Projects
None yet
2 participants
@bobjflong
Contributor

bobjflong commented Apr 16, 2016

SameSite is an interesting cookie option that asserts that a cookie
should not be included in cross-origin requests. This commit adds a sum
type SameSiteOption and a function setCookieSameSite for interacting
with this option.

Here is the draft: https://tools.ietf.org/html/draft-west-first-party-cookies-07

Adds `setCookieSameSite` security option.
SameSite is an interesting cookie option that asserts that a cookie
should not be included in cross-origin requests. This commit adds a sum
type `SameSiteOption` and a function `setCookieSameSite` for interacting
with this option.

Here is the draft: https://tools.ietf.org/html/draft-west-first-party-cookies-07
Show outdated Hide outdated Web/Cookie.hs Outdated
Show outdated Hide outdated Web/Cookie.hs Outdated
Show outdated Hide outdated Web/Cookie.hs Outdated

@snoyberg snoyberg merged commit daf2c0d into snoyberg:master Apr 20, 2016

@snoyberg

This comment has been minimized.

Show comment
Hide comment
@snoyberg

snoyberg Apr 20, 2016

Owner

Awesome, merged. I added a small fix to the new NFData instance. Just confirming with Travis and then I'll release to Hackage.

Owner

snoyberg commented Apr 20, 2016

Awesome, merged. I added a small fix to the new NFData instance. Just confirming with Travis and then I'll release to Hackage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment