Skip to content

Commit c05030a

Browse files
committed
Manually blacklist all devtools errors
1 parent f671ceb commit c05030a

File tree

1 file changed

+9
-7
lines changed

1 file changed

+9
-7
lines changed

packages/common/src/utils/analytics/sentry.ts

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,16 @@ export async function initialize(dsn: string) {
2323
// Chrome extensions
2424
/extensions\//i,
2525
/^chrome:\/\//i,
26+
27+
// react devtools Outside of our scope for now, but we definitely want to check this out.
28+
// TODO: check what's happening here: https://sentry.io/organizations/codesandbox/issues/1239466583/?project=155188&query=is%3Aunresolved+release%3APROD-1573653062-4134efc0a
29+
/because a node with that id is already in the Store/,
30+
/Node \d* was removed before its children\./,
31+
/Cannot remove node \d* because no matching node was found in the Store\./,
32+
/Cannot add child \d* to parent \d* because parent node was not found in the Store\./,
33+
/Children cannot be added or removed during a reorder operation\./,
2634
],
27-
// whitelistUrls: [/https?:\/\/((uploads|www)\.)?codesandbox\.io/],
35+
whitelistUrls: [/https?:\/\/((uploads|www)\.)?codesandbox\.io/],
2836
/**
2937
* Don't send messages from the sandbox, so don't send from eg.
3038
* new.codesandbox.io or new.csb.app
@@ -66,12 +74,6 @@ export async function initialize(dsn: string) {
6674
// We need to add sourcemaps
6775
return null;
6876
}
69-
70-
if (filename.includes('react-devtools-inline')) {
71-
// Outside of our scope for now, but we definitely want to check this out.
72-
// TODO: check what's happening here: https://sentry.io/organizations/codesandbox/issues/1239466583/?project=155188&query=is%3Aunresolved+release%3APROD-1573653062-4134efc0a
73-
return null;
74-
}
7577
}
7678

7779
const customError = ((hint && (hint.originalException as any)) || {})

0 commit comments

Comments
 (0)