Page MenuHomePhabricator

Implement module wrapping for Sentry
Closed, ResolvedPublic

Description

Add module wrapping code to Sentry per T513.

Currently ResourceLoader outputs something like

mw.loader.implement("modulename",function($,jQuery){<minified source code>});

This should be replaced with

mw.loader.implement("modulename",Raven.wrap(function($,jQuery){<minified source code>}));

(or the equivalent done inside mw.loader.implement) and Raven should be loaded early enough for this to work.

This might require adding a new hook or two to ResourceLoader.

Related Objects

Event Timeline

Tgr claimed this task.
Tgr raised the priority of this task from to Medium.
Tgr updated the task description. (Show Details)
Tgr added a project: Multimedia.
Tgr changed Security from none to None.
Tgr added subscribers: Tgr, He7d3r, Gilles and 6 others.
gerritbot subscribed.

Change 188330 had a related patch set uploaded (by Gergő Tisza):
Update Sentry to use mw.track

https://gerrit.wikimedia.org/r/188330

Patch-For-Review

Change 188330 merged by jenkins-bot:
Update Sentry to use mw.track

https://gerrit.wikimedia.org/r/188330