Revert using StrictMode VmPolicy only in debug build type (fixes #2032)…

… (#2033) This reverts commit f4ccad8.
syncthing · Jan 2, 2024 · 58e0230 · 58e0230
1 parent 9ed273a
commit 58e0230
Showing 1 changed file with 4 additions and 7 deletions.
diff --git a/app/src/main/java/com/nutomic/syncthingandroid/SyncthingApp.java b/app/src/main/java/com/nutomic/syncthingandroid/SyncthingApp.java
@@ -25,13 +25,10 @@ public void onCreate() {
 
         new Languages(this).setLanguage(this);
 
-        if (BuildConfig.DEBUG) {
-            setStrictMode();
-        }
-    }
-
-    private void setStrictMode() {
-        // Set VM policy to avoid crash when sending folder URI to file manager.
+        // The main point here is to use a VM policy without
+        // `detectFileUriExposure`, as that leads to exceptions when e.g.
+        // opening the ignores file. And it's enabled by default.
+        // We might want to disable `detectAll` and `penaltyLog` on release (non-RC) builds too.
         StrictMode.VmPolicy policy = new StrictMode.VmPolicy.Builder()
                 .detectAll()
                 .penaltyLog()