Properly clean up child jobs on ICE reconnect timeout (#870)

Author: davidliu

.changeset/metal-mirrors-arrive.md

@@ -0,0 +1,5 @@
+---
+"client-sdk-android": patch
+---
+
+Ensure child jobs are cleaned up on ICE reconnect timeout

livekit-android-sdk/src/main/java/io/livekit/android/room/RTCEngine.kt

@@ -1,5 +1,5 @@
 /*
- * Copyright 2023-2025 LiveKit, Inc.
+ * Copyright 2023-2026 LiveKit, Inc.
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -620,20 +620,20 @@ internal constructor(
                     break
                 }
 
-                // wait until publisher ICE connected
-                var publisherWaitJob: Job? = null
-                if (hasPublished) {
-                    publisherWaitJob = launch {
-                        publisherObserver.waitUntilConnected()
+                withTimeoutOrNull(MAX_ICE_CONNECT_TIMEOUT_MS.toLong()) {
+                    // wait until publisher ICE connected
+                    var publisherWaitJob: Job? = null
+                    if (hasPublished) {
+                        publisherWaitJob = launch {
+                            publisherObserver.waitUntilConnected()
+                        }
                     }
-                }
 
-                // wait until subscriber ICE connected
-                val subscriberWaitJob = launch {
-                    subscriberObserver.waitUntilConnected()
-                }
+                    // wait until subscriber ICE connected
+                    val subscriberWaitJob = launch {
+                        subscriberObserver.waitUntilConnected()
+                    }
 
-                withTimeoutOrNull(MAX_ICE_CONNECT_TIMEOUT_MS.toLong()) {
                     listOfNotNull(publisherWaitJob, subscriberWaitJob)
                         .joinAll()
                 }