Use different ktor implementation for JS

Signed-off-by: Bence Hornák <[email protected]>

Author: bencehornak

gradle/libs.versions.toml

@@ -8,6 +8,7 @@ ktor = "3.1.3"
 [libraries]
 ktor-core = { module="io.ktor:ktor-client-core", version.ref="ktor" }
 ktor-cio = { module="io.ktor:ktor-client-cio", version.ref="ktor" }
+ktor-js = { module="io.ktor:ktor-client-js", version.ref="ktor" }
 ktor-client-content-negotiation = { module="io.ktor:ktor-client-content-negotiation", version.ref="ktor" }
 ktor-serialization-kotlinx-json = { module="io.ktor:ktor-serialization-kotlinx-json", version.ref="ktor" }
 ktor-client-mock = { module="io.ktor:ktor-client-mock", version.ref="ktor" }

providers/ofrep/build.gradle.kts

@@ -34,20 +34,39 @@ kotlin {
     }
 
     sourceSets {
-        commonMain.dependencies {
-            api(libs.openfeature.kotlin.sdk)
+        val commonMain by getting {
+            dependencies {
+                api(libs.openfeature.kotlin.sdk)
 
-            api(libs.kotlinx.coroutines.core)
-            api(libs.ktor.core)
-            implementation(libs.ktor.cio)
-            implementation(libs.ktor.client.content.negotiation)
-            implementation(libs.ktor.serialization.kotlinx.json)
+                api(libs.kotlinx.coroutines.core)
+                api(libs.ktor.core)
+                implementation(libs.ktor.client.content.negotiation)
+                implementation(libs.ktor.serialization.kotlinx.json)
+            }
         }
         commonTest.dependencies {
             implementation(libs.kotlin.test)
             implementation(libs.kotlinx.coroutines.test)
             implementation(libs.ktor.client.mock)
         }
+        val nonJsMain by creating {
+            dependsOn(commonMain)
+            dependencies {
+                implementation(libs.ktor.cio)
+            }
+        }
+        androidMain {
+            dependsOn(nonJsMain)
+        }
+        jvmMain {
+            dependsOn(nonJsMain)
+        }
+        linuxX64Main {
+            dependsOn(nonJsMain)
+        }
+        jsMain.dependencies {
+            implementation(libs.ktor.js)
+        }
     }
 }
 

providers/ofrep/src/commonMain/kotlin/dev/openfeature/kotlin/contrib/providers/ofrep/bean/Http.kt

@@ -2,19 +2,10 @@ package dev.openfeature.kotlin.contrib.providers.ofrep.bean
 
 import io.ktor.client.HttpClient
 import io.ktor.client.engine.HttpClientEngine
-import io.ktor.client.engine.cio.CIO
-import io.ktor.client.engine.cio.endpoint
 import io.ktor.client.plugins.contentnegotiation.ContentNegotiation
 import io.ktor.serialization.kotlinx.json.json
 
-private fun defaultHttpEngine(options: OfrepOptions): HttpClientEngine =
-    CIO.create {
-        maxConnectionsCount = options.maxIdleConnections
-        endpoint {
-            keepAliveTime = options.keepAliveDuration.inWholeMilliseconds
-            connectTimeout = options.timeout.inWholeMilliseconds
-        }
-    }
+internal expect fun defaultHttpEngine(options: OfrepOptions): HttpClientEngine
 
 internal fun createHttpClient(options: OfrepOptions): HttpClient {
     val httpEngine = options.httpClientEngine ?: defaultHttpEngine(options)

providers/ofrep/src/jsMain/kotlin/dev/openfeature/kotlin/contrib/providers/ofrep/bean/Http.js.kt

@@ -0,0 +1,12 @@
+package dev.openfeature.kotlin.contrib.providers.ofrep.bean
+
+import io.ktor.client.engine.HttpClientEngine
+import io.ktor.client.engine.js.Js
+
+internal actual fun defaultHttpEngine(options: OfrepOptions): HttpClientEngine =
+    Js.create {
+        // TODO: consider the following options:
+        //   - options.maxIdleConnections
+        //   - options.keepAliveDuration
+        //   - options.timeout
+    }

providers/ofrep/src/nonJsMain/kotlin/dev/openfeature/kotlin/contrib/providers/ofrep/bean/Http.nonJsMain.kt

@@ -0,0 +1,14 @@
+package dev.openfeature.kotlin.contrib.providers.ofrep.bean
+
+import io.ktor.client.engine.HttpClientEngine
+import io.ktor.client.engine.cio.CIO
+import io.ktor.client.engine.cio.endpoint
+
+internal actual fun defaultHttpEngine(options: OfrepOptions): HttpClientEngine =
+    CIO.create {
+        maxConnectionsCount = options.maxIdleConnections
+        endpoint {
+            keepAliveTime = options.keepAliveDuration.inWholeMilliseconds
+            connectTimeout = options.timeout.inWholeMilliseconds
+        }
+    }