Browse Source

ref(ui): Update Sentry to js sdk 7 beta (#34286)

* ref(ui): Update Sentry to js sdk 7 beta

This will let us try out the new beta sdk on Sentry itself to find out if we have any issues earlier and give feedback to the sdk team.
Kev 2 years ago
parent
commit
619cdcb526

+ 6 - 6
package.json

@@ -22,13 +22,13 @@
     "@emotion/react": "^11.4.0",
     "@emotion/styled": "^11.3.0",
     "@sentry-internal/global-search": "^0.0.43",
-    "@sentry/integrations": "6.18.1-beta.0",
-    "@sentry/node": "6.18.1-beta.0",
-    "@sentry/react": "6.18.1-beta.0",
+    "@sentry/integrations": "7.0.0-beta.1",
+    "@sentry/node": "7.0.0-beta.1",
+    "@sentry/react": "7.0.0-beta.1",
     "@sentry/release-parser": "^1.3.0",
     "@sentry/rrweb": "^0.3.1",
-    "@sentry/tracing": "6.18.1-beta.0",
-    "@sentry/utils": "6.18.1-beta.0",
+    "@sentry/tracing": "7.0.0-beta.1",
+    "@sentry/utils": "7.0.0-beta.1",
     "@testing-library/jest-dom": "^5.16.1",
     "@testing-library/react": "^12.1.2",
     "@testing-library/react-hooks": "^7.0.2",
@@ -173,7 +173,7 @@
     "jest-canvas-mock": "^2.3.1",
     "jest-fetch-mock": "^3.0.3",
     "jest-junit": "13.0.0",
-    "jest-sentry-environment": "^1.6.0",
+    "jest-sentry-environment": "1.7.1",
     "postcss-jsx": "0.36.4",
     "postcss-syntax": "0.36.2",
     "react-refresh": "0.11.0",

+ 2 - 2
static/app/actionCreators/release.tsx

@@ -46,7 +46,7 @@ export function getProjectRelease(api: Client, params: ParamsGet) {
 
       ReleaseActions.loadReleaseError(projectSlug, releaseVersion, err);
       Sentry.withScope(scope => {
-        scope.setLevel(Sentry.Severity.Warning);
+        scope.setLevel('warning');
         scope.setFingerprint(['getRelease-action-creator']);
         Sentry.captureException(err);
       });
@@ -80,7 +80,7 @@ export function getReleaseDeploys(api: Client, params: ParamsGet) {
 
       ReleaseActions.loadDeploysError(projectSlug, releaseVersion, err);
       Sentry.withScope(scope => {
-        scope.setLevel(Sentry.Severity.Warning);
+        scope.setLevel('warning');
         scope.setFingerprint(['getReleaseDeploys-action-creator']);
         Sentry.captureException(err);
       });

+ 1 - 1
static/app/actionCreators/repositories.tsx

@@ -30,7 +30,7 @@ export function getRepositories(api: Client, params: ParamsGet) {
     .catch(err => {
       RepositoryActions.loadRepositoriesError(err);
       Sentry.withScope(scope => {
-        scope.setLevel(Sentry.Severity.Warning);
+        scope.setLevel('warning');
         scope.setFingerprint(['getRepositories-action-creator']);
         Sentry.captureException(err);
       });

+ 2 - 3
static/app/api.tsx

@@ -1,5 +1,4 @@
 import {browserHistory} from 'react-router';
-import {Severity} from '@sentry/react';
 import Cookies from 'js-cookie';
 import isUndefined from 'lodash/isUndefined';
 import * as qs from 'query-string';
@@ -422,7 +421,7 @@ export class Client {
             errorObjectToUse.removeFrames(3);
 
             // Setting this to warning because we are going to capture all failed requests
-            scope.setLevel(Severity.Warning);
+            scope.setLevel('warning');
             scope.setTag('http.statusCode', String(resp.status));
             scope.setTag('error.reason', errorThrown);
             Sentry.captureException(errorObjectToUse);
@@ -554,7 +553,7 @@ export class Client {
         // The request failed for other reason
         run(Sentry =>
           Sentry.withScope(scope => {
-            scope.setLevel(Severity.Warning);
+            scope.setLevel('warning');
             Sentry.captureException(err);
           })
         );

+ 1 - 1
static/app/bootstrap/initializeLocale.tsx

@@ -23,7 +23,7 @@ async function getTranslations(language: string) {
     return await import(`sentry-locale/${language}/LC_MESSAGES/django.po`);
   } catch (e) {
     Sentry.withScope(scope => {
-      scope.setLevel(Sentry.Severity.Warning);
+      scope.setLevel('warning');
       scope.setFingerprint(['sentry-locale-not-found']);
       scope.setExtra('locale', language);
       Sentry.captureException(e);

+ 1 - 1
static/app/bootstrap/initializeSdk.tsx

@@ -82,7 +82,7 @@ export function initializeSdk(config: Config, {routes}: {routes?: Function} = {}
      * deployed separately from backend.
      */
     release: SENTRY_RELEASE_VERSION ?? sentryConfig?.release,
-    whitelistUrls: SPA_DSN
+    allowUrls: SPA_DSN
       ? ['localhost', 'dev.getsentry.net', 'sentry.dev', 'webpack-internal://']
       : sentryConfig?.whitelistUrls,
     integrations: getSentryIntegrations(hasReplays, routes),

+ 1 - 1
static/app/components/asyncComponent.tsx

@@ -328,7 +328,7 @@ class AsyncComponent<
       Sentry.addBreadcrumb({
         message: error.responseText,
         category: 'xhr',
-        level: Sentry.Severity.Error,
+        level: 'error',
       });
     }
     this.setState(prevState => {

+ 1 - 1
static/app/components/dropdownMenu.tsx

@@ -160,7 +160,7 @@ class DropdownMenu extends Component<Props, State> {
     if (!this.dropdownActor) {
       // Log an error, should be lower priority
       Sentry.withScope(scope => {
-        scope.setLevel(Sentry.Severity.Warning);
+        scope.setLevel('warning');
         Sentry.captureException(new Error('DropdownMenu does not have "Actor" attached'));
       });
     }

+ 1 - 1
static/app/components/events/eventEntries.tsx

@@ -235,7 +235,7 @@ const EventEntries = memo(
 
       if (proGuardImage) {
         Sentry.withScope(function (s) {
-          s.setLevel(Sentry.Severity.Warning);
+          s.setLevel('warning');
           if (event.sdk) {
             s.setTag('offending.event.sdk.name', event.sdk.name);
             s.setTag('offending.event.sdk.version', event.sdk.version);

+ 1 - 1
static/app/components/events/interfaces/debugMeta-v2/debugImage/processingIcon.tsx

@@ -77,7 +77,7 @@ function ProcessingIcon({status}: Props) {
 
     default: {
       Sentry.withScope(scope => {
-        scope.setLevel(Sentry.Severity.Warning);
+        scope.setLevel('warning');
         Sentry.captureException(new Error('Unknown image ProcessingIcon status'));
       });
       return null; // This shall not happen

Some files were not shown because too many files changed in this diff