-
Notifications
You must be signed in to change notification settings - Fork 9.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
OkHttp 5 blockers #7366
Comments
@swankjesse is Loom support in or out? #6385 Even just confirming it's supported, if not a distinct TaskRunner.Backend? |
I think Loom is out only because I don't expect it'll need API changes. |
I do think we'll need a new module with JDK 19+ dependencies, and probably change the implementation, potentially changes, maybe minor API changes, to TaskRunner. |
Going to mark #7393 as done based on confirmation of current set against Chrome. |
@swankjesse Adding back Loom as I think #7367 showed we need to remove wait()/notify() and use Conditions instead in okhttp and okio. |
Also for KMP, I'm looking at using okcurl as a cross platform CLI buildable with all of graal, native, js. #7416 |
HttpUrl supported, but without IDN. |
Suspend JS support is likely just the current example shown for ktor https://github.com/square/okhttp/pull/7326/files#diff-772553bc9e67a5b9842ab416490ff3fd80a8e7bd878ac641a9c0d644bce96492 Response will be read as part of the suspending executeAsync method. |
Is there any news on when we can expect OkHttp 5 to be released? |
#5874 (HTTP 1.1 Upgrades) would be great to be included. It might not be a release blocker, though. |
I suspect #7164 could be a 5.1 feature if not in 5.0 |
Suspend JS supportThe text was updated successfully, but these errors were encountered: