More event filtering for Sentry error reporting.

changelog.html

@@ -1,3 +1,9 @@
+<div class="list-header">4.0.0-beta2.9<span>@665575cf426293ec11da</span> <time datetime="2018-04-11">(2018-04-11)</time></div>
+<ul class="chat-menu-content menu-side-padding">
+	<li>Changed: Allow clicking on a rich chat embed even if it fails to load.</li>
+	<li>Fixed: Clean up more errors for Sentry logging.</li>
+</ul>
+
 <div class="list-header">4.0.0-beta2.6<span>@b85fa005ec1f3929cdd8</span> <time datetime="2018-04-11">(2018-04-11)</time></div>
 <ul class="chat-menu-content menu-side-padding">
 	<li>Changed: Filter a bunch of errors out from Sentry logging.</li>

src/experiments.json

@@ -0,0 +1,10 @@
+{
+	"api_load": {
+		"name": "New API Stress Testing",
+		"description": "Send duplicate requests to the new API server for load testing.",
+		"groups": [
+			{"value": true, "weight": 0},
+			{"value": false, "weight": 100}
+		]
+	}
+}

src/main.js

@@ -30,7 +30,7 @@ class FrankerFaceZ extends Module {
 		// Error Reporting and Logging
 		// ========================================================================
 
-		//if ( ! DEBUG )
+		if ( ! DEBUG )
 			this.inject('raven', RavenLogger);
 
 		this.log = new Logger(null, null, null, this.raven);
@@ -100,7 +100,7 @@ class FrankerFaceZ extends Module {
 FrankerFaceZ.Logger = Logger;
 
 const VER = FrankerFaceZ.version_info = {
-	major: 4, minor: 0, revision: 0, extra: '-beta2.7',
+	major: 4, minor: 0, revision: 0, extra: '-beta2.9',
 	build: __webpack_hash__,
 	toString: () =>
 		`${VER.major}.${VER.minor}.${VER.revision}${VER.extra || ''}${DEBUG ? '-dev' : ''}`

src/modules/chat/rich_providers.js

@@ -31,6 +31,8 @@ export const Clips = {
 			return;
 
 		return {
+			url: token.url,
+
 			getData: async () => {
 				const result = await apollo.client.query({
 					query: GET_CLIP,
@@ -39,6 +41,9 @@ export const Clips = {
 					}
 				});
 
+				if ( ! result || ! result.data || ! result.data.clip )
+					return null;
+
 				const clip = result.data.clip,
 					user = clip.broadcaster.displayName,
 					game = clip.game,

src/sites/twitch-twilight/modules/chat/rich_content.jsx

@@ -48,8 +48,16 @@ export default class RichContent extends Module {
 							data = await data;
 					}
 
+					if ( ! data )
+						data = {
+							error: true,
+							title: t.i18n.t('card.error', 'An error occured.'),
+							desc_1: t.i18n.t('card.empty', 'No data was returned.')
+						}
+
 					this.setState(Object.assign({
-						loaded: true
+						loaded: true,
+						url: this.props.url
 					}, data));
 
 				} catch(err) {
@@ -59,6 +67,7 @@ export default class RichContent extends Module {
 					this.setState({
 						loaded: true,
 						error: true,
+						url: this.props.url,
 						title: t.i18n.t('card.error', 'An error occured.'),
 						desc_1: String(err)
 					});

src/utilities/compat/apollo.js

@@ -117,7 +117,7 @@ export default class Apollo extends Module {
 							next: result => {
 								if ( result.errors ) {
 									const name = operation.operationName;
-									if ( name.includes('FFZ') || has(this.modifiers, name) || has(this.post_modifiers, name) ) {
+									if ( name && (name.includes('FFZ') || has(this.modifiers, name) || has(this.post_modifiers, name)) ) {
 										for(const err of result.errors) {
 											if ( skip_error(err) )
 												continue;