Skip to content

Add OkHttp transport#1284

Merged
bsideup merged 6 commits intomasterfrom
add_okhttp_transport
Dec 7, 2019
Merged

Add OkHttp transport#1284
bsideup merged 6 commits intomasterfrom
add_okhttp_transport

Conversation

@bsideup
Copy link
Copy Markdown
Member

@bsideup bsideup commented Dec 6, 2019

This is a backport of the OkHttp transport from http://github.com/testcontainers/testcontainers-java

OkHttp provides a lightweight API, supports the same feature set as Netty does but also has Windows Npipe support since it can use Java's Socket abstraction.


This change is Reviewable

@bsideup bsideup added this to the 3.2.0 milestone Dec 6, 2019
@gesellix
Copy link
Copy Markdown
Contributor

gesellix commented Dec 6, 2019

🎉

@bsideup bsideup merged commit 217c14b into master Dec 7, 2019
@bsideup bsideup deleted the add_okhttp_transport branch December 7, 2019 18:36
@jshook
Copy link
Copy Markdown

jshook commented Jan 23, 2020

I know this was merged up before 3.2.0-rc2, but the pain of managing netty dependency conflicts is real. Can we get the transport-okhttp added to an rc3 or a 3.3 pre release soonish?

@jshook
Copy link
Copy Markdown

jshook commented Jan 23, 2020

@KostyaSha Since you seem to be the release manager, can you give us an idea on whether this is feasible for an rc3?

@bsideup
Copy link
Copy Markdown
Member Author

bsideup commented Jan 23, 2020

@jshook we're working on rc3 and will release shortly after :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants