From 03d023e23feed6491d6587aaecd23cb16c9b88e0 Mon Sep 17 00:00:00 2001
From: Sergejs Luhmirins <sergejs@yetanotherapp.dev>
Date: Tue, 15 Jul 2025 09:30:20 +0300
Subject: [PATCH] Disable file log writer on per-oreo Android versions

---
 .../overview/OverviewFragment.kt              |  5 ++++
 .../fragment_troubleshooting_overview.xml     |  1 +
 .../simprints/infra/logging/SimberBuilder.kt  | 27 ++++++++++++-------
 3 files changed, 24 insertions(+), 9 deletions(-)

diff --git a/feature/troubleshooting/src/main/java/com/simprints/feature/troubleshooting/overview/OverviewFragment.kt b/feature/troubleshooting/src/main/java/com/simprints/feature/troubleshooting/overview/OverviewFragment.kt
index 2dcd232bce..41c9961c12 100644
--- a/feature/troubleshooting/src/main/java/com/simprints/feature/troubleshooting/overview/OverviewFragment.kt
+++ b/feature/troubleshooting/src/main/java/com/simprints/feature/troubleshooting/overview/OverviewFragment.kt
@@ -1,9 +1,11 @@
 package com.simprints.feature.troubleshooting.overview
 
 import android.content.Intent
+import android.os.Build
 import android.os.Bundle
 import android.view.View
 import androidx.core.content.FileProvider
+import androidx.core.view.isVisible
 import androidx.fragment.app.Fragment
 import androidx.fragment.app.viewModels
 import com.simprints.feature.troubleshooting.R
@@ -68,6 +70,9 @@ internal class OverviewFragment : Fragment(R.layout.fragment_troubleshooting_ove
         binding.troubleshootOverviewPing.setOnClickListener {
             viewModel.pingServer()
         }
+
+        // Log export is only Available starting from Android 8
+        binding.troubleshootOverviewExportLogsBlock.isVisible = Build.VERSION.SDK_INT >= Build.VERSION_CODES.O
         binding.troubleshootOverviewExportLogs.setOnClickListener { v ->
             viewModel.exportLogs()
         }
diff --git a/feature/troubleshooting/src/main/res/layout/fragment_troubleshooting_overview.xml b/feature/troubleshooting/src/main/res/layout/fragment_troubleshooting_overview.xml
index 6f15691b04..8b7dbcff5f 100644
--- a/feature/troubleshooting/src/main/res/layout/fragment_troubleshooting_overview.xml
+++ b/feature/troubleshooting/src/main/res/layout/fragment_troubleshooting_overview.xml
@@ -127,6 +127,7 @@
             android:textIsSelectable="true" />
 
         <LinearLayout
+            android:id="@+id/troubleshootOverviewExportLogsBlock"
             android:layout_width="match_parent"
             android:layout_height="wrap_content"
             android:layout_marginVertical="16dp"
diff --git a/infra/logging/src/main/java/com/simprints/infra/logging/SimberBuilder.kt b/infra/logging/src/main/java/com/simprints/infra/logging/SimberBuilder.kt
index 5c4992362f..aa75570ee5 100644
--- a/infra/logging/src/main/java/com/simprints/infra/logging/SimberBuilder.kt
+++ b/infra/logging/src/main/java/com/simprints/infra/logging/SimberBuilder.kt
@@ -1,6 +1,7 @@
 package com.simprints.infra.logging
 
 import android.content.Context
+import android.os.Build
 import co.touchlab.kermit.LogcatWriter
 import co.touchlab.kermit.Logger
 import co.touchlab.kermit.Severity
@@ -12,21 +13,29 @@ import com.simprints.infra.logging.writers.FileLogWriter
 
 object SimberBuilder {
     /**
-     * Simber needs to be initialized with the application context. It can be initialized multiple
-     * times without issue. Re-initializing Simber uproots and replants all trees.
-     * @param context Application Context
+     * Initializes the Simber logging framework with appropriate writers and severity based on build type.
+     *
+     * - Debug builds use `LogcatWriter` with `Severity.Debug`.
+     * - Release builds use `AnalyticsPropertyLogWriter` and `CrashlyticsLogWriter` with `Severity.Info`.
+     *   - `FileLogWriter` is added for Android versions starting from API26.
      */
     fun initialize(context: Context) {
-        if (BuildConfig.DEBUG) {
-            Logger.setLogWriters(LogcatWriter())
-            Logger.setMinSeverity(Severity.Debug)
-        } else {
-            Logger.setLogWriters(
+        when {
+            BuildConfig.DEBUG -> Logger.setLogWriters(LogcatWriter())
+
+            Build.VERSION.SDK_INT < Build.VERSION_CODES.O -> Logger.setLogWriters(
+                AnalyticsPropertyLogWriter(FirebaseAnalytics.getInstance(context)),
+                CrashlyticsLogWriter(FirebaseCrashlytics.getInstance()),
+                // Skip file logger to avoid crashes on Android 6-7 - https://simprints.atlassian.net/browse/MS-1060
+            )
+
+            else -> Logger.setLogWriters(
                 AnalyticsPropertyLogWriter(FirebaseAnalytics.getInstance(context)),
                 CrashlyticsLogWriter(FirebaseCrashlytics.getInstance()),
                 FileLogWriter(context),
             )
-            Logger.setMinSeverity(Severity.Info)
         }
+
+        Logger.setMinSeverity(if (BuildConfig.DEBUG) Severity.Debug else Severity.Info)
     }
 }