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

Invalid source URL for Google Analytics #47

Merged
merged 4 commits into from
Dec 15, 2014

Conversation

chris1312
Copy link
Contributor

google-analytics.com/ga.js should redirect to google-analytics.com/u/ga_debug.js when Google Analytics debugger extension is On for Chrome. In order to achieve this it is needed to use ssl or www prefix on google-analytics.com domain.

@juriansluiman
Copy link
Owner

The ssl. and www. subdomains were removed in #37 (here) because I didn't find a difference:

  1. HTTPS with ssl. and without ssl. works
  2. HTTP with www. and without www. works

But, you argue this breaks an extension on Chrome, probably because it explicitly checks for https://ssl. or http://www.? I see you updated the tests, thanks :) One minor note, can you rename $httpOrHttps to $scheme?

Invalid source URL for Google Analytics
@chris1312
Copy link
Contributor Author

Yes, exactly.
Thanks for the explanation :)

@juriansluiman
Copy link
Owner

I'm on mobile now (so can't merge) but I will merge them when I'm back on desktop. Thanks for the changes!

Lolz for the commit messages ;-) not sure what you did, but you could have just committed and pushed the changes. What happened?

juriansluiman pushed a commit that referenced this pull request Dec 15, 2014
Invalid source URL for Google Analytics
@juriansluiman juriansluiman merged commit 26b3477 into juriansluiman:master Dec 15, 2014
@juriansluiman
Copy link
Owner

v0.4.2 is released with this change.

@chris1312
Copy link
Contributor Author

Thanks!

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.

2 participants