Skip to content
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

supports cronet #318

Open
wants to merge 1 commit into
base: minor
Choose a base branch
from
Open

supports cronet #318

wants to merge 1 commit into from

Conversation

MichealReed
Copy link
Contributor

We should be using cronet for streaming use cases, it will fail and respond much faster than OkHttp. I am not sure what we would have to do to support Apple with this, but I know it is possible. Exoplayer does make this easier on android though from what I can tell. This will also give us support for HTTP3/QUIC if the server supports, but Cronet falls back to http 1.1 fine in my tests.

@MichealReed
Copy link
Contributor Author

https://medium.com/the-react-native-log/using-cronet-in-your-mobile-app-7dda3a89c132

May be able to use the approach here, not familiar enough with Apple native development to make a call.

@ryanheise
Copy link
Owner

This looks interesting. Would it break HTTP 0.9 streams? Would it best be implemented as an option?

@MichealReed
Copy link
Contributor Author

It should still support 0.9, 1.0, 1.1, 2.0, and 3. QUIC should be revealed as an option, but I feel cronet is safer and better to use than okhttp out of the box.

@tiloc
Copy link

tiloc commented Sep 11, 2022

FWIW: Dart 2.18 is now supporting 2 platform specific implementations for http, one optimised for iOS, one optimised for Android (based on cronet). Both still considered a preview.

https://medium.com/dartlang/dart-2-18-f4b3101f146c

@ryanheise
Copy link
Owner

Interesting, so we could implement that for the proxy code (which is written in dart) while also using the ExoPlayer code for the native side.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants