Merge pull request #208 from lightsparkdev/fix/explicittimeout

Add explicit timeouts to the Requester
lightsparkdev · Sep 23, 2024 · 8ad4c0b · 8ad4c0b
2 parents 1fffb04 + 0427a3f
commit 8ad4c0b
Showing 1 changed file with 7 additions and 0 deletions.
diff --git a/core/src/commonMain/kotlin/com/lightspark/sdk/core/requester/Requester.kt b/core/src/commonMain/kotlin/com/lightspark/sdk/core/requester/Requester.kt
@@ -10,6 +10,7 @@ import com.lightspark.sdk.core.crypto.NodeKeyCache
 import com.lightspark.sdk.core.crypto.nextLong
 import com.lightspark.sdk.core.util.getPlatform
 import io.ktor.client.HttpClient
+import io.ktor.client.plugins.HttpTimeout
 import io.ktor.client.plugins.compression.ContentEncoding
 import io.ktor.client.plugins.websocket.WebSockets
 import io.ktor.client.request.headers
@@ -52,6 +53,12 @@ class Requester constructor(
             gzip()  // Switch to deflate() when https://youtrack.jetbrains.com/issue/KTOR-6999 is fixed
         }
         install(WebSockets)
+        install(HttpTimeout) {
+            // See https://ktor.io/docs/client-timeout.html for more info.
+            requestTimeoutMillis = 20000
+            connectTimeoutMillis = 5000
+            socketTimeoutMillis = 10000
+        }
     }
     private val userAgent =
         "lightspark-kotlin-sdk/${LightsparkCoreConfig.VERSION} ${getPlatform().platformName}/${getPlatform().version}"