-
Notifications
You must be signed in to change notification settings - Fork 24.4k
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 version 3.12.12 #29741
okhttp version 3.12.12 #29741
Conversation
Base commit: 871e14f |
Base commit: 871e14f |
please bump okhttp in Buck too, so that CI can use the change |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mdvacca please review and merge
@mdvacca @fkgozali it looks like this PR would fix many crashes according to OkHTTP changelog, please merge it as soon as possible https://square.github.io/okhttp/changelog_3x/#version-3126 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@mdvacca has imported this pull request. If you are a Facebook employee, you can view this diff on Phabricator.
This pull request was successfully merged by @halaei in 0f6fcb2. When will my fix make it into a release? | Upcoming Releases |
(I'm a RN user trying to resolve an okhttp/okhttp-urlconnection version conflict between react-native (which wants a 3.x version) and another library (which wants a 4.x version). I need to decide whether to force use of 3.x or 4.x in our app, with both choices potentially breaking something, based on semver.) So, is someone able to elaborate on why the "newest compatible version" of okhttp etc. isn't a 4.x version? I've checked the open github issues mentioning "okhttp", plus the recent closed ones, and while there is #27282, it doesn't explain what problems exist with 4.x. |
@jgreen210 I am not sure, maybe okhttp 4.x doesn't work on Android < 5.0? React Native supports Android >=4.1. |
@jgreen210 - @halaei is exactly right. react-native minSdkVersion is API16 so they must stay on the 3.12.x okhttp branch |
Summary: This updates okhttp to the newest compatible version with a couple of fixes and improvements. See https://github.com/square/okhttp/commits/okhttp_3.12.x ## Changelog [Android] [Changed] - Update Okhttp to version 3.12.12 Pull Request resolved: #29741 Test Plan: Current tests should pass. Reviewed By: shergin Differential Revision: D23406613 Pulled By: mdvacca fbshipit-source-id: b0b4ec52a6a8345f1c36e18e384761386096f1d8
Summary: This updates okhttp to the newest compatible version with a couple of fixes and improvements. See https://github.com/square/okhttp/commits/okhttp_3.12.x ## Changelog [Android] [Changed] - Update Okhttp to version 3.12.12 Pull Request resolved: facebook#29741 Test Plan: Current tests should pass. Reviewed By: shergin Differential Revision: D23406613 Pulled By: mdvacca fbshipit-source-id: b0b4ec52a6a8345f1c36e18e384761386096f1d8
Summary
This updates okhttp to the newest compatible version with a couple of fixes and improvements. See https://github.com/square/okhttp/commits/okhttp_3.12.x
Changelog
[Android] [Changed] - Update Okhttp to version 3.12.12
Test Plan
Current tests should pass.