Last modified: 2013-08-06 16:58:45 UTC

Wikimedia Bugzilla is closed!

Wikimedia migrated from Bugzilla to Phabricator. Bug reports are handled in Wikimedia Phabricator.
This static website is read-only and for historical purposes. It is not possible to log in and except for displaying bug reports and their history, links might be broken. See T50402, the corresponding Phabricator task for complete and up-to-date bug report information.
Bug 48402 - rel=canonical of https pages should point to http
rel=canonical of https pages should point to http
Status: RESOLVED FIXED
Product: Wikimedia
Classification: Unclassified
SSL related (Other open bugs)
wmf-deployment
All All
: High normal (vote)
: ---
Assigned To: Nobody - You can work on this!
http://support.google.com/webmasters/...
:
Depends on:
Blocks: ssl
  Show dependency treegraph
 
Reported: 2013-05-13 13:13 UTC by Derk-Jan Hartman
Modified: 2013-08-06 16:58 UTC (History)
9 users (show)

See Also:
Web browser: ---
Mobile Platform: ---
Assignee Huggle Beta Tester: ---


Attachments
SSL cluster eqiad network_report (60.15 KB, image/png)
2013-08-06 16:58 UTC, Nemo
Details

Description Derk-Jan Hartman 2013-05-13 13:13:22 UTC
According to Google, HTTPS versions of our pages should have the rel=canonical head tag that points to the http version. See http://support.google.com/webmasters/bin/answer.py?hl=en&answer=139394

I can't remember that we ever had it like this, but if Google suddenly started enforcing this, then it might explain the http-https google split and google rank penalty that Wikid77 has been discussing on WP:VP/T.
Comment 1 Andre Klapper 2013-05-13 14:04:54 UTC
(In reply to comment #0)
> it might explain the http-https google split and google
> rank penalty that Wikid77 has been discussing on WP:VP/T.

Link: https://en.wikipedia.org/w/index.php?title=Wikipedia:Village_pump_%28technical%29&oldid=554883644#Relinking_Google_for_SSL_https
Comment 2 James Forrester 2013-07-09 14:23:08 UTC
I would strongly suggest WONTFIXing this bug in favour of 51002, which feels like a better solution given our long terms goals (see bug 47832).
Comment 3 Ryan Lane 2013-07-30 23:02:17 UTC
Until we're ready to switch to HTTPS for anons by default it makes sense to point HTTPS versions of pages to HTTP via rel=canonical. We should be setting one or the other, rather than having a split.

The load on the SSL cluster has been doubling month by month and this split may be the reason why. Unfortunately we don't have conclusive metrics to prove this, but making this setting explicit rules that out as a possibility.
Comment 4 James Forrester 2013-07-31 01:25:42 UTC
(In reply to comment #3)
> Until we're ready to switch to HTTPS for anons by default it makes sense to
> point HTTPS versions of pages to HTTP via rel=canonical. We should be setting
> one or the other, rather than having a split.
> 
> The load on the SSL cluster has been doubling month by month and this split
> may be the reason why. Unfortunately we don't have conclusive metrics to prove
> this, but making this setting explicit rules that out as a possibility.

So we should do this one now and bug 51002 next month/quarter/year/decade when we get around to it?
Comment 5 Ryan Lane 2013-07-31 02:24:58 UTC
Yep
Comment 6 MZMcBride 2013-07-31 18:35:28 UTC
https://gerrit.wikimedia.org/r/76856 has been submitted and merged. I imagine it'll be deployed shortly.
Comment 7 Nemo 2013-08-06 16:58:45 UTC
Created attachment 13066 [details]
SSL cluster eqiad network_report

Also for the records, from #wikimedia-operations:

18.15 < Ryan_Lane> looks like the rel=canonical change helped
18.15 < Ryan_Lane> https://ganglia.wikimedia.org/latest/graph.php?r=week&z=xlarge&c=SSL+cluster+eqiad&m=cpu_report&s=by+name&mc=2&g=network_report
18.15 < Ryan_Lane> I had a feeling that was causing the surge
18.16 < Ryan_Lane> I'm surprised how quickly it helped, though

Note You need to log in before you can comment on or make changes to this bug.


Navigation
Links