feat: lower log level for overly frequent logs

Author: ovitrif

app/src/main/java/to/bitkit/data/BlocktankHttpClient.kt

@@ -16,7 +16,7 @@ class BlocktankHttpClient @Inject constructor(
 ) {
     suspend fun fetchLatestRates(): FxRateResponse {
         val response = client.get(Env.btcRatesServer)
-        Logger.debug("Http call: $response")
+        Logger.verbose("Http call: $response")
 
         return when (response.status.isSuccess()) {
             true -> response.body()

app/src/main/java/to/bitkit/data/backup/VssBackupClient.kt

@@ -25,7 +25,7 @@ class VssBackupClient @Inject constructor(
     suspend fun setup() = withContext(bgDispatcher) {
         try {
             withTimeout(30.seconds) {
-                Logger.debug("VSS client setting up…", context = TAG)
+                Logger.verbose("VSS client setting up…", context = TAG)
                 vssNewClient(
                     baseUrl = Env.vssServerUrl,
                     storeId = vssStoreIdProvider.getVssStoreId(),
@@ -44,34 +44,34 @@ class VssBackupClient @Inject constructor(
         data: ByteArray,
     ): Result<VssItem> = withContext(bgDispatcher) {
         isSetup.await()
-        Logger.debug("VSS 'putObject' call for '$key'", context = TAG)
+        Logger.verbose("VSS 'putObject' call for '$key'", context = TAG)
         runCatching {
             vssStore(
                 key = key,
                 value = data,
             )
         }.onSuccess {
-            Logger.debug("VSS 'putObject' success for '$key' at version: ${it.version}", context = TAG)
+            Logger.verbose("VSS 'putObject' success for '$key' at version: ${it.version}", context = TAG)
         }.onFailure { e ->
-            Logger.error("VSS 'putObject' error for '$key'", e = e, context = TAG)
+            Logger.verbose("VSS 'putObject' error for '$key'", e = e, context = TAG)
         }
     }
 
     suspend fun getObject(key: String): Result<VssItem?> = withContext(bgDispatcher) {
         isSetup.await()
-        Logger.debug("VSS 'getObject' call for '$key'", context = TAG)
+        Logger.verbose("VSS 'getObject' call for '$key'", context = TAG)
         runCatching {
             vssGet(
                 key = key,
             )
         }.onSuccess {
             if (it == null) {
-                Logger.warn("VSS 'getObject' success null for '$key'", context = TAG)
+                Logger.verbose("VSS 'getObject' success null for '$key'", context = TAG)
             } else {
-                Logger.debug("VSS 'getObject' success for '$key'", context = TAG)
+                Logger.verbose("VSS 'getObject' success for '$key'", context = TAG)
             }
         }.onFailure { e ->
-            Logger.error("VSS 'getObject' error for '$key'", e = e, context = TAG)
+            Logger.verbose("VSS 'getObject' error for '$key'", e = e, context = TAG)
         }
     }
 

app/src/main/java/to/bitkit/repositories/BackupRepo.kt

@@ -153,15 +153,15 @@ class BackupRepo @Inject constructor(
             cacheStore.updateBackupStatus(category) {
                 it.copy(required = System.currentTimeMillis())
             }
-            Logger.debug("Marked backup required for: '$category'", context = TAG)
+            Logger.verbose("Marked backup required for: '$category'", context = TAG)
         }
     }
 
     private fun scheduleBackup(category: BackupCategory) {
         // Cancel existing backup job for this category
         backupJobs[category]?.cancel()
 
-        Logger.debug("Scheduling backup for: '$category'", context = TAG)
+        Logger.verbose("Scheduling backup for: '$category'", context = TAG)
 
         backupJobs[category] = scope.launch {
             delay(BACKUP_DEBOUNCE)

app/src/main/java/to/bitkit/repositories/BlocktankRepo.kt

@@ -116,7 +116,7 @@ class BlocktankRepo @Inject constructor(
         isRefreshing = true
 
         try {
-            Logger.debug("Refreshing blocktank orders…", context = TAG)
+            Logger.verbose("Refreshing blocktank orders…", context = TAG)
 
             val paidOrderIds = cacheStore.data.first().paidOrders.keys
 
@@ -142,10 +142,7 @@ class BlocktankRepo @Inject constructor(
                 )
             }
 
-            Logger.debug(
-                "Orders refreshed: ${orders.size} orders, ${cjitEntries.size} cjit entries",
-                context = TAG
-            )
+            Logger.debug("Orders refreshed: ${orders.size} orders, ${cjitEntries.size} cjit entries", context = TAG)
         } catch (e: Throwable) {
             Logger.error("Failed to refresh orders", e, context = TAG)
         } finally {

app/src/main/java/to/bitkit/repositories/WidgetsRepo.kt

@@ -145,10 +145,10 @@ class WidgetsRepo @Inject constructor(
         service.fetchData()
             .onSuccess { data ->
                 updateStore(data)
-                Logger.debug("Updated $widgetType widget successfully")
+                Logger.verbose("Updated $widgetType widget successfully")
             }
-            .onFailure { error ->
-                Logger.warn(e = error, msg = "Failed to update $widgetType widget", context = TAG)
+            .onFailure { e ->
+                Logger.verbose("Failed to update $widgetType widget", e = e, context = TAG)
             }
 
         _refreshStates.update { it + (widgetType to false) }

app/src/main/java/to/bitkit/utils/Logger.kt

@@ -18,7 +18,7 @@ import java.util.concurrent.Executors
 
 object Logger {
     private const val TAG = "APP"
-    private const val COMPACT = false
+    private const val COMPACT = true
 
     private val singleThreadDispatcher = Executors
         .newSingleThreadExecutor { Thread(it, "bitkit.log").apply { priority = Thread.NORM_PRIORITY - 1 } }
@@ -91,12 +91,13 @@ object Logger {
 
     fun verbose(
         msg: String?,
+        e: Throwable? = null,
         context: String = "",
         file: String = getCallerFile(),
         line: Int = getCallerLine(),
     ) {
         val message = format("VERBOSE: $msg", context, file, line)
-        Log.v(TAG, message)
+        if (COMPACT) Log.v(TAG, message) else Log.v(TAG, message, e)
         saveToFile(message)
     }