algolia / search-insights.js

Library for reporting click, conversion and view metrics using the Algolia Insights API

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Firefox warning on _ALGOLIA Cookie soon to be rejected

tkrugg opened this issue · comments

image

To reproduce:

  1. Open https://180jd.csb.app/ in firefox
  2. Open the console, the warning should appear.

I'm wondering if cookies are even the right approach here. Wouldn't localStorage or similar be enough? I don't think you can set a sameSite cookie from js: https://stackoverflow.com/a/58816189/3185307

Alternatively we should get rid of it altogether (planned to make it non-default in v2) and only support non-generated user token?

Checking this again, I no longer see that warning in my Firefox 94.0.2 (64-bit). Did something change?

@eunjae-lee, the sandbox uses search-insights@latest, so would be updated with v2 without cookie now

EDIT: same example with v1, not checked in ff https://codesandbox.io/s/jovial-shadow-imhqq

Oh you're right. I'm seeing it again 🤦🏻‍♂️