search

open-telemetry / opentelemetry-java-contrib

https://opentelemetry.io
Apache License 2.0
145 stars 118 forks source link

Remove okhttp3 as dependency #1351 #1364

Open driverpt opened 1 week ago

driverpt commented 1 week ago

Description: Removed Okhttp3 dependency. Okhttp3 brings Kotlin into the classpath and some of us do not want to bring Kotlin into the classpath as it creates "dependency hell"

Existing Issue(s):

1351

Testing: Tested Locally (No EKS Scenario) with OTLP Exporter

Documentation: N/A

Outstanding items: If someone could please test EKSResource and SSLSocketFactory, would be appreciated.

trask commented 3 days ago

hi @driverpt! check out the discussion in #1061

also, it may be worth waiting for https://github.com/open-telemetry/opentelemetry-java/pull/6546 to land and get released (cc @SylvainJuge)

driverpt commented 2 days ago

Is there an ETA when this can be reviewed and possibly merged? This is causing us some classpath hell in Micronaut + GraalVM

trask commented 2 days ago

Is there an ETA when this can be reviewed and possibly merged?

are you asking about this PR, or https://github.com/open-telemetry/opentelemetry-java/pull/6546? were you able to read through the concerns expressed in https://github.com/open-telemetry/opentelemetry-java-contrib/pull/1061? thanks

driverpt commented 2 days ago

This PR