update google-analytics and add ipv6 localhost address

[ettoreleandrotognoli]

resolves #2215

[joshka]

This looks reasonable based on the installation instructions at https://developers.google.com/tag-platform/tag-manager/datalayer#installation

It would be worth showing you've tested this with a couple of screenshots.

[ettoreleandrotognoli]

The site that I'm using it is this one:
https://ettoreleandrotognoli.gitlab.io/unimar-object-oriented-programming-i/

This print any one can reproduce inspecting the site:

This one is from the google analytics site, it needs authentication

I'm using the mdBook version from my personal repo to build it
https://gitlab.com/ettoreleandrotognoli/unimar-object-oriented-programming-i/-/blob/main/Dockerfile?ref_type=heads#L9

[joshka]

LGTM

[ehuss]

I'm reluctant to merge this since the google analytics option is deprecated and is intended to be removed (there is also a large warning telling you not to use it). I also don't particularly want to research to see what the possible consequences of changing this are. I have a vague understanding that Google has changed from one system to another, but I don't know if that migration would break existing configurations, or what the status is of that migration.

I'm going to suggest that instead of fixing the code here that you use a custom theme/head.hbs file instead.

[KFearsoff]

I'm going to suggest that instead of fixing the code here that you use a custom theme/head.hbs file instead.

I think it's a good idea, but it would be nice to include the code snippet into documentation (into the separate "user-contributed" section in mdbook's book, perhaps?). This is a good contribution and I think it shouldn't go to waste even if we pull the plug on supporting Google Analytics. After that we can consider closing the PR?