Use plain text values for Sentry + env-injected token (#1850)

mokagio · web-flow · commit 24f01a07359d · 2026-07-30T19:36:45.000+10:00
diff --git a/.configure b/.configure
@@ -8,11 +8,6 @@
       "destination": "Simplenote/gradle.properties",
       "encrypt": true
     },
-    {
-      "file": "android/simplenote/sentry.properties",
-      "destination": "sentry.properties",
-      "encrypt": true
-    },
     {
       "file": "android/automattic.jks",
       "destination": ".configure-files/release.jks",
@@ -32,4 +27,4 @@
   "file_dependencies": [
 
   ]
-}
+}
diff --git a/.configure-files/sentry.properties.enc b/.configure-files/sentry.properties.enc
diff --git a/Simplenote/build.gradle b/Simplenote/build.gradle
@@ -42,6 +42,8 @@ android {
 
         minSdkVersion 23
         targetSdkVersion 35
+
+        buildConfigField "String", "SENTRY_DSN", "\"https://89e443ca3fd84e588d1809cfbfcc40f4@sentry.io/1455558\""
     }
 
 
@@ -69,6 +71,13 @@ android {
     }
 }
 
+sentry {
+    org = "a8c"
+    projectName = "simplenote-android"
+    authToken = providers.environmentVariable("SENTRY_AUTH_TOKEN").orNull
+    autoInstallation.enabled = false
+}
+
 buildscript {
     dependencies {
         classpath "io.sentry:sentry-android-gradle-plugin:$sentryVersion"
@@ -219,7 +228,6 @@ android.applicationVariants.all { variant ->
             key == "GOOGLE_ANALYTICS_ID" ||
             key == "LOGIN_EMAIL" ||
             key == "LOGIN_PASSWORD" ||
-            key == "SENTRY_DSN" ||
             key == "WPCOM_CLIENT_ID" ||
             key == "SCREENSHOT_EMAIL" ||
             key == "SCREENSHOT_PASSWORD") {
diff --git a/Simplenote/gradle.properties-example b/Simplenote/gradle.properties-example
@@ -2,7 +2,6 @@ crashlyticsApiKey = 0
 googleAnalyticsId =
 loginEmail =
 loginPassword =
-sentryDsn=https://00000000000000000000000000000000@sentry.io/00000000
 simperiumAppId = history-analyst-dad
 simperiumAppKey = dccacc59bbef4982a15abaeafaf7bc8a
 wpcomClientId =
diff --git a/Simplenote/src/main/java/com/automattic/simplenote/utils/crashlogging/SimplenoteCrashLoggingDataProvider.kt b/Simplenote/src/main/java/com/automattic/simplenote/utils/crashlogging/SimplenoteCrashLoggingDataProvider.kt
@@ -45,6 +45,10 @@ class SimplenoteCrashLoggingDataProvider @Inject constructor(
         get() = PerformanceMonitoringConfig.Disabled
 
     override fun crashLoggingEnabled(): Boolean {
+        if (BuildConfig.DEBUG) {
+            return false
+        }
+
         return Simplenote.analyticsIsEnabled()
     }
 

Original file line number	Diff line number	Diff line change
`@@ -45,6 +45,10 @@ class SimplenoteCrashLoggingDataProvider @Inject constructor(`
`45`	`45`	`get() = PerformanceMonitoringConfig.Disabled`
`46`	`46`
`47`	`47`	`override fun crashLoggingEnabled(): Boolean {`
	`48`	`+ if (BuildConfig.DEBUG) {`
	`49`	`+ return false`
	`50`	`+ }`
	`51`	`+`
`48`	`52`	`return Simplenote.analyticsIsEnabled()`
`49`	`53`	`}`
`50`	`54`