Remove raven.js from legacy + devhub pages.
Unless I'm mistaken there isn't anyone to really look at this and most errors and problems should be related to legacy pages anyway. Let's remove things we're not using or looking at. Fixes #4845
This commit is contained in:
Christopher Grebs
Родитель
fcfa453264
Коммит
69aba0e9ef
|
|
@ -32,7 +32,6 @@ jquery.cookie/jquery.cookie.js \
|
|||
@claviska/jquery-minicolors/jquery.minicolors.js \
|
||||
jquery-pjax/jquery.pjax.js \
|
||||
jszip/dist/jszip.js \
|
||||
raven-js/dist/raven.js \
|
||||
timeago/jquery.timeago.js \
|
||||
underscore/underscore.js \
|
||||
|
||||
|
|
|
|||
|
|
@ -22,7 +22,6 @@
|
|||
"jszip": "3.1.4",
|
||||
"less": "3.9.0",
|
||||
"source-map": "0.7.3",
|
||||
"raven-js": "3.27.0",
|
||||
"timeago": "1.6.2",
|
||||
"uglify-js": "3.4.9",
|
||||
"underscore": "1.9.1"
|
||||
|
|
|
|||
|
|
@ -142,10 +142,6 @@ CORS_ENDPOINT_OVERRIDES = cors_endpoint_overrides(
|
|||
['amo.addons-dev.allizom.org', 'localhost:3000']
|
||||
)
|
||||
|
||||
RAVEN_JS_DSN = (
|
||||
'https://5686e2a8f14446a3940c651c6a14dc73@sentry.prod.mozaws.net/75')
|
||||
RAVEN_JS_ALLOW_LIST = ['addons-dev.allizom.org', 'addons-dev-cdn.allizom.org']
|
||||
|
||||
FXA_SQS_AWS_QUEUE_URL = (
|
||||
'https://sqs.us-east-1.amazonaws.com/927034868273/'
|
||||
'amo-account-change-dev')
|
||||
|
|
|
|||
|
|
@ -110,11 +110,6 @@ VALIDATOR_TIMEOUT = 360
|
|||
|
||||
ES_DEFAULT_NUM_SHARDS = 10
|
||||
|
||||
RAVEN_JS_DSN = (
|
||||
'https://8c1c5936578948a9a0614cbbafccf049@sentry.prod.mozaws.net/78')
|
||||
RAVEN_JS_ALLOW_LIST = ['addons.mozilla.org', 'addons.cdn.mozilla.net']
|
||||
|
||||
|
||||
RECOMMENDATION_ENGINE_URL = env(
|
||||
'RECOMMENDATION_ENGINE_URL',
|
||||
default='https://taar.prod.mozaws.net/api/recommendations/')
|
||||
|
|
|
|||
|
|
@ -134,10 +134,6 @@ FXA_CONFIG = {
|
|||
DEFAULT_FXA_CONFIG_NAME = 'default'
|
||||
ALLOWED_FXA_CONFIGS = ['default', 'amo', 'local']
|
||||
|
||||
RAVEN_JS_DSN = (
|
||||
'https://e35602be5252460d97587478bcc642df@sentry.prod.mozaws.net/77')
|
||||
RAVEN_JS_ALLOW_LIST = ['addons.allizom.org', 'addons-cdn.allizom.org']
|
||||
|
||||
TAAR_LITE_RECOMMENDATION_ENGINE_URL = env(
|
||||
'TAAR_LITE_RECOMMENDATION_ENGINE_URL',
|
||||
default=('https://taarlite.prod.mozaws.net/taarlite/api/v1/'
|
||||
|
|
|
|||
|
|
@ -44,8 +44,6 @@
|
|||
{% set user_authenticated = request.user.is_authenticated %}
|
||||
|
||||
<body class="html-{{ DIR }} {{ request.APP.short }} {% if user_authenticated %}user-signedin{% endif %} {% block bodyclass %}{% endblock %}"
|
||||
data-raven-dsn="{{ settings.RAVEN_JS_DSN }}"
|
||||
data-raven-urls="{{ settings.RAVEN_JS_ALLOW_LIST }}"
|
||||
{% block bodyattrs %}{% endblock %}>
|
||||
|
||||
|
||||
|
|
|
|||
|
|
@ -786,8 +786,6 @@ MINIFY_BUNDLES = {
|
|||
'js': {
|
||||
# JS files common to the entire site (pre-impala).
|
||||
'common': (
|
||||
'js/node_lib/raven.js',
|
||||
'js/common/raven-config.js',
|
||||
'js/node_lib/underscore.js',
|
||||
'js/zamboni/browser.js',
|
||||
'js/amo2009/addons.js',
|
||||
|
|
@ -875,8 +873,6 @@ MINIFY_BUNDLES = {
|
|||
# Impala: Things to be loaded at the bottom
|
||||
'impala': (
|
||||
'js/lib/ngettext-overload.js',
|
||||
'js/node_lib/raven.js',
|
||||
'js/common/raven-config.js',
|
||||
'js/node_lib/underscore.js',
|
||||
'js/impala/carousel.js',
|
||||
'js/zamboni/browser.js',
|
||||
|
|
|
|||
|
|
@ -51,8 +51,6 @@
|
|||
data-readonly="{{ settings.READ_ONLY|json }}"
|
||||
data-media-url="{{ MEDIA_URL }}"
|
||||
data-static-url="{{ STATIC_URL }}"
|
||||
data-raven-dsn="{{ settings.RAVEN_JS_DSN }}"
|
||||
data-raven-urls="{{ settings.RAVEN_JS_ALLOW_LIST }}"
|
||||
{% block bodyattrs %}{% endblock %}>
|
||||
|
||||
<div id="main-wrapper">
|
||||
|
|
|
|||
|
|
@ -40,8 +40,6 @@
|
|||
data-readonly="{{ settings.READ_ONLY|json }}"
|
||||
data-media-url="{{ MEDIA_URL }}"
|
||||
data-static-url="{{ STATIC_URL }}"
|
||||
data-raven-dsn="{{ settings.RAVEN_JS_DSN }}"
|
||||
data-raven-urls="{{ settings.RAVEN_JS_ALLOW_LIST }}"
|
||||
data-fxa-config="{{ fxa_config()|json }}"
|
||||
{% block bodyattrs %}{% endblock %}>
|
||||
|
||||
|
|
|
|||
|
|
@ -1,12 +0,0 @@
|
|||
(function() {
|
||||
var $body = $('body');
|
||||
var dsn = $body.data('raven-dsn');
|
||||
var urls = $body.data('raven-urls');
|
||||
if (dsn && urls) {
|
||||
Raven.config(dsn, {
|
||||
whitelistUrls: urls,
|
||||
}).install();
|
||||
} else {
|
||||
console.log('Skipping ravenJS installation');
|
||||
}
|
||||
})();
|
||||
|
|
@ -44,16 +44,6 @@
|
|||
json = JSON.parse(response);
|
||||
} catch(err) {
|
||||
errors = [gettext("There was a problem contacting the server.")];
|
||||
try {
|
||||
Raven.captureMessage('Error parsing upload status JSON.', {
|
||||
extra: {
|
||||
status_code: statusCode,
|
||||
content: response,
|
||||
},
|
||||
});
|
||||
} catch (e) {
|
||||
console.log(e);
|
||||
}
|
||||
}
|
||||
if (!errors.length) {
|
||||
errors = settings.getErrors(json);
|
||||
|
|
|
|||
Загрузка…
Ссылка в новой задаче