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

Security Error from localStorage #44

Open
sammacbeth opened this issue Jun 12, 2017 · 2 comments
Open

Security Error from localStorage #44

sammacbeth opened this issue Jun 12, 2017 · 2 comments

Comments

@sammacbeth
Copy link
Collaborator

With analytics script running on https://sammacbeth.eu/, I get a security error on both Cliqz and Chrome when loading the page. No requests are made to the collector due to the exception.

screen shot 2017-06-12 at 22 38 51
screen shot 2017-06-12 at 22 39 11

As well as fixing the cause of this exception, we should also consider handling the case when localstorage is not available, and at least sending a page_load message in that case.

@konark-cliqz
Copy link
Contributor

Strange, I tested the same website https://sammacbeth.eu/ , the message seem to go through.
Do you have any settings related to localStorage and third parties ?
screen shot 2017-06-12 at 22 45 41

@sammacbeth
Copy link
Collaborator Author

This issue is caused by the browser's third-party cookie setting. Turning off third-party cookies also prevents used of localStorage in third-party context. This effectively breaks the green-analytics model for these users.

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

No branches or pull requests

2 participants