Page MenuHomePhabricator

Please make Cortado path configurable
Closed, ResolvedPublic

Description

We need to move cortado.jar from upload to bits and in attempt to do so we noticed that the cortado.jar URL is hardcoded in the code and specifically in MwEmbedModules/EmbedPlayer/resources/mw.EmbedPlayerJava.js.

Please make the path configurable, so that we can switch it. For comparison, OggHandler has a $wgCortadoJarFile setting in CommonSettings.php.

Thanks,


Version: unspecified
Severity: normal

Details

Reference
bz40070

Event Timeline

bzimport raised the priority of this task from to Needs Triage.Nov 22 2014, 1:09 AM
bzimport set Reference to bz40070.

mdale wrote:

Cortado has to be on the same domain as the media to not trigger the java security warnings. ( i.e you can't really move it to bits .. unless your serving all media assets from bits )

But we can relatively easily add support for a $wgCortadoJarFile override if you want to change its wmf location. https://gerrit.wikimedia.org/r/24476

That used to be the case but it's not anymore. Java from v6 upd 10 supports a limited set of Flash's crossdomain.xml functionality. I've added a crossdomain.xml on upload.wm.org last week, did a quick version survey using logs & User-Agent strings and Tim deployed a switch of Cortado to bits this week for OggHandler.

Note that there's a slight exception to this: IcedTea does not support crossdomain.xml, even at the latest version. However, Cortado never worked with IcedTea for different reasons and as far me and Tim can see, so this isn't a concern.

mdale wrote:

24476 was merged closing bug.

<paravoid> this was solved but I'm not sure it's actually configured
<paravoid> I tried to check but with browser autodetection and everything I only get an HTML5 <video>

Jan was quick enough to answer me on IRC and immediately fix this in TMH:
https://gerrit.wikimedia.org/r/#/c/33173

Gilles triaged this task as Unbreak Now! priority.Dec 4 2014, 10:10 AM
Gilles moved this task from Untriaged to Done on the Multimedia board.
Gilles lowered the priority of this task from Unbreak Now! to Needs Triage.Dec 4 2014, 11:21 AM