Browse Source

fix(feedback): check for data before accessing property (#66637)

fixes JAVASCRIPT-2S08
Michelle Zhang 1 year ago
parent
commit
b64382fddb

+ 6 - 6
static/app/components/onboarding/gettingStartedDoc/utils/feedbackOnboarding.tsx

@@ -144,11 +144,11 @@ export const getCrashReportJavaScriptInstallStep = params => [
 
 export function getCrashReportSDKInstallFirstStep(params) {
   const version =
-    params.sourcePackageRegistries.data['sentry.javascript.browser'].version;
+    params.sourcePackageRegistries.data?.['sentry.javascript.browser'].version ?? '';
   const hash =
-    params.sourcePackageRegistries.data['sentry.javascript.browser'].files[
+    params.sourcePackageRegistries.data?.['sentry.javascript.browser']?.files[
       'bundle.min.js'
-    ].checksums['sha384-base64'];
+    ].checksums['sha384-base64'] ?? '';
 
   return {
     description: t('Make sure you have the JavaScript SDK available:'),
@@ -225,11 +225,11 @@ export const getCrashReportBackendInstallStep = params => [
 
 export function getCrashReportSDKInstallFirstStepRails(params) {
   const version =
-    params.sourcePackageRegistries.data['sentry.javascript.browser'].version;
+    params.sourcePackageRegistries.data?.['sentry.javascript.browser'].version ?? '';
   const hash =
-    params.sourcePackageRegistries.data['sentry.javascript.browser'].files[
+    params.sourcePackageRegistries.data?.['sentry.javascript.browser']?.files[
       'bundle.min.js'
-    ].checksums['sha384-base64'];
+    ].checksums['sha384-base64'] ?? '';
 
   return {
     description: t('Make sure you have the JavaScript SDK available:'),