Add cookie banner - Githubissues

dtekcth / dtek.se

A rewrite of the dtek portal (the one stolen from IT) in Django, started in 2018 by BS

2 stars 4 forks source link

Add cookie banner #9

Open dheurlin opened 5 years ago

dheurlin commented 5 years ago

The site uses cookies apparently, so we should either add an annoying cookie banner or try to nuke the cookies

viktorhallen commented 5 years ago

I vote nuke cookies. Those banners are the worst, and it's not like we use cookies for anything, right?

If we do use cookies for stuff, and instead switch to local storage, do we have to display a banner?

Rembane commented 5 years ago

I also vote nuke cookies to the extent it is possible. What cookies do we use and when?

Here is quite good information about the cookie law: https://webbriktlinjer.se/lagkrav/kaklagen-i-praktiken/ A relevant part is that we might not need to nuke all cookies. And that all ways to track users are covered by the law, so we cannot simply change to LocalStorage and say that we're compliant without getting consent from the users.

Kirens commented 5 years ago

I think the cookies mainly are used to select language, which would be fine. Although I'd also vote to try to nuke 'em if it's not to complicated. I mean we could use the path or sub-domains to select locale.

Or are cookies preferable over path to set the locale? Pros of cookies:

A Swedish user shares a site to an English speaker and both get their preferred language. Pros of path/domain:
Both languages are more easily crawlable.

Are there more things to consider?