Skip to content

chore(deps): bump okhttp to v5.3.2#104

Open
marcindabrowski wants to merge 1 commit into
launchdarkly:mainfrom
marcindabrowski:deps/bump-okhttp-v5
Open

chore(deps): bump okhttp to v5.3.2#104
marcindabrowski wants to merge 1 commit into
launchdarkly:mainfrom
marcindabrowski:deps/bump-okhttp-v5

Conversation

@marcindabrowski
Copy link
Copy Markdown
Contributor

@marcindabrowski marcindabrowski commented Apr 3, 2026

v5.3.2 is the latest version of OkHttp.
Version 5 of OkHttp works on Android 5.0+ (API level 21+) and Java 8+.


Note

Medium Risk
Upgrading the core HTTP client dependency can introduce behavioral/compatibility changes at runtime even though the diff is a single version bump.

Overview
Updates the OkHttp dependency version in build.gradle.kts from 4.12.0 to 5.3.2, so consumers of this library will build and run against OkHttp v5.

Written by Cursor Bugbot for commit 03de884. This will update automatically on new commits. Configure here.

v5.3.2 is the latest version of okhttp
Copy link
Copy Markdown

@cursor cursor Bot left a comment

Choose a reason for hiding this comment

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

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Fix All in Cursor

Comment thread build.gradle.kts
object Versions {
const val launchdarklyLogging = "1.1.1"
const val okhttp = "4.12.0"
const val okhttp = "5.3.2"
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Kotlin plugin version not updated to match OkHttp 5

High Severity

Bumping OkHttp to 5.3.2 without updating the Kotlin plugin version from 1.6.10 (line 21) creates a major version mismatch. OkHttp 5.3.2 depends on kotlin-stdlib 2.2.21 and requires Kotlin 2.2.0+ to read its metadata. The build file's own comment (lines 32–34) explicitly warns that the Kotlin plugin version must match OkHttp's Kotlin version to avoid extra Kotlin runtime dependencies in the published pom. This mismatch may cause build failures or incorrect transitive dependencies for consumers of this library.

Fix in Cursor Fix in Web

@tanderson-ld
Copy link
Copy Markdown
Contributor

Hi, thank you for submitting this PR. What is the main driver for updating to v5?

We may have to consider how this will roll out with many other dependencies customers may be using.

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.

2 participants