safe-react icon indicating copy to clipboard operation
safe-react copied to clipboard

Cookie consent: store in localStorage, not in cookies

Open iamacook opened this issue 2 years ago • 1 comments

Description

Cookies are sometimes prefixed with our legacy immortal keys. In line with migrating out localStorage keys, we should remove the prefix from our cookies.

Requirements

  • Remove 'v1_MAINNET__' prefix from any new cookies being set
  • Migrate current cookies

alternatively

Move the only single cookie we have (cookie banner consent) to localStorage.

iamacook avatar Mar 03 '22 12:03 iamacook

Move the only single cookie we have (cookie banner consent) to localStorage.

I like this idea. The only thing to keep in mind is that we use a 7-day expiration policy if the user didn't accept GA. I.e. we show them the banner again in a week in this case.

katspaugh avatar Mar 31 '22 13:03 katspaugh