Page MenuHomePhabricator
Create Task
Maniphest T105363

Catch errors thrown in site module (MediaWiki:Common.js, etc.)
Closed, DeclinedPublic
Actions

Description

This is related to T71445 and the like (e.g. T71519)

We've had a couple of instances where common.js has been throwing exceptions is certain configurations (e.g. using a deprecated IE method in IE11 which caused T97788). If common.js crashes most people don't notice because it usually only disables a few progressive enhancements or browser bug workarounds, but more critical features such as VisualEditor or MediaViewer can suffer from these fatal exceptions.

These errors could be rethrown as warnings or logged.

Related Objects

Event Timeline

Esanders created this task.Jul 9 2015, 4:55 PM
Esanders raised the priority of this task from to Needs Triage.
Esanders updated the task description. (Show Details)
Esanders added projects: WMF-General-or-Unknown, JavaScript, VisualEditor.
Esanders subscribed.
Restricted Application added a subscriber: Aklapper. · View Herald TranscriptJul 9 2015, 4:55 PM
Krenair subscribed.Jul 18 2015, 7:42 AM
Jdforrester-WMF added a project: MediaWiki-ResourceLoader.Jul 28 2015, 7:24 PM
Jdforrester-WMF set Security to None.
Jdforrester-WMF moved this task from To Triage to Freezer on the VisualEditor board.
Jdforrester-WMF added a subscriber: Krinkle.
Krinkle closed this task as Declined.EditedJul 29 2015, 2:06 AM
Krinkle claimed this task.

The site module has always been its own HTTP request (dedicated script tag) for legacy reasons. One side effect (that I intent to keep) is that its exceptions (syntax errors or runtime exceptions) don't affect any other call stacks. It's ignored.

The exceptions in Common.js aren't causing VE to fail. The exception is not interrupting VE's runtime at all.

What's happening is that ResourceLoader's state machine is not updating the site module's state from "loading" to "error", rather it stays pending indefinitely. If it reaches resolution (as either ready or error) then VE will automatically continue.

The module executor in ResourceLoader already catches all exceptions. There are no exceptions to that.

Closing in favour of T106736.

Legoktm renamed this task from Catch errors thrown in common.js to Catch errors thrown in site module (MediaWiki:Common.js, etc.).Jul 29 2015, 2:22 AM
• MZMcBride subscribed.Jul 29 2015, 2:23 AM
Content licensed under Creative Commons Attribution-ShareAlike (CC BY-SA) 4.0 unless otherwise noted; code licensed under GNU General Public License (GPL) 2.0 or later and other open source licenses. By using this site, you agree to the Terms of Use, Privacy Policy, and Code of Conduct. · Wikimedia Foundation · Privacy Policy · Code of Conduct · Terms of Use · Disclaimer · CC-BY-SA · GPL