close the APM client response body regardless of the response (#1224)

kamon-io · Nov 2, 2022 · 92f2e3a · 92f2e3a
1 parent 5b96b05
commit 92f2e3a
Showing 1 changed file with 4 additions and 0 deletions.
diff --git a/reporters/kamon-apm-reporter/src/main/scala/kamon/apm/ApiClient.scala b/reporters/kamon-apm-reporter/src/main/scala/kamon/apm/ApiClient.scala
@@ -73,6 +73,10 @@ class ApiClient(settings: Settings) {
 
     tryPosting match {
       case Success(response) =>
+        // The response body must be closed (even if there is no body in the response),
+        // otherwise OkHttp leaks connections.
+        response.body().close()
+
         response.code() match {
           case 200 =>
             _logger.trace("Request to the Kamon APM [{}] endpoint succeeded", endpointName)