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.

Aklapper removed Tpt as the assignee of this task.Jun 19 2020, 4:13 PM
Aklapper added a subscriber: Tpt.

This task has been assigned to the same task owner for more than two years. Resetting task assignee due to inactivity, to decrease task cookie-licking and to get a slightly more realistic overview of plans. Please feel free to assign this task to yourself again if you still realistically work or plan to work on this task - it would be welcome!

For tips how to manage individual work in Phabricator (noisy notifications, lists of task, etc.), see https://phabricator.wikimedia.org/T228575#6237124 for available options.
(For the records, two emails were sent to assignee addresses before resetting assignees. See T228575 for more info and for potential feedback. Thanks!)