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

Add the cookie consent banner as required by the EU #327

Closed
wants to merge 2 commits into from

Conversation

jamo
Copy link
Member

@jamo jamo commented Aug 12, 2015

WIP - RFC
eu

@jamo
Copy link
Member Author

jamo commented Aug 12, 2015

@mpartel Please review -- I chose the http://cookiepedia.co.uk/ for the more info as it seems to be nicely treating the cookie consent thingy.

Also about the message - now it is pretty much the least amount of text as possible - should we maybe elaborate on this.

@mpartel
Copy link
Member

mpartel commented Aug 12, 2015

I think this is not necessary. http://ec.europa.eu/ipg/basics/legal/cookies/index_en.htm says e.g. "authentication cookies, to identify the user once he has logged in, for the duration of a session" are exempt from the requirement.

It will become necessary if we ever add analytics cookies or such.

@mpartel
Copy link
Member

mpartel commented Aug 12, 2015

I think the above is only sure* to apply when the login cookie is non-persistent, which ours is.

[*] this stuff is infuriatingly vague

@jamo jamo closed this Aug 25, 2015
@jamo
Copy link
Member Author

jamo commented Aug 25, 2015

Closed as this turns out to be irrelevant. But kept safe should we ever add more tracking.

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