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

Remove Google Analytics tracking from entire site #119

Merged
merged 2 commits into from Mar 2, 2019

Conversation

Projects
None yet
2 participants
@jeremyfelt
Copy link
Member

jeremyfelt commented Mar 2, 2019

We aren't using this data for anything and it's enforcing the idea that everything in the world needs to be tracked by Google's analytic service. :)

I think I'm more comfortable knowing that 3rd party tracking is not used on varyingvagrantvagrants.org.

Nginx access logs are still stored, so we can always determine what popular pages are as necessary. I'll do some more thinking about rotating those out and anonymizing them.

jeremyfelt added some commits Mar 2, 2019

Remove Google Analytics configuration from entire site
We aren't using this data for anything and it's enforcing the idea
that everything in the world needs to be tracked by Google's
analytic service. :)
Remove Google Analytics from governance document
This account is no longer active (as of the time this is deployed)
and will be deleted.

@jeremyfelt jeremyfelt requested a review from tomjn Mar 2, 2019

@tomjn

tomjn approved these changes Mar 2, 2019

Copy link
Member

tomjn left a comment

Looks good to me, as an aside any chance of setting up a 404 handler in the nginx config?

@jeremyfelt

This comment has been minimized.

Copy link
Member Author

jeremyfelt commented Mar 2, 2019

any chance of setting up a 404 handler in the nginx config?

I don't think I ever realized 404s were showing up as 403s 🤦‍♂️

I'll take a look!

@jeremyfelt jeremyfelt merged commit 3a22a37 into master Mar 2, 2019

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@jeremyfelt jeremyfelt deleted the remove-analytics branch Mar 2, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.