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

Changed empty tracker response message #10395

Closed
wants to merge 1 commit into from
Closed

Changed empty tracker response message #10395

wants to merge 1 commit into from

Conversation

da2x
Copy link
Contributor

@da2x da2x commented Aug 13, 2016

Google don’t like the link scheme currently deployed by Piwik. It’s considered quite spammy. It doesn’t help that there are two links, one of which is broken.

This patch adds a slightly more meaningful error response message, and adds HTTPS and a nofollow attribute to the link.

* Indicate some context for the message
* Added rel=`nofollow` to stay on the good side of Google
@@ -78,7 +78,7 @@ public function outputResponse(Tracker $tracker)
Common::sendResponseCode(400);
}
Common::printDebug("Empty request => Piwik page");
echo "<a href='/'>Piwik</a> is a free/libre web <a href='http://piwik.org'>analytics</a> that lets you keep control of your data.";
echo 'This resource is part of <a href="https://piwik.org/" rel="nofollow">Piwik</a>, a free/libre web analytics tool that lets you keep control of your data.';
Copy link
Member

@tsteur tsteur Aug 31, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think @mattab will want to have the link without the rel="nofollow". Otherwise looks good to me. 👍

Copy link
Contributor Author

@da2x da2x Sep 1, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

See the link provided in the initial commit message. Google doesn’t want links like this without the rel=nofollow attribute. Violates Google’s link scheme policy: “low-quality links embedded in widgets that are distributed across various sites”

@tsteur tsteur added the Needs Review PRs that need a code review label Aug 31, 2016
@da2x
Copy link
Contributor Author

da2x commented Sep 8, 2016

Google just published A reminder about widget links. So yeah, nofollow or no link.

@tsteur
Copy link
Member

tsteur commented Sep 13, 2016

I'd be ok with that (nofollow). ping @mattab

@mattab
Copy link
Member

mattab commented Sep 23, 2016

@Aeyoun could you remove the first link and leave the second one to piwik.org? Clearly the link is nowhere spammy as piwik.org is high quality website. It is actually helpful to show this message and a link in the tracking API endpoint as it is helpful to people wondering what this endpoint is. google would not consider our link in their definition "of keyword-rich, hidden or low-quality links embedded in widgets that are distributed across various sites." <- clearly that's not us 👍

@da2x
Copy link
Contributor Author

da2x commented Sep 23, 2016

@mattab, no but you’re missing the intention of Google’s policy there. Automatically distributed links like this one should use nofollow. It wasn’t intentionally put there by the site owner. This link will in no way help market Piwik or boost it in search listings (very low value page with the same string repeated over and over on thousands of websites). However, it could potentially hurt Piwik’s link reputation for not complying with Google’s link scheme policy.

@mattab
Copy link
Member

mattab commented Sep 23, 2016

I definitely understand your point and also why Gogle does this. I feel like Google will never penalise us for linking in our API endpoint to the official website of the API/ Product, as they clearly refer to other practises in their public communication (spammy links, within the content of the website, which is not the case for this link as it never shows up in the content itself).

Either way I think we could put the nofollow just to play it safe, but I still think it's un-necessary.

@mattab mattab added this to the 3.0.0-b2 milestone Sep 23, 2016
@mattab mattab closed this in 08a3ec0 Oct 30, 2016
@mattab
Copy link
Member

mattab commented Oct 31, 2016

Fixed in 3.0.0-beta2: fixed text, making link SSL and fixing tests. (didn't add nofollow as linking to piwik.org is not seens as spammy by google)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Needs Review PRs that need a code review
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants