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

Cookiesync #21

Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Cookiesync #21

wants to merge 38 commits into from

Conversation

mattkrau
Copy link
Collaborator

Additional Check for Cookie Synchronisation

mattkrau and others added 30 commits May 21, 2019 11:21
gtm/js loads the configuration for Google Tag Manager. It is not a
tracking request itself. You can actually include gtm/js without
having Google Analytics.

Moreover, we checked whether gtm/js has a aip=1 parameter set, which is
not a valid parameter for gtm/js.
There is no utm.gif file in Google Analytics but only a __utm.gif.
To be able to fetch the resource content the content must actually be
rendered in a frame. However, some pages, e.g. redirects, do not render
in a frame and have no content.
We run some JavaScript snippet inside a call frame to extract the
arguments of certain JavaScript function calls. However, our JavaScript
snippet may throw errors in a few cases. We should fix the snippet, but
for now we just handle those errors properly and go without arguments.
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

Successfully merging this pull request may close these issues.

None yet

2 participants