Page MenuHomePhabricator

Tool "corenlp" loads assets from google, cloudflare, bootstrapcdn
Open, Needs TriagePublic

Description

https://tools.wmflabs.org/corenlp/: https://code.jquery.com/jquery-2.1.4.min.js
https://tools.wmflabs.org/corenlp/: https://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/css/bootstrap.min.css
https://tools.wmflabs.org/corenlp/: https://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/css/bootstrap-theme.min.css
https://tools.wmflabs.org/corenlp/: https://maxcdn.bootstrapcdn.com/bootstrap/3.3.1/js/bootstrap.min.js
https://tools.wmflabs.org/corenlp/: https://cdnjs.cloudflare.com/ajax/libs/chosen/1.4.2/chosen.css
https://tools.wmflabs.org/corenlp/: https://cdnjs.cloudflare.com/ajax/libs/chosen/1.4.2/chosen.jquery.min.js
https://tools.wmflabs.org/corenlp/: https://storage.googleapis.com/corenlp/style-vis.css
https://tools.wmflabs.org/corenlp/: https://storage.googleapis.com/corenlp/js/brat/client/lib/head.load.min.js
https://tools.wmflabs.org/corenlp/: https://cdnjs.cloudflare.com/ajax/libs/d3/3.5.17/d3.min.js
https://tools.wmflabs.org/corenlp/: https://cdnjs.cloudflare.com/ajax/libs/dagre-d3/0.4.17/dagre-d3.min.js
https://tools.wmflabs.org/corenlp/: https://storage.googleapis.com/corenlp/loading.gif

Loading from external providers are discouraged. Please see parent tasks and J65.
In your case a direct replacement for some of the resources would be:

I'm not sure about style-vis.css and loading.gif. If they are free software you might want to host a local copy.

Event Timeline

Restricted Application added a subscriber: Aklapper. · View Herald TranscriptAug 3 2017, 2:39 PM
Liuxinyu970226 added a subscriber: Liuxinyu970226.EditedJul 13 2018, 1:00 PM

@Framawiki My personally suggestion is that we should also copy-paste those common java files (.class?) to our /cdnjs/ajax/libs/, because for e.g. Nokia S40 users, they can also have WYSIWYG-like search boxes.

@Framawiki My personally suggestion is that we should also copy-paste those common java files (.class?) to our /cdnjs/ajax/libs/,

Not gonna happen. Browsers don't load these Java code. Even if they were to be added it would need to be done in cdnjs upstream; we don't have our own cdnjs fork nor clone.