Do not setup content encoding for js and native

.github/workflows/build.yaml

@@ -36,15 +36,15 @@ jobs:
         set -eux
         ./gradlew compileTestKotlinJvm $GRADLE_ARGS
 
-    - name: Compile JS
+    - name: Compile native
       run: |
         set -eux
-        ./gradlew compileTestKotlinJs $GRADLE_ARGS
+        ./gradlew linkDebugTestNative $GRADLE_ARGS
 
-    - name: Compile native
+    - name: Compile JS
       run: |
         set -eux
-        ./gradlew linkDebugTestNative $GRADLE_ARGS
+        ./gradlew compileTestKotlinJs $GRADLE_ARGS
 
     - name: Compile samples
       run: |

elasticmagic-transport-ktor/src/commonMain/kotlin/dev/evo/elasticmagic/transport/ElasticsearchKtorTransport.kt

@@ -18,6 +18,8 @@ import io.ktor.http.contentType
 import io.ktor.http.path
 import io.ktor.http.takeFrom
 
+internal expect val setupContentEncoding: ContentEncoding.Config.() -> Unit
+
 class ElasticsearchKtorTransport(
     baseUrl: String,
     engine: HttpClientEngine,
@@ -30,11 +32,7 @@ class ElasticsearchKtorTransport(
         expectSuccess = false
 
         // Enable compressed response from Elasticsearch
-        install(ContentEncoding) {
-            gzip()
-            deflate()
-            identity()
-        }
+        install(ContentEncoding, setupContentEncoding)
 
         when (val auth = config.auth) {
             is dev.evo.elasticmagic.transport.Auth.Basic -> {

elasticmagic-transport-ktor/src/commonTest/kotlin/dev.evo.elasticmagic.transport/ElasticsearchKtorTransportTests.kt → elasticmagic-transport-ktor/src/commonTest/kotlin/dev/evo/elasticmagic/transport/ElasticsearchKtorTransportTests.kt

@@ -1,9 +1,11 @@
 package dev.evo.elasticmagic.transport
 
+import dev.evo.elasticmagic.serde.platform
+import dev.evo.elasticmagic.serde.Platform
+import dev.evo.elasticmagic.serde.toMap
 import dev.evo.elasticmagic.serde.kotlinx.JsonDeserializer
 import dev.evo.elasticmagic.serde.kotlinx.JsonSerde
 import dev.evo.elasticmagic.serde.kotlinx.JsonSerializer
-import dev.evo.elasticmagic.serde.toMap
 
 import io.kotest.assertions.throwables.shouldThrow
 import io.kotest.matchers.comparables.shouldBeGreaterThan
@@ -38,13 +40,19 @@ import kotlin.time.Duration
 
 @ExperimentalStdlibApi
 class ElasticsearchKtorTransportTests {
+    private val expectedAcceptEncoding = if (platform == Platform.JVM) {
+        "gzip,deflate,identity"
+    } else {
+        ""
+    }
+
     @Test
     fun headRequest() = runTest {
         val client = ElasticsearchKtorTransport(
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Head
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products"
                 request.body.contentType.shouldBeNull()
                 request.body.toByteArray().decodeToString().shouldBeEmpty()
@@ -74,7 +82,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Put
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products/_settings"
                 request.body.contentType shouldBe ContentType.Application.Json
                 request.body.toByteArray().decodeToString() shouldBe """{"index":{"number_of_replicas":2}}"""
@@ -113,7 +121,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Delete
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products_v2"
                 request.body.toByteArray().decodeToString().shouldBeEmpty()
 
@@ -144,7 +152,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Post
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/_bulk"
                 request.body.contentType shouldBe ContentType("application", "x-ndjson")
                 val rawBody = request.body.toByteArray().decodeToString()
@@ -210,7 +218,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Get
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/_cat/nodes"
                 request.body.contentType.shouldBeNull()
                 respond(
@@ -238,7 +246,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Post
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products_v2/_forcemerge"
                 request.body.toByteArray().decodeToString().shouldBeEmpty()
                 respondError(
@@ -266,7 +274,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Get
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products/_count"
                 request.body.contentType.shouldBeNull()
                 request.body.toByteArray().decodeToString().shouldBeEmpty()
@@ -322,7 +330,7 @@ class ElasticsearchKtorTransportTests {
             "http://example.com:9200",
             MockEngine { request ->
                 request.method shouldBe HttpMethod.Get
-                request.headers["accept-encoding"] shouldBe "gzip,deflate,identity"
+                request.headers["accept-encoding"] shouldBe expectedAcceptEncoding
                 request.url.encodedPath shouldBe "/products/_search"
                 request.body.contentType shouldBe ContentType.Application.Json
                 request.body.toByteArray().decodeToString() shouldBe expectedContent

elasticmagic-transport-ktor/src/jsMain/kotlin/dev/evo/elasticmagic/transport/js.kt

@@ -0,0 +1,5 @@
+package dev.evo.elasticmagic.transport
+
+import io.ktor.client.plugins.compression.ContentEncoding
+
+internal actual val setupContentEncoding: ContentEncoding.Config.() -> Unit = {}

elasticmagic-transport-ktor/src/jvmMain/kotlin/dev/evo/elasticmagic/transport/jvm.kt

@@ -0,0 +1,9 @@
+package dev.evo.elasticmagic.transport
+
+import io.ktor.client.plugins.compression.ContentEncoding
+
+internal actual val setupContentEncoding: ContentEncoding.Config.() -> Unit = {
+    gzip()
+    deflate()
+    identity()
+}

elasticmagic-transport-ktor/src/nativeMain/kotlin/dev/evo/elasticmagic/transport/native.kt

@@ -0,0 +1,5 @@
+package dev.evo.elasticmagic.transport
+
+import io.ktor.client.plugins.compression.ContentEncoding
+
+internal actual val setupContentEncoding: ContentEncoding.Config.() -> Unit = {}