Fixed HTTP Logging interceptor to show all request content

jonathanarodr · Jul 23, 2024 · 2e0ff83 · 2e0ff83
1 parent 431d95e
commit 2e0ff83
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 7 deletions.
diff --git a/network/src/main/kotlin/br/com/jonathanarodr/playmovie/network/builder/OkHttpBuilder.kt b/network/src/main/kotlin/br/com/jonathanarodr/playmovie/network/builder/OkHttpBuilder.kt
@@ -1,7 +1,9 @@
 package br.com.jonathanarodr.playmovie.network.builder
 
+import br.com.jonathanarodr.playmovie.network.BuildConfig
 import okhttp3.Interceptor
 import okhttp3.OkHttpClient
+import okhttp3.logging.HttpLoggingInterceptor
 import java.util.concurrent.TimeUnit
 
 class OkHttpBuilder(
@@ -18,6 +20,16 @@ class OkHttpBuilder(
         }
     }
 
+    private fun OkHttpClient.Builder.addHttpLoggingInterceptor() {
+        if (BuildConfig.DEBUG) {
+            addNetworkInterceptor(
+                HttpLoggingInterceptor().apply {
+                    level = HttpLoggingInterceptor.Level.BODY
+                }
+            )
+        }
+    }
+
     private fun OkHttpClient.Builder.setTimeouts() {
         connectTimeout(CLIENT_TIMEOUT, TimeUnit.SECONDS)
             .writeTimeout(CLIENT_TIMEOUT, TimeUnit.SECONDS)
@@ -26,6 +38,7 @@ class OkHttpBuilder(
 
     fun build(): OkHttpClient {
         return OkHttpClient.Builder().apply {
+            addHttpLoggingInterceptor()
             addInterceptors()
             setTimeouts()
         }.build()

diff --git a/network/src/main/kotlin/br/com/jonathanarodr/playmovie/network/di/NetworkModule.kt b/network/src/main/kotlin/br/com/jonathanarodr/playmovie/network/di/NetworkModule.kt
@@ -4,18 +4,11 @@ import br.com.jonathanarodr.playmovie.network.builder.OkHttpBuilder
 import br.com.jonathanarodr.playmovie.network.builder.RetrofitBuilder
 import br.com.jonathanarodr.playmovie.network.interceptor.AuthorizationInterceptor
 import okhttp3.Interceptor
-import okhttp3.logging.HttpLoggingInterceptor
 import org.koin.dsl.bind
 import org.koin.dsl.module
 
 val networkModule = module {
 
-    factory {
-        HttpLoggingInterceptor().apply {
-            level = HttpLoggingInterceptor.Level.BODY
-        }
-    } bind Interceptor::class
-
     factory { AuthorizationInterceptor() } bind Interceptor::class
 
     factory { OkHttpBuilder(getAll()) }