fix(expo): Upload script should not log out env file not found errors (…

…getsentry#4332)
blitzstudios · Dec 3, 2024 · 9282172 · 9282172
1 parent f6c37bb
commit 9282172
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 2 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -38,6 +38,7 @@
 ### Fixes
 
 - Return `lastEventId` export from `@sentry/core` ([#4315](https://github.com/getsentry/sentry-react-native/pull/4315))
+- Don't log file not found errors when loading envs in `sentry-expo-upload-sourcemaps` ([#4332](https://github.com/getsentry/sentry-react-native/pull/4332))
 - Navigation Span should have no parent by default ([#4326](https://github.com/getsentry/sentry-react-native/pull/4326))
 
 ### Dependencies

diff --git a/packages/core/scripts/expo-upload-sourcemaps.js b/packages/core/scripts/expo-upload-sourcemaps.js
@@ -111,8 +111,12 @@ function loadDotenv(dotenvPath) {
 
     Object.assign(process.env, dotenvResult);
   } catch (error) {
-    console.warn('⚠️ Failed to load environment variables using dotenv.');
-    console.warn(error);
+    if (error.code === 'ENOENT') {
+      // noop if file does not exist
+    } else {
+      console.warn('⚠️ Failed to load environment variables using dotenv.');
+      console.warn(error);
+    }
   }
 }