Revert "Add ACRA for crash reports (#180)" (#185)

This reverts commit 3d0cf8b902.

app/build.gradle.kts

@@ -22,19 +22,6 @@ android {
         testInstrumentationRunner = "gq.kirmanak.mealient.MealientTestRunner"
         testInstrumentationRunnerArguments += mapOf("clearPackageData" to "true")
         resourceConfigurations += listOf("en", "es", "ru", "fr", "nl", "pt", "de")
-
-        buildConfigField(
-            "String",
-            "ACRA_HOST",
-            System.getenv("ACRA_HOST")?.let { "\"$it\"" } ?: "\"\"")
-        buildConfigField(
-            "String",
-            "ACRA_LOGIN",
-            System.getenv("ACRA_LOGIN")?.let { "\"$it\"" } ?: "\"\"")
-        buildConfigField(
-            "String",
-            "ACRA_PASSWORD",
-            System.getenv("ACRA_PASSWORD")?.let { "\"$it\"" } ?: "\"\"")
     }
 
     signingConfigs {
@@ -152,9 +139,6 @@ dependencies {
     implementation(libs.coil)
     implementation(libs.coil.compose)
 
-    implementation(libs.acra.http)
-    implementation(libs.acra.scheduler)
-
     testImplementation(libs.junit)
 
     implementation(libs.jetbrains.kotlinx.coroutinesAndroid)

app/src/main/java/gq/kirmanak/mealient/App.kt

@@ -1,14 +1,12 @@
 package gq.kirmanak.mealient
 
 import android.app.Application
-import android.content.Context
 import coil.Coil
 import coil.ImageLoader
 import com.google.android.material.color.DynamicColors
 import dagger.hilt.android.HiltAndroidApp
 import gq.kirmanak.mealient.architecture.configuration.BuildConfiguration
 import gq.kirmanak.mealient.data.migration.MigrationDetector
-import gq.kirmanak.mealient.extensions.setupCrashReporting
 import gq.kirmanak.mealient.logging.Logger
 import kotlinx.coroutines.CoroutineScope
 import kotlinx.coroutines.Dispatchers
@@ -33,11 +31,6 @@ class App : Application() {
 
     private val appCoroutineScope = CoroutineScope(Dispatchers.Main + Job())
 
-    override fun attachBaseContext(base: Context?) {
-        super.attachBaseContext(base)
-        setupCrashReporting()
-    }
-
     override fun onCreate() {
         super.onCreate()
         logger.v { "onCreate() called" }

app/src/main/java/gq/kirmanak/mealient/extensions/ContextExtensions.kt

@@ -1,16 +1,8 @@
 package gq.kirmanak.mealient.extensions
 
 import android.app.Activity
-import android.app.Application
-import android.app.job.JobInfo
 import android.content.Context
 import android.content.ContextWrapper
-import gq.kirmanak.mealient.BuildConfig
-import org.acra.config.httpSender
-import org.acra.config.scheduler
-import org.acra.data.StringFormat
-import org.acra.ktx.initAcra
-import org.acra.sender.HttpSender
 
 fun Context.findActivity(): Activity? {
     var context = this
@@ -20,27 +12,3 @@ fun Context.findActivity(): Activity? {
     }
     return null
 }
-
-internal fun Application.setupCrashReporting() {
-    val acraHost = BuildConfig.ACRA_HOST.takeUnless { it.isBlank() } ?: return
-    val acraLogin = BuildConfig.ACRA_LOGIN.takeUnless { it.isBlank() } ?: return
-    val acraPassword = BuildConfig.ACRA_PASSWORD.takeUnless { it.isBlank() } ?: return
-    initAcra {
-        reportFormat = StringFormat.JSON
-        alsoReportToAndroidFramework = true
-
-        httpSender {
-            uri = "$acraHost/report"
-            basicAuthLogin = acraLogin
-            basicAuthPassword = acraPassword
-            httpMethod = HttpSender.Method.POST
-            // TODO compressed reports are failing due to https://github.com/F43nd1r/Acrarium/issues/458
-            compress = false
-        }
-
-        scheduler {
-            requiresNetworkType = JobInfo.NETWORK_TYPE_UNMETERED
-            requiresBatteryNotLow = true
-        }
-    }
-}