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

Enable analytics piwik on http #2172

Closed
sebglon opened this issue Oct 12, 2016 · 10 comments
Closed

Enable analytics piwik on http #2172

sebglon opened this issue Oct 12, 2016 · 10 comments
Assignees
Labels
Bug Bugs with Cachet Hacktoberfest
Milestone

Comments

@sebglon
Copy link

sebglon commented Oct 12, 2016

If cachet site is on http; is it possible to use piwik call on http?

For exemple you can change your code by:

<script type="text/javascript">
  var _paq = _paq || [];
  _paq.push(['trackPageView']);
  _paq.push(['enableLinkTracking']);
  (function() {
    var u= window.location.protocol+"://default.piwik.t-statuspage.vs.cloud.socrate.vsct.fr";
    _paq.push(['setTrackerUrl', u+'/piwik.php']);
    _paq.push(['setSiteId', 1]);
    var d=document, g=d.createElement('script'), s=d.getElementsByTagName('script')[0];
    g.type='text/javascript'; g.async=true; g.defer=true; g.src=u+'/piwik.js'; s.parentNode.insertBefore(g,s);
  })();
</script>
@GrahamCampbell
Copy link
Contributor

No reason to use http to them when we can use https.

@jbrooksuk jbrooksuk reopened this Oct 12, 2016
@jbrooksuk jbrooksuk added the Bug Bugs with Cachet label Oct 12, 2016
@jbrooksuk jbrooksuk added this to the V2.4.0 milestone Oct 12, 2016
@jbrooksuk jbrooksuk self-assigned this Oct 12, 2016
@sebglon
Copy link
Author

sebglon commented Oct 12, 2016

My piwik is not on https.

@ConnorVG
Copy link
Contributor

@GrahamCampbell but it can be self hosted - there is definitely a reason to support http.

@sebglon
Copy link
Author

sebglon commented Oct 12, 2016

use piwik syntax     var u= "//default.piwik.t-statuspage.vs.cloud.socrate.vsct.fr";

without schema; dona by piwik script generaor

@jbrooksuk
Copy link
Member

I'll update this don't worry 👍

@jbrooksuk
Copy link
Member

You'll have to supply the full URL to your Piwik instance.

@jbrooksuk jbrooksuk reopened this Oct 12, 2016
@iaidan
Copy link
Contributor

iaidan commented Oct 18, 2016

Is this being worked on?

@jbrooksuk
Copy link
Member

@iaidan we've not started it as of yet, if you'd like to open a PR, please do.

@iaidan
Copy link
Contributor

iaidan commented Oct 19, 2016

I updated the view to rely on the HTTP/HTTPS being part of the URL provided in settings. May be a good idea to add a little note mentioning that on the analytics settings page.

Could also add a migration to update the old values to include https:// at the front so existing installations using Piwik will still work without updating the setting.

@jbrooksuk
Copy link
Member

Closed by #2197

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Bugs with Cachet Hacktoberfest
Projects
None yet
Development

No branches or pull requests

5 participants