Ensure publication isn't attempted after timeout rejected the promise (#1725)

Author: lukasIO

.changeset/spicy-scissors-bake.md

@@ -0,0 +1,5 @@
+---
+"livekit-client": patch
+---
+
+Ensure publication isn't attempted after timeout rejected the promise

examples/demo/demo.ts

@@ -401,14 +401,16 @@ const appActions = {
           if (shouldPublish) {
             await room.localParticipant.enableCameraAndMicrophone();
             appendLog(`tracks published in ${Date.now() - startTime}ms`);
-            updateButtonsForPublishState();
           }
           resolve();
         } catch (error) {
           reject(error);
         }
       });
-      await Promise.all([room.connect(url, token, connectOptions), publishPromise]);
+      await Promise.all([
+        room.connect(url, token, connectOptions),
+        publishPromise.catch(appendLog),
+      ]);
       const elapsed = Date.now() - startTime;
       appendLog(
         `successfully connected to ${room.name} in ${Math.round(elapsed)}ms`,

src/room/participant/LocalParticipant.ts

@@ -882,7 +882,11 @@ export default class LocalParticipant extends Participant {
             track: getLogContextFromTrack(track),
           });
 
+          let publicationTimedOut = false;
+
           const timeout = setTimeout(() => {
+            publicationTimedOut = true;
+            track.stop();
             reject(
               new PublishTrackError(
                 'publishing rejected as engine not connected within timeout',
@@ -892,6 +896,9 @@ export default class LocalParticipant extends Participant {
           }, 15_000);
           await this.waitUntilEngineConnected();
           clearTimeout(timeout);
+          if (publicationTimedOut) {
+            return;
+          }
           const publication = await this.publish(track, opts, isStereo);
           resolve(publication);
         } else {