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

Remove Google Analytics or make this opt-in when self-hosted #243

Closed
mprasil opened this Issue Aug 14, 2018 · 7 comments

Comments

2 participants
@mprasil
Copy link

mprasil commented Aug 14, 2018

The Google Analytics can't be disabled anywhere in the interface as far as I'm aware and even if it was, this should be opt-in not opt-out otherwise it is very likely not GDPR compliant.

Obviously you can make this decision for your service, but I was wondering if there could be some support to set this in self-hosted scenario, where whoever is hosting the service might be required to disable it to be compliant.

Not sure if this is the right place. I'll post this on the community forum if it's better suited? This came up as discussion around disabling the analytics when bundled with our 3rd part API, so I didn't want to add unrelated stuff to community forums.

@kspearrin

This comment has been minimized.

Copy link
Collaborator

kspearrin commented Aug 14, 2018

Google Analytics already is disabled for self-hosted installations of Bitwarden.

@mprasil

This comment has been minimized.

Copy link
Author

mprasil commented Aug 14, 2018

Since when? I'm running 2.1.1 locally and I can see Privacy Badger blocking the requests.

@kspearrin

This comment has been minimized.

Copy link
Collaborator

kspearrin commented Aug 14, 2018

It always has, though I am guessing you are not using the official self hosted build.

@mprasil

This comment has been minimized.

Copy link
Author

mprasil commented Aug 14, 2018

Just for the record, here's how we build the Vault app:

npm run sub:init && npm install
npm run dist

And then we host the files from the build directory.

@kspearrin

This comment has been minimized.

@kspearrin

This comment has been minimized.

Copy link
Collaborator

kspearrin commented Aug 14, 2018

npm run dist:selfhost will remove GA.

@mprasil

This comment has been minimized.

Copy link
Author

mprasil commented Aug 14, 2018

Ah that's perfect! Thanks a lot for help.

@kspearrin kspearrin closed this Aug 14, 2018

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment