opentelemetry-android icon indicating copy to clipboard operation
opentelemetry-android copied to clipboard

Okhttp 4.12 forces kotlin upgrade to 1.9 which might be problematic for some customers

Open surbhiia opened this issue 2 years ago • 3 comments

An issue was raised on same here.

Should this repo try to have two versions of okhttp auto-instrumentation -> okhttp3+ and okhttp4.12+ to cover all customers?

surbhiia avatar Dec 01 '23 17:12 surbhiia

One option is to force the okhttp auto-instrumentation to set apiVersion and languageVersion to at least 1.6 which is the oldest version not deprecated yet. https://kotlinlang.org/docs/gradle-compiler-options.html#attributes-common-to-jvm-and-js Maybe that works? not sure if it works when it pulls a compiled jar already or if the compiler is yet able to desugar everything newer, I guess the build would break at least if not supported.

marandaneto avatar Dec 04 '23 09:12 marandaneto

Indeed. This is a problem. Is there an issue like this on OkHttp's side somewhere?

theAkito avatar May 28 '24 05:05 theAkito

Indeed. This is a problem. Is there an issue like this on OkHttp's side somewhere?

Yes, we discussed this in this OkHttp issue.

LikeTheSalad avatar May 28 '24 07:05 LikeTheSalad