Skip to content

Commit

Permalink
replace server: netty -> ktorcio, replace client: apache -> okhttp
Browse files Browse the repository at this point in the history
  • Loading branch information
bartekpacia committed Sep 18, 2023
1 parent 521baf7 commit 4495cf0
Show file tree
Hide file tree
Showing 5 changed files with 29 additions and 28 deletions.
6 changes: 3 additions & 3 deletions packages/patrol/android/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -66,9 +66,9 @@ android {
api "androidx.test.uiautomator:uiautomator:2.2.0"

implementation platform("org.http4k:http4k-bom:5.7.4.0")
implementation "org.http4k:http4k-core:5.7.4.0"
implementation "org.http4k:http4k-client-apache:5.7.4.0"
implementation "org.http4k:http4k-server-netty:5.7.4.0"
implementation "org.http4k:http4k-core"
implementation "org.http4k:http4k-client-okhttp"
implementation "org.http4k:http4k-server-ktorcio"
implementation "org.jetbrains.kotlinx:kotlinx-serialization-json:1.6.0"
implementation "org.jetbrains.kotlinx:kotlinx-coroutines-guava:1.5.2"

Expand Down
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
package pl.leancode.patrol

import org.apache.hc.core5.util.Timeout
import okio.Timeout
import pl.leancode.patrol.contracts.Contracts
import pl.leancode.patrol.contracts.PatrolAppServiceClientException
import pl.leancode.patrol.contracts.PatrolAppServiceClient as Client
Expand All @@ -13,15 +13,15 @@ class PatrolAppServiceClient {
private var client: Client

// https://github.com/leancodepl/patrol/issues/1683
private val timeout = Timeout.ofHours(2)
// private val timeout = Timeout.ofHours(2)

constructor() {
client = Client(address = "localhost", port = 8082, timeout = timeout)
client = Client(address = "localhost", port = 8082)
Logger.i("Created PatrolAppServiceClient: ${client.serverUrl}")
}

constructor(address: String) {
client = Client(address = address, port = 8082, timeout = timeout)
client = Client(address = address, port = 8082)
Logger.i("Created PatrolAppServiceClient: ${client.serverUrl}")
}

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import com.google.common.util.concurrent.SettableFuture
import org.http4k.core.ContentType
import org.http4k.filter.ServerFilters
import org.http4k.server.Http4kServer
import org.http4k.server.Netty
import org.http4k.server.KtorCIO
import org.http4k.server.asServer
import java.util.concurrent.Future

Expand All @@ -29,7 +29,7 @@ class PatrolServer {
.withFilter(catcher)
.withFilter(printer)
.withFilter(ServerFilters.SetContentType(ContentType.TEXT_PLAIN))
.asServer(Netty(port))
.asServer(KtorCIO(port))
.start()

Logger.i("Created and started PatrolServer, port: $port")
Expand Down

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

10 changes: 5 additions & 5 deletions packages/patrol/example/android/app/build.gradle
Original file line number Diff line number Diff line change
Expand Up @@ -34,11 +34,11 @@ android {
}

//TODO verify
packagingOptions {
pickFirst "META-INF/INDEX.LIST"
pickFirst "META-INF/io.netty.versions.properties"
pickFirst "META-INF/DEPENDENCIES"
}
// packagingOptions {
// pickFirst "META-INF/INDEX.LIST"
// pickFirst "META-INF/io.netty.versions.properties"
// pickFirst "META-INF/DEPENDENCIES"
// }

sourceSets {
main.java.srcDirs += "src/main/kotlin"
Expand Down

0 comments on commit 4495cf0

Please sign in to comment.