Skip to content

okhttp version 3.12.12 - #29741

Closed
halaei wants to merge 1 commit into
react:masterfrom
halaei:update-okhttp
Closed

okhttp version 3.12.12#29741
halaei wants to merge 1 commit into
react:masterfrom
halaei:update-okhttp

Conversation

@halaei

Copy link
Copy Markdown
Contributor

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-botreact-native-bot added the Platform: Android Android applications. label Aug 24, 2020
@facebook-github-botfacebook-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

analysis-bot commented Aug 24, 2020

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
androidhermesarm64-v8a7,004,557-204,276
androidhermesarmeabi-v7a6,668,155-189,854
androidhermesx867,424,850-218,615
androidhermesx86_647,315,761-218,652
androidjscarm64-v8a9,164,310-203,244
androidjscarmeabi-v7a8,820,038-188,812
androidjscx869,012,712-217,558
androidjscx86_649,589,785-217,629

Base commit: 871e14f

@analysis-bot

analysis-bot commented Aug 24, 2020

Copy link
Copy Markdown
PlatformEngineArchSize (bytes)Diff
ios-universaln/a--

Base commit: 871e14f

@dulmandakh

dulmandakh commented Aug 28, 2020

Copy link
Copy Markdown
Contributor

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

@dulmandakhdulmandakh left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
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

@dulmandakhdulmandakh mentioned this pull request Aug 28, 2020

@mdvaccamdvacca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@facebook-github-botfacebook-github-bot left a comment

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Collaborator

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

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

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

Copy link
Copy Markdown

(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
Copy Markdown
ContributorAuthor

@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
Copy Markdown
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
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.Platform: AndroidAndroid applications.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants

@halaei@analysis-bot@dulmandakh@react-native-bot@jgreen210@mikehardy@mdvacca@facebook-github-bot