fix(samples): Close route work OkHttp responses

Close Retrofit response bodies for both synchronous and asynchronous route-work requests so repeated navigation cannot leak OkHttp connections.

Co-Authored-By: Codex <noreply@openai.com>

Author: 0xadam-brown

sentry-samples/sentry-samples-android/src/main/java/io/sentry/samples/android/navigation/common/RouteWorkApi.kt

@@ -32,12 +32,29 @@ internal object RouteWorkApi {
 
   private val service: RouteWorkService = retrofit.create(RouteWorkService::class.java)
 
-  fun enqueueRequest(callback: retrofit2.Callback<ResponseBody>) {
-    service.request(ROUTE_WORK_URL).enqueue(callback)
+  fun enqueueRequest(onSuccess: () -> Unit, onFailure: (Throwable) -> Unit) {
+    service
+      .request(ROUTE_WORK_URL)
+      .enqueue(
+        object : retrofit2.Callback<ResponseBody> {
+          override fun onResponse(
+            call: retrofit2.Call<ResponseBody>,
+            response: retrofit2.Response<ResponseBody>,
+          ) {
+            response.body()?.close()
+            response.errorBody()?.close()
+            onSuccess()
+          }
+
+          override fun onFailure(call: retrofit2.Call<ResponseBody>, t: Throwable) {
+            onFailure(t)
+          }
+        }
+      )
   }
 
   suspend fun runRequest() {
-    service.requestAsync(ROUTE_WORK_URL)
+    service.requestAsync(ROUTE_WORK_URL).use {}
   }
 }
 

sentry-samples/sentry-samples-android/src/main/java/io/sentry/samples/android/navigation/nav2/Nav2Activity.kt

@@ -45,10 +45,6 @@ import io.sentry.samples.android.navigation.common.hasOnlyActivityUiLoadTransact
 import io.sentry.samples.android.navigation.common.showRouteWorkDialog
 import io.sentry.samples.android.navigation.nav2.Nav2Destination.Home
 import io.sentry.samples.android.navigation.nav2.Nav2Destination.Landing
-import okhttp3.ResponseBody
-import retrofit2.Call
-import retrofit2.Callback
-import retrofit2.Response
 
 /**
  * Sample activity for testing Sentry's
@@ -396,16 +392,11 @@ class Nav2Activity : AppCompatActivity() {
       when (option) {
         RouteWorkOption.HTTP_REQUEST -> {
           RouteWorkApi.enqueueRequest(
-            object : Callback<ResponseBody> {
-              override fun onResponse(call: Call<ResponseBody>, response: Response<ResponseBody>) {
-                Thread { Sentry.flush(SENTRY_FLUSH_TIMEOUT_MILLIS) }.start()
-              }
-
-              override fun onFailure(call: Call<ResponseBody>, t: Throwable) {
-                Sentry.captureException(t)
-                Thread { Sentry.flush(SENTRY_FLUSH_TIMEOUT_MILLIS) }.start()
-              }
-            }
+            onSuccess = { Thread { Sentry.flush(SENTRY_FLUSH_TIMEOUT_MILLIS) }.start() },
+            onFailure = { t ->
+              Sentry.captureException(t)
+              Thread { Sentry.flush(SENTRY_FLUSH_TIMEOUT_MILLIS) }.start()
+            },
           )
         }