Last modified: 2012-12-13 11:21:39 UTC
There were two place you're calling Http::get(), it would be nice if they could both be forced to use https. It looks like you're using the data pulled back to populate objects, which is fine for the objects you're currently using, but if you start building objects that could effect access control, then you want to make sure you have the correct data.
Partial fix in Idf0a6547, but that does not address the issues raised about HTTP requests from the MediaWikiSite class. If I remember the discussion correctly, we agreed that any protocol-relative base URL should default to using HTTPs - at least for Wikimedia sites. That needs additional work.
With the patch in https://gerrit.wikimedia.org/r/#/c/25509/ and doing a fresh install of MW + Wikibase repo (running maintenance/update.php), I get the following error: Warning: Invalid argument supplied for foreach() in /Library/WebServer/Documents/wikidata-repo/extensions/Wikibase/lib/includes/Utils.php on line 88 When I change the "https" back to "http", then it works. There must be some other solution for this to work correctly.
Verified in Wikidata demo time for sprint 18
