Jelajahi Sumber

Merge pull request #4837 from overleaf/tm-cdn-load-error-reporting

Move CDN load test to separate file and report error to sentry

GitOrigin-RevId: 5870fae9939a62848d3a39899a1da3608935988b
Jakob Ackermann 5 tahun lalu
induk
melakukan
d70937e0da

+ 1 - 7
services/web/app/views/layout.pug

@@ -134,10 +134,4 @@ html(
 					window.location.search += "&nocdn=true";
 				}
 			if hasFeature('saas')
-				script(type="text/javascript", nonce=scriptNonce).
-					//- Test for CDN availability and report to sentry if loading failed
-					var cdnLoadTest = document.createElement('img')
-					cdnLoadTest.addEventListener('error', function () {
-						throw new Error('CDN test image load error (cdn.overleaf.net)')
-					})
-					cdnLoadTest.src = 'https://cdn.overleaf.net/img/1p.gif'
+				script(type="text/javascript", nonce=scriptNonce, src=buildJsPath('cdn-load-test.js'))

+ 7 - 0
services/web/frontend/js/cdn-load-test.js

@@ -0,0 +1,7 @@
+import { captureMessage } from './infrastructure/error-reporter'
+
+var cdnLoadTest = document.createElement('img')
+cdnLoadTest.addEventListener('error', function (event) {
+  captureMessage('CDN test image load error (cdn.overleaf.net)')
+})
+cdnLoadTest.src = 'https://cdn.overleaf.net/img/1p.gif'

+ 1 - 0
services/web/webpack.config.js

@@ -14,6 +14,7 @@ const entryPoints = {
   serviceWorker: './frontend/js/serviceWorker.js',
   main: './frontend/js/main.js',
   ide: './frontend/js/ide.js',
+  'cdn-load-test': './frontend/js/cdn-load-test.js',
   style: './frontend/stylesheets/style.less',
   'ieee-style': './frontend/stylesheets/ieee-style.less',
   'light-style': './frontend/stylesheets/light-style.less',