Skip to content

change: [M3-10534] - Error in Sentry: window.crypto.randomUUID is not a function #12818

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bill-akamai merged 10 commits into from
Sep 10, 2025
Merged

change: [M3-10534] - Error in Sentry: window.crypto.randomUUID is not a function #12818

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
79afeed
Add crypto API availability check for OAuth
bill-akamai Sep 4, 2025
f8c91b8
Remove comment
bill-akamai Sep 4, 2025
207a301
Added changeset: Guard against missing window.crypto.randomUUID in OAuth
bill-akamai Sep 4, 2025
0ae0d86
Add comment
bill-akamai Sep 4, 2025
5d1d701
Disable sonarjs rule
bill-akamai Sep 5, 2025
469e01e
Merge branch 'develop' into M3-10534
bill-akamai Sep 9, 2025
656e6f6
Ignore error in initSentry.ts
bill-akamai Sep 9, 2025
7da89c0
Update changeset
bill-akamai Sep 9, 2025
8a045b8
Add beforeSend test
bill-akamai Sep 9, 2025
632c5a1
Add type for error
bill-akamai Sep 10, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 5 additions & 0 deletions packages/manager/.changeset/pr-12818-changed-1757021047591.md
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,5 @@
---
'@linode/manager': Changed
---

Ignore window.crypto.randomUUID error in Sentry config file ([#12818](https://github.com/linode/manager/pull/12818))
38 changes: 37 additions & 1 deletion packages/manager/src/initSentry.test.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,4 +1,6 @@
import { normalizeErrorMessage } from './initSentry';
import { ErrorEvent } from '@sentry/react';

Check warning on line 1 in packages/manager/src/initSentry.test.ts

View workflow job for this annotation

GitHub Actions / ESLint Review (manager) 

[eslint] reported by reviewdog 🐶
All imports in the declaration are only used as types. Use `import type`.

Raw Output:
{"ruleId":"@typescript-eslint/consistent-type-imports","severity":1,"message":"All imports in the declaration are only used as types. Use `import type`.","line":1,"column":1,"nodeType":"ImportDeclaration","messageId":"typeOverValue","endLine":1,"endColumn":44,"fix":{"range":[6,6],"text":" type"}}

import { beforeSend, normalizeErrorMessage } from './initSentry';

import type { APIError } from '@linode/api-v4/lib/types';

Expand All @@ -21,3 +23,37 @@
expect(normalizeErrorMessage(null as any)).toBe('null');
});
});

describe('beforeSend', () => {
it('should return null when user agent contains Catchpoint', () => {
const mockSentryEvent: ErrorEvent = {
message: 'Some error occurred',

Check warning on line 30 in packages/manager/src/initSentry.test.ts

View workflow job for this annotation

GitHub Actions / ESLint Review (manager) 

[eslint] reported by reviewdog 🐶
Define a constant instead of duplicating this literal 3 times.

Raw Output:
{"ruleId":"sonarjs/no-duplicate-string","severity":1,"message":"Define a constant instead of duplicating this literal 3 times.","line":30,"column":16,"nodeType":"Literal","endLine":30,"endColumn":37}
request: {
headers: {
'User-Agent': 'Mozilla/5.0 (compatible; Catchpoint)',
},
},
type: undefined,

Check warning on line 36 in packages/manager/src/initSentry.test.ts

View workflow job for this annotation

GitHub Actions / ESLint Review (manager) 

[eslint] reported by reviewdog 🐶
Use null instead.

Raw Output:
{"ruleId":"sonarjs/no-undefined-assignment","severity":1,"message":"Use null instead.","line":36,"column":13,"nodeType":"Identifier","messageId":"useNull","endLine":36,"endColumn":22}
};

const result = beforeSend(mockSentryEvent);
expect(result).toBeNull();
});

it('should process normal events when user agent does not contain Catchpoint', () => {
const mockSentryEvent: ErrorEvent = {
message: 'Some error occurred',
request: {
headers: {
'User-Agent':
'Mozilla/5.0 (Windows NT 10.0; Win64; x64) AppleWebKit/537.36',
},
},
type: undefined,

Check warning on line 52 in packages/manager/src/initSentry.test.ts

View workflow job for this annotation

GitHub Actions / ESLint Review (manager) 

[eslint] reported by reviewdog 🐶
Use null instead.

Raw Output:
{"ruleId":"sonarjs/no-undefined-assignment","severity":1,"message":"Use null instead.","line":52,"column":13,"nodeType":"Identifier","messageId":"useNull","endLine":52,"endColumn":22}
};

const result = beforeSend(mockSentryEvent);
expect(result).not.toBeNull();
expect(result?.message).toBe('Some error occurred');
});
});
9 changes: 8 additions & 1 deletion packages/manager/src/initSentry.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -88,9 +88,16 @@ export const initSentry = () => {
}
};

const beforeSend = (sentryEvent: SentryErrorEvent): null | SentryErrorEvent => {
export const beforeSend = (
sentryEvent: SentryErrorEvent
): null | SentryErrorEvent => {
const normalizedErrorMessage = normalizeErrorMessage(sentryEvent.message);

const userAgent = sentryEvent.request?.headers?.['User-Agent'];
if (userAgent?.includes('Catchpoint')) {
return null;
}

if (
errorsToIgnore.some((eachRegex) =>
Boolean(normalizedErrorMessage?.match(eachRegex))
Expand Down