add LOGGER

Author: frankois944

KtorKMPFileCaching/src/browserStorageSystemMain/kotlin/fr/frankois944/ktorfilecaching/KtorFileCaching.localStorageSystemMain.kt

@@ -6,6 +6,8 @@ import io.ktor.client.plugins.cache.storage.CacheStorage
 import io.ktor.client.plugins.cache.storage.CachedResponseData
 import io.ktor.http.Url
 import io.ktor.util.collections.ConcurrentMap
+import io.ktor.util.logging.KtorSimpleLogger
+import io.ktor.util.logging.trace
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.coroutineScope
 import kotlinx.coroutines.launch
@@ -46,6 +48,9 @@ internal class FileCacheStorage(
 ) : CacheStorage {
     private val mutexes = ConcurrentMap<String, Mutex>()
 
+    @Suppress("ktlint:standard:property-naming")
+    private val LOGGER = KtorSimpleLogger("KtorFileCaching")
+
     override suspend fun store(
         url: Url,
         data: CachedResponseData,
@@ -115,8 +120,7 @@ internal class FileCacheStorage(
                 if (Database.getItem("${prefix}_$urlHex") == null) return@withLock
                 Database.removeItem("${prefix}_$urlHex")
             } catch (cause: Exception) {
-                println("Exception during cache deletion in a file: ${cause.stackTraceToString()}")
-                //  LOGGER.trace { "Exception during cache deletion in a file: ${cause.stackTraceToString()}" }
+                LOGGER.trace { "Exception during cache deletion in a file: ${cause.stackTraceToString()}" }
             }
         }
     }
@@ -129,8 +133,7 @@ internal class FileCacheStorage(
             val serializedData = Cbor.encodeToHexString(caches.map { SerializableCachedResponseData(it) })
             Database.setItem("${prefix}_$urlHex", serializedData)
         } catch (cause: Exception) {
-            // LOGGER.trace { "Exception during saving a cache to a file: ${cause.stackTraceToString()}" }
-            println("Exception during saving a cache to a file: ${cause.stackTraceToString()}")
+            LOGGER.trace { "Exception during saving a cache to a file: ${cause.stackTraceToString()}" }
         }
     }
 
@@ -139,8 +142,7 @@ internal class FileCacheStorage(
             val item = Database.getItem("${prefix}_$urlHex") ?: return emptySet()
             Cbor.decodeFromHexString<Set<SerializableCachedResponseData>>(item).map { it.cachedResponseData }.toSet()
         } catch (cause: Exception) {
-            // LOGGER.trace { "Exception during cache lookup in a file: ${cause.stackTraceToString()}" }
-            println("Exception during cache lookup in a file: ${cause.stackTraceToString()}")
+            LOGGER.trace { "Exception during cache lookup in a file: ${cause.stackTraceToString()}" }
             emptySet()
         }
     }

KtorKMPFileCaching/src/okioFileSystemMain/kotlin/fr/frankois944/ktorfilecaching/KtorFileCaching.okioFileSystem.kt

@@ -6,6 +6,8 @@ import io.ktor.client.plugins.cache.storage.CacheStorage
 import io.ktor.client.plugins.cache.storage.CachedResponseData
 import io.ktor.http.Url
 import io.ktor.util.collections.ConcurrentMap
+import io.ktor.util.logging.KtorSimpleLogger
+import io.ktor.util.logging.trace
 import kotlinx.coroutines.CoroutineDispatcher
 import kotlinx.coroutines.Dispatchers
 import kotlinx.coroutines.coroutineScope
@@ -43,6 +45,9 @@ internal class FileCacheStorage(
     private val mutexes = ConcurrentMap<String, Mutex>()
     private val baseDir = "$directoryPath${Path.DIRECTORY_SEPARATOR}$storedCacheDirectory"
 
+    @Suppress("ktlint:standard:property-naming")
+    private val LOGGER = KtorSimpleLogger("KtorFileCaching")
+
     init {
         fileSystem.createDirectories(baseDir.toPath())
     }
@@ -117,8 +122,7 @@ internal class FileCacheStorage(
                 if (!fileSystem.exists(filePath)) return@withLock
                 fileSystem.delete(filePath)
             } catch (cause: Exception) {
-                println("Exception during cache deletion in a file: ${cause.stackTraceToString()}")
-                //  LOGGER.trace { "Exception during cache deletion in a file: ${cause.stackTraceToString()}" }
+                LOGGER.trace { "Exception during cache deletion in a file: ${cause.stackTraceToString()}" }
             }
         }
     }
@@ -134,8 +138,7 @@ internal class FileCacheStorage(
                 fileSystem.write(filePath) { write(serializedData) }
             }
         } catch (cause: Exception) {
-            // LOGGER.trace { "Exception during saving a cache to a file: ${cause.stackTraceToString()}" }
-            println("Exception during saving a cache to a file: ${cause.stackTraceToString()}")
+            LOGGER.trace { "Exception during saving a cache to a file: ${cause.stackTraceToString()}" }
         }
     }
 
@@ -146,8 +149,7 @@ internal class FileCacheStorage(
             val bytes = fileSystem.read(filePath) { readByteArray() }
             Cbor.decodeFromByteArray<Set<SerializableCachedResponseData>>(bytes).map { it.cachedResponseData }.toSet()
         } catch (cause: Exception) {
-            // LOGGER.trace { "Exception during cache lookup in a file: ${cause.stackTraceToString()}" }
-            println("Exception during cache lookup in a file: ${cause.stackTraceToString()}")
+            LOGGER.trace { "Exception during cache lookup in a file: ${cause.stackTraceToString()}" }
             emptySet()
         }
     }