Merge pull request #1214 from MariusVolkhart/okHttpBuilder

[IMPROVEMENT] Change OkHttpClient initialization in Dagger graph

Merge pull request #1214 from MariusVolkhart/okHttpBuilder
[IMPROVEMENT] Change OkHttpClient initialization in Dagger graph
607378a7 · Filipe de Lima Brito · GitHub · 10d7f944 · 9e8e6209 · 607378a7
Unverified Commit 607378a7 authored Apr 30, 2018 by Filipe de Lima Brito Committed by GitHub Apr 30, 2018
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 6 deletions

AppModule.kt .../main/java/chat/rocket/android/dagger/module/AppModule.kt +6 -6

No files found.
--- a/app/src/main/java/chat/rocket/android/dagger/module/AppModule.kt
+++ b/app/src/main/java/chat/rocket/android/dagger/module/AppModule.kt
@@ -111,12 +111,12 @@ class AppModule {
    @Provides
    @Singleton
    fun provideOkHttpClient(logger: HttpLoggingInterceptor): OkHttpClient {
-        return OkHttpClient.Builder().apply {
-            addInterceptor(logger)
-            connectTimeout(15, TimeUnit.SECONDS)
-            readTimeout(20, TimeUnit.SECONDS)
-            writeTimeout(15, TimeUnit.SECONDS)
-        }.build()
+        return OkHttpClient.Builder()
+                .addInterceptor(logger)
+                .connectTimeout(15, TimeUnit.SECONDS)
+                .readTimeout(20, TimeUnit.SECONDS)
+                .writeTimeout(15, TimeUnit.SECONDS)
+                .build()
    }

    @Provides