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

okhttp version 3.12.12 #29741

Closed
wants to merge 1 commit into from
Closed

okhttp version 3.12.12 #29741

wants to merge 1 commit into from

Conversation

halaei
Copy link
Contributor

@halaei halaei commented Aug 24, 2020

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.

@react-native-bot react-native-bot added the Platform: Android Android applications. label Aug 24, 2020
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Aug 24, 2020
@analysis-bot
Copy link

analysis-bot commented Aug 24, 2020

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 7,004,557 -204,276
android hermes armeabi-v7a 6,668,155 -189,854
android hermes x86 7,424,850 -218,615
android hermes x86_64 7,315,761 -218,652
android jsc arm64-v8a 9,164,310 -203,244
android jsc armeabi-v7a 8,820,038 -188,812
android jsc x86 9,012,712 -217,558
android jsc x86_64 9,589,785 -217,629

Base commit: 871e14f

@analysis-bot
Copy link

analysis-bot commented Aug 24, 2020

Platform Engine Arch Size (bytes) Diff
ios - universal n/a --

Base commit: 871e14f

@dulmandakh
Copy link
Contributor

dulmandakh commented Aug 28, 2020

please bump okhttp in Buck too, so that CI can use the change

Copy link
Contributor

@dulmandakh dulmandakh left a 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

@dulmandakh
Copy link
Contributor

@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

Copy link
Contributor

@mdvacca mdvacca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

Copy link
Contributor

@facebook-github-bot facebook-github-bot left a 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.

@react-native-bot
Copy link
Collaborator

This pull request was successfully merged by @halaei in 0f6fcb2.

When will my fix make it into a release? | Upcoming Releases

@react-native-bot react-native-bot added the Merged This PR has been merged. label Aug 29, 2020
@jgreen210
Copy link

(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.

@halaei
Copy link
Contributor Author

halaei commented Sep 4, 2020

@jgreen210 I am not sure, maybe okhttp 4.x doesn't work on Android < 5.0? React Native supports Android >=4.1.
https://github.com/square/okhttp#requirements

@mikehardy
Copy link
Contributor

@jgreen210 - @halaei is exactly right. react-native minSdkVersion is API16 so they must stay on the 3.12.x okhttp branch

kelset pushed a commit that referenced this pull request Nov 27, 2020
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
philippeauriach pushed a commit to philippeauriach/react-native that referenced this pull request May 5, 2021
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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. Merged This PR has been merged. Platform: Android Android applications.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants